When I open Godot it loads my application, not the editor or the project menu.

Operating system or device - Godot version:
Windows 10
Godot v2.0
Issue description (what happened, and what was expected):
What the title says
Steps to reproduce:
Create project, write script, save scene, play, stop playing, close Godot

This is not a bug, it's the intended way of working. If the executable find a engine.cfg on the same folder (or a data.pck/.zip) it runs the project. You need to move the executable out of the game folder or run Godot with flags (-pm for project manager or -e for editor).


Default pool lookup

This change is

docker: Clean packages after installation  why this is a part of this PR ?

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

@pigmej it's small change, something like typo so as it is in separate commit I'm good with this...


[ Application ] Update Logistic Regression to use CCAPI @open sesame 05/16 22:20

This patch includes
. Update Logistic Regression to use CCAPIs
. Update Inference process
. static link about openmp
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1910. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jijoongmoon, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1910-202205162016210.085525035858154-2c66392e0c40437f8fa51801e715f91090a88c45/.

:octocat: cibot: @jijoongmoon, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1910-202205162117540.02387809753418-799f4d466da42909835e674f010713eae1259009/.

:octocat: cibot: @jijoongmoon, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1910-202205162211290.013784885406494-799f4d466da42909835e674f010713eae1259009/.

Tested on device.
Both Training and Inference work.
Using ./tools/package_android.sh and then using the updated packaged libraries doesn't throw the libomp missing error too.
Thanks for the fix!


Fix MessageBuilder#resetFormatting & optimize MessageBuilder#fromString

Closes #85
This fixes the resetFormatting method of MessageBuilder not resetting the color correctly (1.16+). Also, while this was not necessary to fix the issue, the fromString static method was rewritten to output smaller components. Formatting no longer leaks into color codes either (which it should not, at least in the way the notchian client handles legacy colors).

I realize that this breaks text such as &aa&ll. I will try fixing it later.

too old


Release 5.5 on Carthage

Carthage only fetches 5.4 for now. Please publish it for Carthage. Unfortunately CocoaPods is too bloated to use these days.

@fotiDim Thanks for the request.
The Carthage distribution is still experimental and will occasionally lag the CocoaPods and zip releases. Currently, Carthage is still at 5.4.0 versus 5.5.0 for the official releases. I'm hoping to get Carthage back in sync within the next week.

A Carthage distribution for Firebase 5.6.0 has been published. See https://github.com/firebase/firebase-ios-sdk/blob/master/Carthage.md for usage details.


[Question] cls and reg loss of the TPN are same with loss layers of the Classification Head?

Hi, I have a question about the paper.
At Fig 2., It seems that losses of TPN and Classification Head are same.
If they are not, what is the purpose of Classification Head?

Hi, the losses are not exactly the same; they are comparable to how the losses in stage 1 (RPN) and stage 2 (classification/bounding box regression) in mask r-cnn are defined. I'd recommend looking at Mask R-CNN or Faster R-CNN for more details.

Thx.


btop: add rocm version

Description of changes

Things done

Built on platform(s)

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

[x] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4307

Why we expose rocm but not cuda? Is there a specific reason for this?

No particular reason, but since I only have AMD GPUs, that's why I added rocm. Anyway adding a cuda package won't be particularly useful because it won't be build by hydra.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4371

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4575

@SuperSandro2000

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4745


Feature/rails react

Summary of this PR:

Setup webpack for the to compile react.
Integrage Rails API with React application.
Create table to contain 5 greeting messages.
display a random message.
No linters errors.

Hi @M-Anwar-Hussaini,
Good job so far! There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights 💯

No linters errors ✔
Descriptive pull request ✔
Descriptive commit messages ✔

Required Changes ♻
Check the comments under the review.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


private settings in package.json

For publishing to npm, private must be set to false.
But workspaces to work properly, private must be set to true.
Lerna helps solve this problem for monorepos.
@Dhaiwat10 please assign this to me

@Dhaiwat10 so after looking into this a little bit further, Lerna is awesome for managing multiple packages under one repo.  Currently we only really only have one package in this repo.  I think the answer to this is a bit simpler.
I don't think we even need this line
 "private": "true",
https://github.com/Developer-DAO/create-dao/blob/0709815bf658dee5ae747c3591acea46003cb1e1/package.json#L4
I think we can just deleting will solve this issue.  Can you try this and test publishing?

@e-roy if I remove that line:
a) Yes, I can publish the package to npm.
b) I cannot install the dependencies by running yarn.
error Workspaces can only be enabled in private projects.

@Dhaiwat10 I see.
I've created a test for this here for you to see file structure and a working test example.  This has scripts to run for next development, version update, and npm publish.  This will require a huge change in the file structure, but will layout properly for future other packages to be held here in a monorepo.  It wouldn't take too long to do, but since it does have such a huge change, it might be good to clear up any PRs before starting.
The current structure is:
├── helpers
│  ├── git.js
│  ├── install.js
│  └── ...
├── template
│  ├── components
│  │  └── ...
│  ├── contracts
│  │  └── ...
│  ├── pages
│  │  └── ...
│  └──...
├── README.md
├── create-app.js
├── index.js
├── .gitignore
├── package.json
└── ...

adding lerna will need this structure
├── packages
│  └──create-dao
│  │  ├── helpers
│  │  │  ├── git.js
│  │  │  ├── install.js
│  │  │  └── ...
│  │  ├── template
│  │  │  ├── components
│  │  │  │  └── ...
│  │  │  ├── contracts
│  │  │  │  └── ...
│  │  │  ├── pages
│  │  │  │  └── ...
│  │  │  └──...
│  │  ├── README.md
│  │  ├── create-app.js
│  │  ├── index.js
│  │  ├── .gitignore
│  │  ├── package.json
│  │  └── ...
├── README.md
├── .gitignore
├── lerna.json
└── package.json

@e-roy thanks for trying that out. This looks really promising. Can you put in a PR so that we can test this out?


Add energy contributions

Hi Tiziano,
I thought it would be nice to parse in addition to the force consistent energy also the uncorrected total energy and other energy contributions, e.g. to check the impact of dispersion corrections etc. As for the levelparser I added the energies to the InnerSCF block.
I didn't add new tests but adjusted the existing ones to also check the new properties.
Please let me know what you think.
All the best,
Holger

Thank you for the swift review and feedback. Except maybe for the format of the spans I believe it is ready to be merged.


Add a "take_point" method to AABB and BoundingSphere.

This method would grow the AABB or BoundingSphere so it contains the given point.

Would expand_to perhaps be a better name?

A few questions:

Should this become a member of the BoundingVolume trait.
What about naming it "engulf" captures the operation  name well as meaning "sweep over (something) so as to surround or cover it completely. To eat or swallow something whole."
I see some bonds have &mut and & variants eg merge and merged. Do we want &mut and & variants for this method?

What about naming it engulf, or enclose? I think enclose may be the better of the two.

I like both of these, though of the two I think engulf does a better job of evoking the expansion of the existing bounds

@Plotnus :

Yes
Among both engulf seems the most appropriate.
Yes.

Overall, I think expand_to and engulf are two good choices. Though I'd like to have point appear on the method name too for explicitness. So in the end I'd choose engulf_point and engulf_point_mut.

What blocked me on this was implementing the functions for SpatializedNormalCone.
I'm unfamiliar with that bounding volume or how we'd adjust it based on points.

@Plotnus The SpacializedNormalCone combines an AABB that bounds points with a circular cone that bounds vectors. So adding a point to it amounts to calling self.aabb.engulf_point(pt).


resolve lower hostnames only

convert internal hostnames from yml into lower case
compare hostname check with lower case.

should solve issue #730

Coverage increased (+0.03%) to 86.48% when pulling 8f54e0cb89ad679ff3e0d1e9b739c3f8fe767ddc on strehle:bug/730 into 749b2c6e78a33892ebfa614b242512c4b7fcf4c3 on cloudfoundry:develop.

Coverage increased (+0.03%) to 86.48% when pulling 8f54e0cb89ad679ff3e0d1e9b739c3f8fe767ddc on strehle:bug/730 into 749b2c6e78a33892ebfa614b242512c4b7fcf4c3 on cloudfoundry:develop.


Discrepancies in Variables

Hi Vincent,
We're trying to determine what file to use for night-level sleep parameters and have noticed that GGIR is outputting varying values for variables. I have run one file using older versions of GGIR (prior to 1.6 when part5 didn't work), the 1.6 version (part5 works), and have output with difference values for sleep duration between the part4 night summary sleep full (old version), part4 night summary sleep full (new version), and part5 day summary WW. Can you provide some insight as to why this may be? I apologize if I missed it in the documentation.
Thank you!

Thanks Vincent!


Added Telugu translation of README

A large proportion of the population in India and the United States speaks Telugu. A README in Telugu will promote a better outreach and understanding of the technology.

Thanks for taking the time to contribute this localization pull request. However, we are not able to accept contributions to this historical reference repository at this time.


fix access to not use wrong names

Coupla things on this PR:

this might be a breaking change to config and types - I just don't wanna. My argument on it not being is that the previous state was genuinely in error (a patch). I think this is pragmatically fine.
Nothing actually failed to work as intended previously. If you passed --access="private" to npm, it ignored the flag and did the default option, which was restricted. :badpokerface:
I added an option to make individual packages have their own access field. Note that npm's regular publish does not respect this field. I was unsure whether to a) just not implement this feature and only allow the global access (@Andarist I believe you had a use-case for this though), b) nest this so it's changesets: { access: "restricted" }.

Fixes #193

I was actually working on fixing #193 - just didn't have time to test it properly:
https://github.com/Andarist/changesets/commit/82c83baf019720cb3289cf460df05a4b6e150056
I was thinking about using standard publishConfig field for reading this info - wondering why this was not chosen here.

Oh sorry! Didn't mean to to-tread. Happy to wait for your fix. And you're right, we should use publishConfig instead of just --access

Oh sorry! Didn't mean to to-tread.

No worries, I didn't communicate it well to indicate that I've started working on it.

Happy to wait for your fix. And you're right, we should use publishConfig instead of just --access

Gonna combine my changes with yours and prepare a followup PR to this one later today.

Closing in favour of #204


Added the course list for web development from web.dev

What does this PR do?
Added the list of courses for Web development which are present on the official web.dev website
For resources
Description
Explore our growing collection of courses on key web design and development subjects.
Why is this valuable (or not)?
It is valuable.
How do we know it's really free?
Free for lifetime
For book lists, is it a book? For course lists, is it a course? etc.
Course list
Checklist:

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

Follow-up

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

start each PR with a fresh branch


Google places & radius search

Would be great to include google places lookup with a mile radius option when using geo search with a google map that adjusts when searching different cities for example (and within say 100 miles of a particular city)

Hi @spindriftwebs thanks for the issue, could you detail a bit more what you want to achieve UX wise and how instantsearch.js is lacking features to do so nowadays?
Please ask your question on discourse: https://discourse.algolia.com/c/instantsearch
Thanks!


Best practices for applying CSS classes

Hi, I'm working on converting a mostly-unstyled prototype Svelte app to using GOV.UK. IIUC, everything is defined through CSS classes, so
text
 should generally become
text
, and a classless
 is invalid. I'm concerned specifying a class for every element makes the code less maintainable, and makes it easy for someone to use plain HTML and forget a class. Are there any recommendations or real-world examples we should follow?
Some ideas:

Define small Svelte components that configure the gov.uk classes, and use those. For example, a TH component would just wrap  and pass through other attributes the user specifies. We'd have a separate Button, StartButton, SecondaryButton, etc.

Some precedent: https://github.com/govuk-react/govuk-react and https://github.com/PaulioRandall/svelte-gds-design-system

Ideally in CSS, there'd be a way to express something like th { inherit from .govuk-table__header }, but not that I can find

Do some CSS preprocessing magic to change some of the class selectors to HTML element selectors?

Use Nunchucks, which appears to be this sort of templating approach?

CC @Pete-Y-CS and @ADD-William-WaltersDavis. A real example of manually specifying classes everywhere is in Will's project, https://github.com/ADD-William-WaltersDavis/planning_tool/blob/a586a944c31619ccc12f38f23b800263dbb0f703/components/HoverScores.svelte#L185.
Thanks! (And apologies for not following the issue template; there wasn't any for questions. I'm on secondment to Active Travel England, don't have a gov.uk email, and thus can't join the Slack to ask there)

Happy to be the contact point if you need a govuk email

Pete also just found https://frontend.design-system.service.gov.uk/sass-api-reference/#govuk-compatibility-govukelements-default-value, which appears to do the trick for
 and  already.

Note that this is a compatibility setting for Elements, one of the frameworks that pre-dated the Design System. We're currently in the process of removing those compatibility settings, so definitely not something to rely upon!

In terms of extending the placeholder classes to apply to HTML elements, I would recommend doing this inside a specific selector (like .prose as @querkmachine suggested above) and only using it for large areas of text, for example if there is text content provided by a CMS.
If you put other components from the Design System inside an element that has styles applied to type selectors, you may find that they override the component's styles and can cause them to display incorrectly.

Pete also just found https://frontend.design-system.service.gov.uk/sass-api-reference/#govuk-compatibility-govukelements-default-value, which appears to do the trick for
 and  already.

Note that this is a compatibility setting for Elements, one of the frameworks that pre-dated the Design System. We're currently in the process of removing those compatibility settings, so definitely not something to rely upon!

I suspect the link was intended to go to $govuk-global-styles (the next setting on from the one that's linked) which is not part of compatibility mode, but was mainly designed for prototyping rather than use in production.

Thank you both for the help! https://github.com/acteng/atip/pull/267 is a minimal start. As we migrate our app over to the styles, we'll use Sass in scoped blocks as suggested to save us some boilerplate.
I'm happy to close this issue, but I'm wondering about updating the docs to save the next person a bit of time. What if we added a step 6 to https://frontend.design-system.service.gov.uk/get-started/ that:

pointed to https://design-system.service.gov.uk/components/ and https://design-system.service.gov.uk/styles/ as references for the CSS and JS that the previous steps just made available
mentions the Sass pattern from here as an idea for organizing code

I can send a PR if this makes sense.
Also, another possible update to docs is about where to call initAll: https://github.com/acteng/atip/pull/267#discussion_r1260084593. I don't understand why one worked and the other didn't, but documenting could help other Svelte users. (Or maybe it's a Vite specific issue?) I know @ADD-William-WaltersDavis hit the same thing.

I suspect the link was intended to go to $govuk-global-styles

Oops, correct!


DEPR: warnings from style deprecation only in PY3

https://travis-ci.org/pydata/pandas/jobs/108709963
S.......S.S........................................................................./home/travis/build/pydata/pandas/pandas/core/config.py:219: FutureWarning:
mpl_style had been deprecated and will be removed in a future version.
Use `matplotlib.pyplot.style.use` instead.
  return self.__func__(*args, **kwds)

cc @TomAugspurger
prob should bump the stacklevel as well

this fixes: https://github.com/jreback/pandas/commit/ad78ce21fe0d04952eb8919cd65566d14c96b779


Unable to disable or override blueprint action or parameters

There are three problems -

not able  to pass only required parameters for my custom api ,
swagger object in routes.js is not overriding default request body
from get api's , not able to remove where, limit, skip, sort, populate, select, page

for 1 you can check here on how to add extra information to ur custom api in routes.js
https://github.com/theo4u/sails-hook-swagger-generator#custom-route-configuration  you can set the required fields too using required:true

you can send your configuration routes and screenshot of what's generated

This is my client controller

This is routes.js

Swagger generator is generating the custom parameter clientName,

But its not overriding the default params

Is there any way I can show only custom parameters ,not all blueprint params?
please correct me if I am wrong anywhere.

oh i think you can disabled blueprint on per controller basis check this https://sailsjs.com/documentation/reference/blueprint-api#?disabling-blueprints-on-a-percontroller-basis if it doesn't work it means it ain't following the configuration for blueprint per controller, which i can create an issue for later

making rest: 'false' is not serving the purpose

oh just understanding part 3. Its because default sails query params was put in place, so to override those or to add to those, i think overriding is much work, because you might want to add some extra minor query param, which you now have to pass all the default sails find query param like sort, limit and so on. which you tried to do here #16

Currently on a vacation till 18th june,will update the changes after that.
On Thu, Feb 8, 2018, 1:00 PM Theophilus Omoregbee<EMAIL_ADDRESS>wrote:

for 1 you can check here on how to add extra information to ur custom
api in routes.js
https://github.com/theo4u/sails-hook-swagger-generator#custom-route-configuration
you can set the required fields too using required:true
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/theo4u/sails-hook-swagger-generator/issues/14#issuecomment-364025564,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVc-Y87TrbmxkuOyv-Gxmp-daTHPnLxuks5tSqKDgaJpZM4R95J4
.


Merging gh-pages and master

We are now enthusiastic about the transition to the Carpentries lesson format.  A good next step after our next pilot workshop will be to merge our two main branches to simplify maintenance and clean up the repository.

The example lesson recommends keeping gh-pages as the default branch.  If we follow that example, we would merge material from master into gh-pages and then delete master.
Here is a sketch of how we could complete the merge:

master branch content
Merge plan

binder
Keep subdirectory

data
Move into existing subdirectory

figures
Move into fig subdirectory, update attribution

guide
Temporarily keep subdirectory, ultimately want to render and display

scripts
Undecided, could move to bin

.travis.yml
Need to merge

LICENSE.md,README.md
Need to merge

illustration.ipynb
Keep file

Other .md and .pptx files
Keep files

This looks good to me!

Closed by 3ab5c6ac48f3e60f3d0bc593b3a3d5c444595ad9


Refactor 2: Refactor Harder

Per @Fang-'s comment on #1, this is (also) a big one.
(Context: I approached this library as something I inherited, in order to jack it into Bridge.  I obviously haven't tried to be incremental at all here, instead opting to replace one reconstitution with another.)
This PR restructures the library again, mostly keeping @Fang-'s broad changes but exposing them via a purely-functional API where state passing is explicit.  Additionally I've changed the internal structure of the library somewhat -- functions in /internal modules are intended to maintain a one-to-one mapping with the actual contract APIs (to stay utterly "close to the contracts"), while other modules are free to expose their own implementations of these (only notable at present via the type-generic ships API -- functions in which can often take ship objects, and avoid hitting the network).
For the user, the manual state-passing (typically of a contracts object, or occasionally web3 in the case of transactions) is the biggest change.  Previously we would have:
constitution.createGalaxy(galaxy, account)

whereas now we need to pass the contract (actually, all relevant contracts) explicitly as the first argument:
createGalaxy(contracts, galaxy, account)

Similarly, at no point is there the concept of the "current account" -- accounts must be passed explicitly, where required.  The from field of transactions is derived from the private key used to sign them -- not from the (implicit) current state of web3 objects.
This all makes the library easier to modify (there is never the need to assume the existence of a web3 or contract instance in library code, for example), but to the user, it otherwise feels much the same.  Pass in state, get result.  There are also other changes: additional supported contract functions, bug fixes, deriving addresses from HD nodes directly, making more use of ethereumjs-util functions, handling private keys as buffers, etc.
I've begun to use this code in my work on Bridge and find it plays very well with things in that setting.  Someone should let me know if there are any obvious problems with it as they see it being used elsewhere! 😁

Actually, re: regenerating docs, might be nice to have that done by CI if that's not too difficult.

@Fang- I understand your point re: hiding the web3 details from the user.  But I think anyone pulling in constitution-js will probably already be pulling in web3 anyway, since they will probably want to do something or other that we haven't provided for them in the library.  I think the present API works well with web3, and doesn't try to do what web3 can do just as well or better.
To your point on CI: definitely.  I'm planning to get a basic-but-useful Travis config set up when I get a chance.

Ok -- going to merge this one now so I can make future commits a little more sanely-sized.


[com.hedera.hashgraph.proto.Duration] Issue after importing SDK project in eclipse

Please use from maven central or build with maven locally.
https://github.com/hashgraph/hedera-sdk-java#maven


The sploosh.

This can't be added until week 7 comes out on github.
what is with all the excess commits

Kol could make his own notesplash assets
I commit stuff like this lol.

This can't be added until week 7 comes out on github.
what is with all the excess commits

Kade engine has it tho?

This can't be added until week 7 comes out on github.
what is with all the excess commits

Kade engine has it tho?

no?

This can't be added until week 7 comes out on github.
what is with all the excess commits

Kade engine has it tho?

kade engine itself does not feature note splashes, some custom builds do tho


sort out epoch times properly

I've just worked out how to do this properly in postgresql.  i.e. how to convert a timestamp to an epoch time, returned as a string:
  select floor ( extract ( epoch from current_timestamp) * 1000) :: text;

Fixed by 25b283b94c1951f6139f65444bd727bd81c476de.  The ts field is now generated properly from the database and is renamed from date to ts accordingly.


Playlist handling – overwrite instead of append

Lets say there is a playlist named "test".
Clear the current queue and add some new songs. Then save the current queue as a playlist that already exists:
.saveplaylist test
Now the bots appends all songs from the current queue to the already existing playlist and saves it.
Intead I would expect the bot to replace the old playlist with the current queue.

Behavior what I prefer to implement: If a playlist will saved and a playlist with same name exist already, the user is asked if he want to overwrite list or songs should be append.
Is this is ok for you?

Split in 2 commands with different behavior


[PROPOSAL] Add model onError

Prerequisites
Versions

PHP version: 7.4
Laravel version: 7
Package version: 3.1.23

Description
Add model data on error

Example
In some cases, we need to display which model is an error. It will easy to get data base on the model.

Hey thanks for you proposal. That won't be possible as it would introduce a breaking change. Feel free to suggest an alternative that doesn't introduce a breaking change.

Thank you for your information


SQL keywords for column names glitch

Using SQL keywords for column names (i.e. "when" and "where") causes the columns to automatically rename themselves to include an underscore (i.e. "_when" and "_where"), which makes infowindows, wizard, and editing geometries non-responsive when a user tries to revert to the non underscore column names.

It is not possible to change that behaviour becuase is how postgres works. But if you check infowindows, a good thing, you can change the title for any attribute, so you could get rid of that underscores, for example :).


Implement Molecule-of-the-Week Table

I can give a set of molecules of the week (i.e., week X, use key XXX-XX-X). What should the format of the file look like?

I'd say just make a JSON file with an array of dates(YYYY-MM-DD) and keys. That would be the most flexible way.

Or just an array of keys and we can manage the timing

I would like for the file to be a CSV file.
integer_date,key
integer_date,key
...

This way, on server start, we can load everything into memory into a dictionary and just do a quick O(1) lookup for the MotW

See https://github.com/pittquantum/backend/blob/master/pqr/redirects/redirect_file for what I mean

See https://github.com/pittquantum/backend/blob/master/pqr/redirects/redirect_file and https://github.com/pittquantum/backend/blob/master/pqr/views.py#L206 for what I mean

Integer date should be YYYYMMDD

Solved with commit 5b282b967525ae792dd4d4cce555b0dae5180a20


location_aware_upsampling_forward

Where is the ‘LAU.location_aware_upsampling_forward’ in location_aware_upsampling_func.py please?

Please refer to location_aware_upsampling.h.

Oh, thanks!


Add CLI configuration for PHP as well.

Not sure we want xdebug as well in CLI... ?

conf.d is not used for all envs ? Seems strange to me that you have to do it link that.
In any case, a symlink would be better.

Path for the config files changed in Jessie.
Closing that as it was fixed in the related PR.


Allow extending observedAttributes.

Super minor change, but we expect all observedAttributes to come from our statically-declared properties. Instead, we should assume that authors may want to observe additional attributes beyond what they declared in the properties block. That is, we should allow this:
static get observedAttributes() {
  return [...super.observedAttributes, 'some-other-thing'];
}

Wew, I can't believe I left this hanging since May. I will admit I don't fully grok the technical problem, and a code level example would help, but I don't want to level more burden contextualizing this. You had me at "I can add behavior to the other standard methods, why not observedAttributes" — philosophically this is misaligned with the project.


Add Alphabetical order check to Format Validation

Added a check to the format validator to check that each section is individually sorted in alphabetical order. If anyone has a suggestion for something the build system should check for, just let me know. Otherwise, I'm going to self-merge this in to cover new pull requests.

Hah! I was just about to do this, Dave!!! 😂

See, I told you! We've been on the same wavelength for weeks now 🏄

🤙


Page Out Of Date

This page is out of date as "Code Map" and "Live Dependancy Validation" are not available in the VS2022 - 17.1.2 installer
Kind of makes the whole article redundant if you cannot download the tools.

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

ID: 0ae31d3c-d703-6431-8f01-e1d5c579846c
Version Independent ID: 5d6a50bb-d976-2473-fa81-10f6159d562f
Content: Install architecture code tools - Visual Studio (Windows)
Content Source: docs/modeling/install-architecture-tools.md
Product: visual-studio-windows
Technology: vs-ide-modeling
GitHub Login: @mgoertz-msft
Microsoft Alias: mgoertz

@PlaysafePP These tools are available in VS2022 but only in the Enterprise SKU.

Thanks for the update. I guess that info could be in the article?

It's on the previous page: https://docs.microsoft.com/en-us/visualstudio/modeling/analyze-and-model-your-architecture?view=vs-2022#edition-support-for-architecture-and-modeling-tools

So it is. However I came from this page https://docs.microsoft.com/en-us/visualstudio/modeling/map-dependencies-across-your-solutions?view=vs-2022

That page also mentions the need for the Enterprise edition to create these diagrams:

To create and edit code maps, you need Visual Studio Enterprise edition. However, in Visual Studio Community and Professional editions, you can open diagrams that were generated in Enterprise edition, but you cannot edit them.

IIRC, I don't think you need to install anything special to view existing diagrams.

You are entirely correct. Apologies. My problem was reading and actioning the article in the order I read it, so I didn't get further than the preceding sentence. Perhaps I should read the whole article next time.

No worries. This can all be quite confusing. Happy coding!


Update Italian translation

Pull Request Test Coverage Report for Build 725

5 of 6 (83.33%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 82.157%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/format_utils.js
2
3
66.67%

Files with Coverage Reduction
New Missed Lines
%

lib/format_utils.js
1
84.09%

Totals

Change from base Build 724:
-0.02%

Covered Lines:
7163

Relevant Lines:
8480

💛  - Coveralls


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_f5kzbg2tt4/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_f5kzbg2tt4/clone/recipe/meta.yaml:19: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_f5kzbg2tt4/clone/recipe/meta.yaml:36: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_f5kzbg2tt4/clone/recipe/meta.yaml:36: missing_description: The recipe is missing a description
ERROR: /tmp/abs_f5kzbg2tt4/clone/recipe/meta.yaml:36: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_f5kzbg2tt4/clone/recipe/meta.yaml:36: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


Vim beeps when opening a file using the CtrlP plugin

Steps to reproduce

Open Vim.
Use CtrlP to open a file.

Expected behaviour
Vim beeps when opening the file. It should open the file without the beep. I checked :messages but saw nothing there.
I think the commit that triggers this behavior is bf44b69d1f91d9778ae1887128c63d35d9a3d19b. I used git bisect, but I've never built Vim before nor used bisect before so I might be mistaken.
Version of Vim
9.1.76
Environment
macOS 14.3.1
Logs and stack traces
No response

I think the commit that triggers this behavior is https://github.com/vim/vim/commit/bf44b69d1f91d9778ae1887128c63d35d9a3d19b.

If that's indeed the commit, it may not indicate a bug in Vim; :wincmd p beeping when there's no previous window is normal.
However, before that commit, there was a case (seemingly the only such possible) where both the current and previous windows could refer to the same window if closing a different window resulted in Vim choosing the previous as the new current window. That may have resulted in :wincmd p not beeping previously, but still having no effect.

Thanks for the reply. I've been using CtrlP to open files like this for a while and this is the first time I've ever experienced this beeping behavior. CtrlP opens a window at the bottom of the screen from which you can select or fuzzy-find a file to open. When you select your file, it opens and the CtrlP window at the bottom closes. Any idea how to proceed? Is this a Vim issue or a CtrlP issue? Of course, I can still use CtrlP, but I'd much prefer it not beep every single time I open a file.

Any idea how to proceed? Is this a Vim issue or a CtrlP issue?

If things are happening as I assume, I'd consider the old behaviour a bug (and the lack of beeping prior to the patch). Plugins using :wincmd p should consider using :silent! or check winnr('#') != 0 if they don't want potential beeps. Though if the patch causes too much fallout in plugins, maybe reverting it and documenting the old behaviour could be preferable.
I don't use CtrlP, so can't provide specific help, but from looking at the source, maybe this line is responsible for the beeping? You can try adding sil! to the start of that line in your local copy of the file and see if the beeping goes away (other uses of winc p in the file are already protected by sil!).

Indeed, I have made the change to my local copy of CtrlP and the beep doesn't happen. Thank you.

After considering the issue more, my opinion changed, and I decided reverting v9.1.0001 was best, which Chris accepted; instead the possibility of winnr() == winnr('#') has been documented.
Therefore this issue should no longer affect CtrlP after v9.1.0170.  Apologies for the inconvenience!
My decision mostly boils down to more cases of curwin == prevwin being possible than I originally thought, and some cases of it even being accounted for by scripts in the wild; but if you're interested in the specific details, see here.

Thank you for investigating further even after the patch you provided me for CtrlP solved my problem.


Feature/book

@siddghosh108
✔ Status: APPROVED 🏆 🎉 🎊
You have done a great job on this branch, keep it up 👍 🥇. Your part is complete! There is nothing else to say other than... it's time to merge it into the development branch:
Congratulations! 🎉💯🌟
Top highlight 🎯
The Pull Request follows the GitFlow ✅
All the requirements have been implemented ✅
The PR title and description are professional and descriptive ✅
Linters are working properly ✅


Add Dark Mode to UI 🌙

Description
Implement a dark mode option for the virtual assistant’s UI.
Tasks

Add a toggle button for switching between light and dark themes.
Ensure the UI elements are optimized for both modes.
Skills Required: HTML, CSS, JavaScript
Expected Outcome: Users can switch between light and dark modes for a better experience.

Hey, i want to work upon it...

@Harshvardhan5703 Absolutely you can.
Before going to solve this issue I prefer you to complete #1
Comment on the issue #1 for create the UI for the Assistant & after that you can do this.

Hey can you do it under hacktober
On Sun, 13 Oct 2024, 18:52 Mohsin Ansari, @.***> wrote:

@Harshvardhan5703 https://github.com/Harshvardhan5703 Absolutely you
can.
Before going to solve this issue I prefer you to complete #1
https://github.com/mohsinansari0705/Voice-Activated-Virtual-Assistant/issues/1
Comment on the issue #1
https://github.com/mohsinansari0705/Voice-Activated-Virtual-Assistant/issues/1
for create the UI for the Assistant & after that you can do this.
—
Reply to this email directly, view it on GitHub
https://github.com/mohsinansari0705/Voice-Activated-Virtual-Assistant/issues/8#issuecomment-2408978424,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEDA5643FLJKZ5XLQXCQADTZ3JXZNAVCNFSM6AAAAABPW6ZR3WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBYHE3TQNBSGQ
.
You are receiving this because you were mentioned.Message ID:
<mohsinansari0705/Voice-Activated-Virtual-Assistant/issues/8/2408978424@
github.com>

Done, Now you can do it. @Harshvardhan5703

Bro where is the UI!!


PropertyValidator does not work with dynamic validation groups

The Wicket property validator does not work with dynamic validation groups because of the following issues:

It can not cope with the case in which the Default validation group is provided explicitly
The initial validation groups determine whether required should be set on the component. But the PropertyValidator never unsets the flag.

When using a Wicket property validator you can provide a model for the validation groups:
IModel<Class<?>[]> validationGroups = Model.of(new Class[]{Default.class});
new PropertyValidator<>(validationGroups);

Let's say I have the following model object:
public class MyObject {

    @NotNull
    private String name;

    @NotNull(groups=AdditionalValidations.class)
    private String additional;
}

Then the @NotNull constraint on the name attribute is NOT performed when submitting the form.
This is caused by the fact that no validations are performed within FormComponent.validateValidators. On the following line both conditions evaluate to false:
if (isNull == false || validator instanceof INullAcceptingValidator<?>)
The trick to perform "required" validations is that the PropertyValidator sets the component to required within its onConfigure method:
this.component.setRequired(true);
However, this is only performed when isRequired() evaluates to true, which is NOT the case in my example. This is related to issue 1:
When the Default validation group is provided explicitly, isRequired returns the wrong result.
Issue 2 has to do with the required flag only being set once by the PropertyValidator:
Given the example model object above. Assume that when I create the form components I don't set them required explicitly (formComponent.setRequired(true), but I rather let it up to the PropertyValidator because I want to describe my validations on one place, the model object. So, when I initially create the PropertyValidator the "additional" model object attribute is NOT set to required within PropertyValidator.onConfigure, because the validation groups don't match. Then when I would dynamically change the validation groups:
validationGroups.setObject(new Class[]{Default.class, AdditionalValidations.class});
Then subsequently a required/NotNull validation should be performed for the "additional" attribute when submitting the form. However, it is not performed because within PropertyValidator.onConfigure it is only set to required once, and not updated at a later stage.

@paulspeijers Please close this PR and open a new one that uses the correct branches.
This PR tries to merge from master to wicket-7.x and this leads to hundreds of modified files.
Thank you!


Assignment

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
adding all the codes of the questions
What did you learn from the changes you have made?
make sure at the right directory
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
simplify my code
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
yes, not quiet understand Q2. with the help from classmates, I understood the main require of this question.
How were these changes tested?
yes. I tested locally.
A reference to a related issue in your repository (if applicable)
Checklist

[ X] I can confirm that my changes are working as intended

Assignment 1 is completed.


PR #1-Rails_ Association_Project

Hello,
This is the first pull request for the Rails Association Project -Events App.
Kindly review .

Hello,
This is the first pull request  for the Rails Association Project -Events App. It has been modified according to the above corrections.
Kindly review .


Change the name Lookaheadmapper and basicmapper

What is the expected enhancement?
In the folder, it is not clear what is a mapper and what is not.  Mapping is all things combined: swap insertion, fix direction, unroll to native gates, etc.

[x] Unroller
[ ] LookaheadSwap
[ ] StochasticSwap should happend in (#1520)
[ ] BasicSwap
[ ] LayoutSelect should happen in (#1499)
[x] CXDirection (pr #1521)

@ajavadia and @1ucian0 are we going to get these done.

Yes I'll do it.


Update changelog generator

Even though we exclude prs and issues without labels, this doesn't seem to work.
One workaround is going to be to add a tag to exclude from the changelog, as mentioned here:
https://github.com/skywinder/github-changelog-generator/issues/302
I also notice that if the PR's are labeled, they get included in the "enhancements" section, but not the pull request. I think that if we label all the PR's, but for the ones that shouldn't be in the changelog (which is pretty much any of them that address an issue), that might work.

This seems to be solved in the updated script. I'm good to close this.


Segmentation Error in the sample with skiasharp

When i try to deploy one sample with Skiasharp on the emulator there is the segmentation error message. I tried with SkiaSharp2DSample and PlayingWithHWInputs.
I use visual studio 2019.

Hi myroot, this is details

Exe
tizen40
true

portable

None

Please use SkiaSharp preview version if you want to use with CircularUI 1.4.0

It is related issue and resolved on preview version
https://github.com/mono/SkiaSharp/issues/1167

Please use SkiaSharp preview version if you want to use with CircularUI 1.4.0
<PackageReference Include="SkiaSharp.Views.Forms" Version="1.68.2-preview.45" />

Thank you, using this configuration the samples work:
`

Exe
tizen40
true

portable

None

MainPage.xaml

MSBuild:UpdateDesignTimeXaml

`

There is a way to use Microcharts in Tizen.Wearable project? it's a SkiaSharp wrapper

Microcharts is working on Tizen wearable


[BREAKING CHANGE] Pass translation strings as options

Replaces the existing subclass method.
Fixes #89

Looking it this, I realised that we can make the change "less breaking", if we leave in the -de.js translation file and just rename hideText to options.hideText in that file. That would work, wouldn't it?
Ideally I'd still like to go the Leaflet.i18n route because it's nice to be cooperative with other plugins, but I realise it's a bit silly to bring in a whole dependency for just two strings.

options.hideText in the ...-de.js file didn't work when I tried that (gives syntax error). Similarly setting
options: {
    hideText: ...

Would override the defaults thus requiring all of the default options to be present in the ...-de.js file.

What if we move the strings into a sub-object, e.g. options.strings.hideText - That would also make a cleaner options object, especially if we ever need new strings.

I like the idea of a sub-object.
Don't think it helps with the ...-de.js file though. The only way I can see how the sub-classing can be kept in parallel to i18n-via-options is if it sets some override object and there is a call during initialization to override strings if that object is present. To me that sounds less clean than a breaking change.

Yes, I agree. Translation strings as a options sub-object are the way to go. Will you make it happen? :+1:

Done =)

Looking good! We just need to mention the sub-object options in the README and we should be good to go.

Argh, forgot to update the readme. Will do that and add an example file with the german translations

Brilliant, thanks for the work!


AMBARI-24808 : Logs are not showing in Hiveview2 for hive query. (nitirajrathore)

What changes were proposed in this pull request?
handled the cases to not miss logs for short queries and for multiple queries.
How was this patch tested?
manual testing was done and found the logs were getting fetched properly on the same server where bug was getting reproduced.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4353/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4365/
Test FAILed.
Test FAILured.

Hi @adoroszlai : Build of this pull request is failing multiple times on unrelated  test cases.
Can I merge the pull request without the successful build ?

@nitirajrathore components in contrib are not built/tested by the Jenkins jobs, so neither failure nor success are indicative of the change.  I would say that until this oversight in the build system is addressed, you can go ahead and merge once you have enough approving reviews.
Also, to avoid making Jenkins perform unnecessary builds, please include [skip ci] in the pull request text.


Add use case and date elements for expected delivery time

This is on the order response (i.e. actual expected delivery time for this order), and not in the catalog (which would be "the normal time to deliver this")

Add to the use case: this is a reason to change the order


respect XDG Base Directory Specification

Respect the specification here : http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html
One difference though, we only use this when the environment variable is set instead of defaulting to ~/.config and ~/.local/share so that people currently having their configuration in their ~ have no issue.
We could eventually imagine a migration process and respect the spec fully.
If we go this way, we need to take care of Mac OS X (http://stackoverflow.com/questions/3373948/equivalents-of-xdg-config-home-and-xdg-data-home-on-mac-os-x#answer-5084892) and Windows.

Very good point, thanks for the PR. We would really need a migration process though. Also what's missing to fully respect the spec?

Currently we only use XDG_CONFIG_HOME when the env variable if set. The specification says that if the env variable is not set, we must default to ~/.config which will cause taxi to don't take into account the current config file.
I can see two possible solutions :

We move the config from ~ automatically and eventually inform the user
We check if a config file exists in ~ and use if this is the case.

If no config file exists, we create the new one in .config

Seems to me the fix is incomplete because the projects file is strill created in the home dir. I'll have a look

@sephii completely rewrote the PR.
We now use the config and data  in the home directory if they exists.
Otherwise we rely on the appdirs package which works for Mac, Windows and Linux to create the config file and data directory in the relevant place for the OS.
I also corrected a small bug where you didn't use the path given as an argument for the taxi dir correctly ;)
What do you think ?

Merged in dev (https://github.com/sephii/taxi/commit/ca6522d73c516c6fdf9aa4ec4615ad256b5341d6). Thanks!


[5.4] In backwards compatibility mode, be more permissive of Obj-C nulls (#35825)

(This cherry-picks #35825 from main to release/5.4)
The new cast logic checks and aborts if a non-nullable pointer has a null value at runtime.  However, because this was tolerated by the old casting logic, some apps inadvertently rely on being able to cast such null references.
This change adds specific checks for null pointers in compatibility mode and handles them similarly to the previous casting logic:

Casting to Obj-C or CF type: casting a null pointer succeeds
Casting to class-constrained existential: casting a null pointer succeeds
Casting to a Swift class type: cast fails without crashing
Bridging from Obj-C class to Swift struct type: cast fails without crashing
This also adds a guard to avoid trying to lookup the dynamic type of a null class reference.

In non-compatibility mode, all of the above cause an immediate runtime crash. The changes here are only intended to support existing binaries running against new Swift runtimes.
Resolves rdar://72323929

@swift-ci Please test


Icon always shows motion detected

Hi,
First of all thanks for these drivers they are working well for me so far. One thing that i just noticed was the icon for aqara motion sensor stays "motion detected" even when there is no motion. And somehow, illuminance is not working for me. When you have a chance, please help check. Thanks man!!!

Hello, I'll check the icon,
but the icon view is on the ST side and I just copied the snippet from the zigbee motion driver.
Are you sure that there are two icons for this type of device?
as for illuminance, can you please provide a log from cli. it's hard to debug without log

please check now


extract equations in tables and cells

using document.get_text() method might work on documents but I see no way of extracting equations inside a table or a cell. Like in python-docx there is a support of tables and cells and text in a cell can be extracted by cell.text, but how do I use docxlatex to extract the equations and the text only in a particular cell of a table ?

if there would be a method cell.get_text() it would have been great as it would add the support of extracting equations from the tables

Thanks for the issue @arth17singh. Unfortunately, there is no way to extract equations from a particular cell. docxlatex is a simple parser and just extracts the entire text of the document at once, unlike python-docx, which builds the XML tree for the document as attributes on the Document class.
To add support for what you suggest will take some considerable amount of work, and unfortunately I can't dedicate that much time to this project right now. However, I think that it might be possible to support extraction of equations by modifying the source code for python-docx. If you really need to extract equations in particular cells, you may want to fork python-docx and implement the equation extraction used in this library.

That's great! Glad docxlatex could help someone :)


Python: sets

Description :
Briefly explain sets in python
Note :

Changes should be made inside the Python/ directory & Python branch.
Issue only for GWOC'21 contributors.
Issue will be assigned on a first come first serve basis, 1 Issue  ==  1 PR.
In the PR, keep this issue's title as your PR's title.

Must Follow : Contributing Guidelines & Code of Conduct before start Contributing.

i would like to work on this

I would like to work on this please documentation
Peddi sai lohith
Python batch 4

I would like to work on this issue. Thank you!!

Hello @adityabisoi
I want to contribute via documentation on this issue
Tejas prajapati python batch-6

Please allow me to contribute here. I can contribute via any of the three ways of audio/video/documentation.

Hello, I would like to contribute here, in a video format
Thanks and Regards,
Pranav - Python Batch 6

I would like to work on this issue via documentation.

I would like to work on this issue via documentation

If this issue is not yet assigned to anyone, I would be happy to work on it.
Python Batch: 3
Preference: Documentation

@ankurg132 Is the mentor of this issue. Please reach out to him in case of queries.

This issue is open for audio contributions.

Hello Sir,
Myself Rahul Kashyap, Python Batch 6
Sir I would like to work on this issue via audio.
Please assigned me this issue

-please assign me this issue in video section.

@ankurg132 i had created it's pr please review it


Extra requirement found by mlem.api.save

MLEM from release/0.3.0 branch adds dateutil to requirements upon mlem.api.save:
$ git clone https://github.com/iterative/mlem
$ cd mlem
$ git checkout release/0.3.0
$ pip install -e .
$ python train.py
$ tail rf.mlem
object_type: model
requirements:
- module: sklearn
  version: 1.1.2
- module: pandas
  version: 1.5.0
- module: numpy
  version: 1.23.3
- module: dateutil
  version: 2.8.2

# train.py
from mlem.api import save
from sklearn.datasets import load_iris
from sklearn.ensemble import RandomForestClassifier

def main():
    data, y = load_iris(return_X_y=True, as_frame=True)
    rf = RandomForestClassifier(
        n_jobs=2,
        random_state=42,
    )
    rf.fit(data, y)

    save(
        rf,
        "rf",
        sample_data=data,
    )

if __name__ == "__main__":
    main()

Since it's not dateutil, but python-dateutil, due to another bug https://github.com/iterative/mlem/issues/362: not only requirements are incorrect, but also mlem build and mlem deploy don't work (pip can't find pip install dateutil).
Consequently, this also prevents generating example-mlem-get-started for the release https://github.com/iterative/mlem/issues/418

closed by #427 (reopen if not)


I want to pass a object in marks but doesn't looks good

ageGroup: {         '0': '0',         '1': 'All',         '2': '7+',         '4': '13+',         '5': '16+',         '7': '18+'       },
I want to pass this object but this doesn't looks as per the requirements. It should have equal spacing in between and tooltip should show All, 13+, etc. Please check it.

Please provide all the parameters you passed.

options: {         dotSize: 24,         width: 'auto',         height: 8,         backgroundColor: 'black',         direction: 'ltr',         min: 0,         max: 6,         interval: 1,         disabled: false,         clickable: true,         duration: 0.5,         lazy: false,         tooltip: 'focus',         tooltipPlacement: 'top',         tooltipFormatter: void 0,         useKeyboard: true,         enableCross: true,       }

And mark = ageGroup, right?
The key of mark corresponds to the value on the component, why max = 6 but there is '7': '18+'. And there is no definition of 3, so the spacing between 2 and 4 will be longer.
You can use data, e.g. https://jsfiddle.net/NightCatSama/2xy72dod/10388/

Yes. Worked Fine just tell me one more thing where i can put my other options and how can i show those keys(All, 13+, etc.) in tooltip

You can try tooltipFormatter: val => this.ageGroup[val].

I fixed the mark is an object, include does not work. in version 3.0.30.
I wrote an example of whether this is what you need.
e.g. https://jsfiddle.net/NightCatSama/2xy72dod/10400/

If you need the same interval then use data.
Just like the example I wrote before.

Thanks Bud it worked perfectly fine

Is the value empty?

So i just wanted to leave equal space at beginning thats why i assigned to 0 but instead no value should be passed

like this? https://jsfiddle.net/NightCatSama/2xy72dod/10406/

Yes. But the spacing is not equal now

The point is i want to start leaving some space in left

I have done my best.
https://jsfiddle.net/od0xzkfj/

Perfect this is what i am exactly looking for. Thanks a lot


Unable to install sharp

What are you trying to achieve?
Install sharp and subsequently other node modules.
Have you searched for similar questions?
Unable to find a solution
Are you able to provide a standalone code sample that demonstrates this question?
Are you able to provide a sample image that helps explain the question?
Unfortunately no
The problem
When I npm install sharp, I get the following output:
<EMAIL_ADDRESS>install /projects/user-portal/node_modules/sharp
node-gyp rebuild

make ✖ ERR  Missing Makefile / Bakefile
make ℹ info Run "make init" to generate a Makefile.

There seems to be some kind of error but installation still passes. I have run make init but that made no difference for what I write below.
Then when I run other scripts I get the error:
Error: Cannot find module '../build/Release/sharp.node'

And although there has been a folder created called /node_modules/sharp/build, and there is content, it does not include a subfolder called Release.
Googling teaches me “Missing Makefile / Bakefile” has to do with the node module “make”. But how to solve this?
I've also tried running sudo npm install sharp, which results in:
<EMAIL_ADDRESS>install /projects/user-portal/node_modules/sharp
node-gyp rebuild

gyp ERR! configure error
gyp ERR! stack Error: EACCES: permission denied, mkdir '/projects/user-portal/node_modules/sharp/build'
gyp ERR! System Linux 3.10.0-957.21.3.el7.x86_64
gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /projects/user-portal/node_modules/sharp
gyp ERR! node -v v8.16.0
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS><=0.13.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>- 3 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>license should be a valid SPDX license expression
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

Please can you try with the latest v0.22.1.

Sorry, I wasn't using the latest version in the first place. I updated package.json and now npm install sharp results in:
Unhandled rejection Error: EACCES: permission denied, open '/home/user/.npm/_cacache/index-v5/ff/ec/1ac73f831b4be306687ec3ade736ad1a5c148745371b34e6945e71c98510'

Unhandled rejection Error: EACCES: permission denied, open '/home/user/.npm/_cacache/index-v5/63/c8/28772eeccc799c7e548d1a0d34a2fc0607ee0ebb70072c0a17b47e1a8ef6'

npm ERR! cb() never called!

And sudo npm install sharp results in:
><EMAIL_ADDRESS>install /projects/user-portal/node_modules/sharp
> (node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)

info sharp Downloading https://github.com/lovell/sharp-libvips/releases/download/v8.7.4/libvips-8.7.4-linux-x64.tar.gz
ERR! sharp EACCES: permission denied, open '/projects/user-portal/node_modules/sharp/830-libvips-8.7.4-linux-x64.tar.gz'
info sharp Attempting to build from source via node-gyp but this may fail due to the above error
info sharp Please see https://sharp.pixelplumbing.com/page/install for required dependencies
gyp ERR! configure error
gyp ERR! stack Error: EACCES: permission denied, mkdir '/projects/user-portal/node_modules/sharp/build'
gyp ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Avoid using sudo with npm install where possible. If you absolutely have to use sudo then add the npm install --unsafe-perm flag.

Got I solved as follows:

sudo apt remove nodejs npm to remove Node and npm
sudo rm -rf to delete node_modules folder and npm folder
sudo apt-get install nodejs to install again Node and npm
npm install sharp works again.

Avoid using sudo with npm install where possible. If you absolutely have to use sudo then add the npm install --unsafe-perm flag.

This was the only way I could install 'shapr'

Error: EROFS: read-only file system, mkdir '/user' any ideas ?


undefined method `new_registration_path' in rails 3.1 and devise 1.4.7

In older versions i see this issue, but now in devise version 1.4.7 it's again.
NoMethodError in Devise/sessions#new
Showing /usr/local/lib/ruby/gems/1.9.1/gems/devise-1.4.7/app/views/devise/shared/_links.erb where line #6 raised:
undefined method `new_registration_path' for #<#Class:0xb3c20ebc:0xb3c1b14c>
Extracted source (around line #6):
3: <% end -%>
4:
5: <%- if devise_mapping.registerable? && controller_name != 'registrations' %>
6:   <%= link_to "Sign up", new_registration_path(resource_name) %>
7: <% end -%>
8:
9: <%- if devise_mapping.recoverable? && controller_name != 'passwords' %>

It works fine in the app you provided:
Started GET "/admin/login" for <IP_ADDRESS> at 2011-10-09 11:35:52 +0200
Processing by ActiveAdmin::Devise::SessionsController#new as HTML
Rendered /Users/jose/.rvm/gems/ruby-1.9.2-p180/gems/devise-1.4.7/app/views/devise/shared/_links.erb (0.5ms)
Rendered /Users/jose/.rvm/gems/ruby-1.9.2-p180/gems/activeadmin-0.3.2/app/views/active_admin/devise/sessions/new.html.erb within layouts/active_admin_logged_out (5.9ms)
Completed 200 OK in 21ms (Views: 9.2ms | ActiveRecord: 0.5ms)

Restart your server- I think it's because you changed the routes. (Worked for me!)

Thanks!

Cool!! It works!

Thanks,
Restart worked.

Thanks,
I was struggling with that,
yeah! restart worked!

thanks,it worked forme as too


🐛 [firebase_messaging] add Windows support

[firebase_messaging] add Windows support

Hi @allessandrojs, firebase_messaging is not currently supported on the Windows platform, as noted in the Firebase documentation. If you'd like to share your feedback or request support, you can do so on the Firebase UserVoice forum.


About the code

Thanks for your great work! When will the code be released?

Hi! Thanks for your interest in our work. We were occupied by the ICLR deadline and are now working on the RICA2 code release. We plan to release the code within 1 week :)


Unable to start activity ...LoginActivity

Is the bug present in the demo Chat SDK project?
No it is not present
What modifications have you made to the Chat SDK?
No modifications yet
Android Version:
8.1
Steps taken to reproduce the problem:
Just followed the steps given in the Android Quickstart here: [(https://chatsdk.co/docs/android-quickstart/)]
Expected result:
It should start a short app right?
Actual result:
App crashes during start with errormessage:
java.lang.RuntimeException: Unable to start activity ComponentInfo{de.********.chatsdktry/co.chatsdk.ui.login.LoginActivity}: java.lang.NullPointerException: Attempt to read from field 'boolean co.chatsdk.core.session.Configuration.resetPasswordEnabled' on a null object reference
Comments:
I completely followed the given steps and made no changes.
I implemented no extra modules.

What version of Android Studio are you using?

Android Studio 3.2.1
Build #AI-181.55<IP_ADDRESS>56338, built on October 9, 2018
JRE: 1.8.0_152-release-1136-b06 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0

Please kindly update your Jre/JDK. Win10 is quite demanding than Win 8.0, then maybe consider uninstall and reinstalling Studio. Your version in Android Studio is ok as stated above.
Re-clone the repo and re-run in the updated Jre/JDK environment.

Tested and can't reproduce in 4.6.0. If it still persists please reopen the issue.


Tools for test

brocaar, here is some tools that I develop them for test. Maybe useful for other friends.

https://github.com/panyingyun/watch
Watch message received from loraserver/gateway/application by LoRaWAN™

https://github.com/panyingyun/vmloragateway
Visual Machine(Lora Gateway) connect to lora-gateway-bridge for test LoRaWAN™ server benchmark or others(NS+NC+AS)

https://github.com/panyingyun/vmloranode (plan to do)
Vmloranode create Virtual node for test LoRaWAN™ gateway and LoRaWAN™ network(NS + AS+ NC)

Thanks for sharing! I'll add a third-party tools documentation page :-) I'll close the issue once these links are added.

Added in https://github.com/brocaar/docs.loraserver.io/commit/90cd47df94423cb9997b3bdebc777aa7dda0b71a.


Fix travis-ci config

Currently all builds are made for the latest Symfony version. As you can see https://travis-ci.org/sergeyfedotov/FilterBundle/builds/126106446 builds are now failed for the Symfony 2.7. They are failed because Symfony 2.7 doesn't support a fully qualified class name as the form type name.

Thanks. Will fix Bundle for Symfony 2.7 to end of this week.


set mpcG

Updated a minor typo in MPCORB.py (line 93) that set mpcG to the value of mpcH
Review Checklist for Source Code Changes

[x] Does pip install still work?
[ ] Have you written a unit test for any new functions?
[x] Do all the units tests run successfully?
[x] Does adler run successfully on a test set of input files/databases?
[x] Have you used black on the files you have updated to confirm python programming style guide enforcement?

Does this issue warrant a unit test to ensure planetoid gets written with the correct values for some test ssObjectId?

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

Project coverage is 69.84%. Comparing base (6077e15) to head (c54b035).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   69.84%   69.84%
=======================================
  Files           3        3
  Lines          63       63
=======================================
  Hits           44       44
  Misses         19       19

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

Does this issue warrant a unit test to ensure planetoid gets written with the correct values for some test ssObjectId?

Yep. In fact, this would have been caught in a unit test - it's the first thing you'd test.
Once I've got the SQL database for testing sorted I'll be writing all these unit tests. I could probably do it without, given that the dataclasses now have custom constructors, but might as well do it all in one go.


Installation error "void-function pipeline"

I tried to install with the use-package and straight.el and got this error:
Symbols function definition is void: pipeline

The debugger shows
Debugger entered--Lisp error: (void-function pipeline)
  (pipeline color (color-darken-name it (or darkness eyeliner/default-darkness)) (color-desaturate-name it (or desaturation eyeliner/default-desaturation)))
  eyeliner/adjust-color("deep sky blue" nil nil)
  (let* ((color (eyeliner/adjust-color color darkness desaturation)) (symbol (eyeliner/symbol name "inactive")) (props (list ':foreground color))) (eyeliner/make-face symbol props))
  eyeliner/make-inactive-face("unmodified-icon" "deep sky blue" nil nil nil)
  (let ((active-face (eyeliner/make-active-face name color props)) (inactive-face (eyeliner/make-inactive-face name color darkness desaturation props))) (cons active-face inactive-face))
  eyeliner/make-face-pair("unmodified-icon" "deep sky blue" nil nil nil)
  (let ((color (eval color)) (face-name (format "%s-icon" name))) (eyeliner/make-face-pair face-name color darkness desaturation props))
  (let* ((inactive-face (let ((color (eval color)) (face-name (format "%s-icon" name))) (eyeliner/make-face-pair face-name color darkness desaturation props))) (active-face (if inactive-face (car-safe (prog1 inactive-face (setq inactive-face (cdr inactive-face)))) (signal 'wrong-number-of-arguments (list nil (length inactive-face)))))) (let ((symbol (eyeliner/symbol name "icon"))) (list 'defun symbol nil (list 'propertize icon ''display (cons 'or (cons display '('((height 1.0) (raise 0.0))))) ''font-lock-face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)) ''face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face))))))
  (let* ((family it) (icon (if family (car-safe (prog1 family (setq family (cdr family)))) (signal 'wrong-number-of-arguments (list nil (length family)))))) (let* ((inactive-face (let ((color (eval color)) (face-name (format "%s-icon" name))) (eyeliner/make-face-pair face-name color darkness desaturation props))) (active-face (if inactive-face (car-safe (prog1 inactive-face (setq inactive-face (cdr inactive-face)))) (signal 'wrong-number-of-arguments (list nil (length inactive-face)))))) (let ((symbol (eyeliner/symbol name "icon"))) (list 'defun symbol nil (list 'propertize icon ''display (cons 'or (cons display '('((height 1.0) (raise 0.0))))) ''font-lock-face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)) ''face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)))))))
  (progn (let* ((family it) (icon (if family (car-safe (prog1 family (setq family (cdr family)))) (signal 'wrong-number-of-arguments (list nil (length family)))))) (let* ((inactive-face (let ((color (eval color)) (face-name (format "%s-icon" name))) (eyeliner/make-face-pair face-name color darkness desaturation props))) (active-face (if inactive-face (car-safe (prog1 inactive-face (setq inactive-face (cdr inactive-face)))) (signal 'wrong-number-of-arguments (list nil (length inactive-face)))))) (let ((symbol (eyeliner/symbol name "icon"))) (list 'defun symbol nil (list 'propertize icon ''display (cons 'or (cons display '('((height 1.0) (raise 0.0))))) ''font-lock-face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)) ''face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face))))))))
  (if it (progn (let* ((family it) (icon (if family (car-safe (prog1 family (setq family (cdr family)))) (signal 'wrong-number-of-arguments (list nil (length family)))))) (let* ((inactive-face (let ((color (eval color)) (face-name (format "%s-icon" name))) (eyeliner/make-face-pair face-name color darkness desaturation props))) (active-face (if inactive-face (car-safe (prog1 inactive-face (setq inactive-face (cdr inactive-face)))) (signal 'wrong-number-of-arguments (list nil (length inactive-face)))))) (let ((symbol (eyeliner/symbol name "icon"))) (list 'defun symbol nil (list 'propertize icon ''display (cons 'or (cons display '('((height 1.0) (raise 0.0))))) ''font-lock-face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)) ''face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)))))))))
  (let ((it (eyeliner/find-icon icon-name))) (if it (progn (let* ((family it) (icon (if family (car-safe (prog1 family (setq family (cdr family)))) (signal 'wrong-number-of-arguments (list nil (length family)))))) (let* ((inactive-face (let ((color (eval color)) (face-name (format "%s-icon" name))) (eyeliner/make-face-pair face-name color darkness desaturation props))) (active-face (if inactive-face (car-safe (prog1 inactive-face (setq inactive-face (cdr inactive-face)))) (signal 'wrong-number-of-arguments (list nil (length inactive-face)))))) (let ((symbol (eyeliner/symbol name "icon"))) (list 'defun symbol nil (list 'propertize icon ''display (cons 'or (cons display '('((height 1.0) (raise 0.0))))) ''font-lock-face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)) ''face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face))))))))))
  (lambda (name icon-name color &optional darkness desaturation props display) "Define a function which will return propertized text with the\n   proper color based on the value of ACTIVE which is bound by\n   spaceline." (let ((it (eyeliner/find-icon icon-name))) (if it (progn (let* ((family it) (icon (if family (car-safe (prog1 family (setq family (cdr family)))) (signal 'wrong-number-of-arguments (list nil (length family)))))) (let* ((inactive-face (let ((color (eval color)) (face-name (format "%s-icon" name))) (eyeliner/make-face-pair face-name color darkness desaturation props))) (active-face (if inactive-face (car-safe (prog1 inactive-face (setq inactive-face (cdr inactive-face)))) (signal 'wrong-number-of-arguments (list nil (length inactive-face)))))) (let ((symbol (eyeliner/symbol name "icon"))) (list 'defun symbol nil (list 'propertize icon ''display (cons 'or (cons display '('((height 1.0) (raise 0.0))))) ''font-lock-face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)) ''face (list 'if 'active (list 'quote active-face) (list 'quote inactive-face)))))))))))(unmodified "circle-o" eyeliner/cool-color)
  (eyeliner/icon unmodified "circle-o" eyeliner/cool-color)

I believe that https://github.com/dustinlacewell/eyeliner/pull/3 fixes this? It hasn't been merged, but you could easily make the changes yourself locally.

Confirming that the changes solve the issue. Thanks!


x/tools/go/analysis: optionally ignore generated files

This is a proposal to add an option to analysis.Analyzer to declare its disinterest in generated files.  Since such files' contents are not directly within the user's control, it may be better for an analyzer to ignore them.
For example, we have converted ineffassign, which checks for ineffectual assignments, into an Analyzer.  Such ineffectual assignments are not always indicative of an error but only sloppiness, which sometimes a code generator ought to be allowed to get away with.
Previously, we were using dmitri.shuralyov.com/go/generated, which checks if a file has a comment like // Code generated _ DO NOT EDIT., but now we scan Pass.Files[i].Comments rather than reading each whole file again.
Anyway, this proposal is to add a field Analyzer.IgnoreGeneratedFiles which, if true, would result in those files not being present in Pass.Files.

/cc @matloob

/cc @matloob

Why is it a problem to filter which files you're interested in, when looping over Pass.Files?

Why is it a problem to filter which files you're interested in, when looping over Pass.Files?

It's not a problem; it just seemed like it might be common enough that this package should support it.
It is quite trivial for each tool author to implement but they still need to know about the standard.  Better to enshrine it in code than documentation.

It's not a problem; it just seemed like it might be common enough that this package should support it.
It is quite trivial for each tool author to implement but they still need to know about the standard.  Better to enshrine it in code than documentation.

Indeed. But the solution then seems to me to have an API or package to detect the standard, not to add it to the internals of go/analysis. Detecting whether a file is generated is something generally useful, not just as part of go/analysis.

Indeed. But the solution then seems to me to have an API or package to detect the standard, not to add it to the internals of go/analysis. Detecting whether a file is generated is something generally useful, not just as part of go/analysis.

I agree it should live in its own package, and in fact it does: dmitri.shuralyov.com/go/generated.  Additionally, my thought was that if some majority (60%?  90%?) of users of go/analysis used this feature, it would make sense to make it even more discoverable by bundling it in with go/analysis.  It really depends whether it is considered a core feature.
I'm imagining that many users of go/analysis will be writing linters, which should ignore generated code.  But I don't have any idea just what proportion would use it.  Maybe someone could do an analysis of importers of go/analysis to see how many of them check for generated code (whether via dmitri.shuralyov.com/go/generated or by directly checking for a comment matching // Code generated ... DO NOT EDIT)?  I recall @campoy used to do some BigQuery analyses to aid in design decisions – is that still easy to do?

I agree it should live in its own package, and in fact it does: dmitri.shuralyov.com/go/generated.  Additionally, my thought was that if some majority (60%?  90%?) of users of go/analysis used this feature, it would make sense to make it even more discoverable by bundling it in with go/analysis.  It really depends whether it is considered a core feature.
I'm imagining that many users of go/analysis will be writing linters, which should ignore generated code.  But I don't have any idea just what proportion would use it.  Maybe someone could do an analysis of importers of go/analysis to see how many of them check for generated code (whether via dmitri.shuralyov.com/go/generated or by directly checking for a comment matching // Code generated ... DO NOT EDIT)?  I recall @campoy used to do some BigQuery analyses to aid in design decisions – is that still easy to do?

https://github.com/golang/go/issues/28089 is about adding this API to x/tools, which is official and more discoverable.
I still think there's no strong argument in favor of adding it straight to go/analysis. At least, most of the analyzers I've written apply to generated code as well.

https://github.com/golang/go/issues/28089 is about adding this API to x/tools, which is official and more discoverable.
I still think there's no strong argument in favor of adding it straight to go/analysis. At least, most of the analyzers I've written apply to generated code as well.

Thanks for pointing out #28089.  I somehow didn't come across it in my search.
It looks like the resolution of that issue will make for an easily discoverable API.  I'll close this issue in favor of that.

Thanks for pointing out #28089.  I somehow didn't come across it in my search.
It looks like the resolution of that issue will make for an easily discoverable API.  I'll close this issue in favor of that.


Add --google_cloud_sdk_version flag to object storage benchmark

As the title says. This lets users choose a specific "known good" version of the Google Cloud SDK, which lets them work around issues like https://github.com/GoogleCloudPlatform/gsutil/issues/348 .

@stfeng please take a look.

I just pushed a new commit that makes PKB default to a known good version of the Cloud SDK (96.0.0), along with a TODO to go back to the latest version once the current issue is resolved.

LGTM! Thanks!


Add some quality of life features and fix a corner-case bug in the spec

As I said in the title. Namely:

Checks now store and return a result (boolean). This is used to keep track of success/failure.
The output now ends with a list of failed and inconclusive (returned None) tests.

The only tests that should return inconclusive are TLSVerified (not really a pass/fail, although an error will cause it to fail), URLByIPAddress (which IIRC just asks the user to verify the intended behavior themselves), TLSVerified (will return inconclusive on TLS v1.2 negotiation).

I changed the message for URLSchemeMissing (since I feel like saying an error is disingenuous when the test specifically checks for 59 Bad Request).
Also (I forgot to include this in the original message hence the edit), the corner case bug is that, according to the current spec, 4x, 5x, and 6x codes no longer need to have META (as in, the META is optional). Now, the response class only errors if the code is 1x, 2x, or 3x and doesn't have a META.

solid changes, thanks!


[Flatbuffer] Add flatbuffer_opnode @open sesame 03/13 15:00

Add flatbuffer_opnode.cpp & flatbuffer_opnode.h

For flatbuffer interpreter : bring opnode and make variable
Now only support Fully Connected Layer
Now only support NCHW format tensor

It will be updated
Signed-off-by: DongHak Park<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2114. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @DonghakPark, The last line of a text file must have a newline character. Please append a new line at the end of the line in nntrainer/compiler/flatbuffer_opnode.h.

:octocat: cibot: @DonghakPark, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2114-202302141958270.01513409614563-2bdf1d632889292ef10ed893b8f8ac005c62945d/.

:octocat: cibot: @DonghakPark, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2114-202302150817490.47127890586853-fdd7b9a32aee6286bb344f96becec03820cbee4c/.

:octocat: cibot: @DonghakPark, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2114-202302281051590.42586207389832-8c9d2397f448e69d0768c61abe2e4dbb3c10bced/.

:octocat: cibot: @DonghakPark, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2114-202302281613480.1723210811615-b6be7a854e591a02264e4b5d2cdd6dfe339ab56f/.

:octocat: cibot: @DonghakPark, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2114-202303131506480.93666005134583-70b4e7e10ea4dcca5cbc1676274c1a79ac7a0f89/.


Avoid a failing test affecting other tests' results

Close in favor of #73, which is a better alternative.


Add "FastNoiseSIMD" package

I've followed this guide
step by step carefully. Yes

I've tested this package remotely and have excluded all broken builds.
Here is the links to the Travis/AppVeyor with status "All passed":

https://ci.appveyor.com/project/caseymcc/hunter/builds/23957512
https://travis-ci.org/caseymcc/hunter/builds/521892308

I have submitted CI configs to https://github.com/ingenue/hunter targeting pkg.template branch,
see this merged pull request https://github.com/ingenue/hunter/pull/356

Yeah not compatible, I'll just remove the original info it's there in
github anyway. Don't really want to change the name though as it it will
work like the original with a few just a few modifications.
On Fri, Apr 19, 2019, 10:52 AM Ruslan Baratov<EMAIL_ADDRESS>wrote:

@ruslo commented on this pull request.
In docs/packages/pkg/FastNoiseSIMD.rst
https://github.com/ruslo/hunter/pull/1824#discussion_r277021427:

@@ -0,0 +1,22 @@
+.. spelling::

FastNoiseSIMD

+.. index:: math ; FastNoiseSIMD
+
+.. pkg.FastNoiseSIMD:
+
+FastNoiseSIMD
+=============
+
+-  Original <https://github.com/Auburns/FastNoiseSIMD>_
+-  Official <https://github.com/caseymcc/FastNoiseSIMD>__
I modified it heavily to templates and altered its interface
So they are not compatible? Would it be better to rename the project and
introduce completely new API? You can leave the "Originally based on ...
work" note somewhere.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ruslo/hunter/pull/1824#discussion_r277021427, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWE6D5UGLUS3C6JPYN7ZQDPRHTFDANCNFSM4HHAX26Q
.

Yeah not compatible, I'll just remove the original info it's there in github anyway. Don't really want to change the name though as it it will work like the original with a few just a few modifications.

That is pretty strange from the user point of view. You hack the API, you're not planning to send your patches upstream, you haven't changed the name of the project and just set it here as "official".

I did send the changes up stream about 6 months ago, no word.
However I will go ahead and change the name to something else.
On Mon, Apr 22, 2019, 7:18 AM Ruslan Baratov<EMAIL_ADDRESS>wrote:

Yeah not compatible, I'll just remove the original info it's there in
github anyway. Don't really want to change the name though as it it will
work like the original with a few just a few modifications.
That is pretty strange from the user point of view. You hack the API,
you're not planning to send your patches upstream, you haven't changed the
name of the project and just set it here as "official".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ruslo/hunter/pull/1824#issuecomment-485404642, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWE6D5EUSZWWIPJ57KUVWDPRWUKTANCNFSM4HHAX26Q
.

I did send the changes up stream about 6 months ago, no word.

Hm, yeah I don't see much activity there. Have you tried to contact the author https://github.com/Auburns ?

He works for "No Man's Sky"(https://www.nomanssky.com/) now, I am guessing this kind of open source work is off limits for him now. I will just re-name it and re-upload. Ill go ahead and close this PR and send a new one when I get the changes made. You will likely need to revert the pkg.fastnoisesimd merge.

I am guessing this kind of open source work is off limits for him now

My point was: if he is not planning to maintain the project you can ask him to give you such a role. Just adding a note to the README about your fork and pressing "Archive this repository" is not something that will require a lot of time.

I will just re-name it and re-upload

It sounds good for me too.

You will likely need to revert the pkg.fastnoisesimd merge.

Ok, I will remove pkg.fastnoisesimd branch.


Issues with BT and FFMpeg extension

Version
Media3 1.3.1
More version details
No response
Devices that reproduce the issue
Android: 10 - 29 [armeabi-v7a,armeabi] / Infinix - Infinix X657C [X657C-GL - Infinix-X657C]
Devices that do not reproduce the issue
Most of them
Reproducible in the demo app?
Not tested
Reproduction steps
Play music when connected to BT on that device when using ffmpegextension.
Expected result
Should work
Actual result
Fails:
2024-04-30 23:08:46.366 Error/ExoPlayer: DecoderAudioRenderer - Audio sink error
q4.t: AudioTrack init failed 0 Config(48000, 12, 1152000) Format(null, null, null, audio/raw, null, -1, null, [-1, -1, -1.0, null], [2, 48000])
	at q4.j0.a(Unknown Source:61)
	at q4.p0.u(Unknown Source:22)
	at q4.p0.y(Unknown Source:164)
	at zy.a.y(Unknown Source:4)
	at androidx.media3.decoder.ffmpeg.b.F(Unknown Source:256)
	at androidx.media3.decoder.ffmpeg.b.y(Unknown Source:99)
	at o4.e0.e(Unknown Source:1017)
	at o4.e0.handleMessage(Unknown Source:308)
	at android.os.Handler.dispatchMessage(Handler.java:103)
	at android.os.Looper.loop(Looper.java:264)
	at android.os.HandlerThread.run(HandlerThread.java:67)
	Suppressed: q4.t: AudioTrack init failed 0 Config(48000, 12, 1000000) Format(null, null, null, audio/raw, null, -1, null, [-1, -1, -1.0, null], [2, 48000])
		at q4.j0.a(Unknown Source:61)
		at q4.p0.u(Unknown Source:96)
		... 9 more
	Caused by: java.lang.UnsupportedOperationException: Cannot create AudioTrack
		at android.media.AudioTrack$Builder.build(AudioTrack.java:1018)
		at q4.j0.b(Unknown Source:60)
		at q4.j0.a(Unknown Source:4)
		... 10 more
Caused by: java.lang.UnsupportedOperationException: Cannot create AudioTrack
	at android.media.AudioTrack$Builder.build(AudioTrack.java:1018)
	at q4.j0.b(Unknown Source:60)
	at q4.j0.a(Unknown Source:4)
	... 10 more

2024-04-30 23:08:46.367 Error/ExoPlayer: DecoderAudioRenderer - Audio sink error
q4.t: AudioTrack init failed 0 Config(48000, 12, 1000000) Format(null, null, null, audio/raw, null, -1, null, [-1, -1, -1.0, null], [2, 48000])
	at q4.j0.a(Unknown Source:61)
	at q4.p0.u(Unknown Source:96)
	at q4.p0.y(Unknown Source:164)
	at zy.a.y(Unknown Source:4)
	at androidx.media3.decoder.ffmpeg.b.F(Unknown Source:256)
	at androidx.media3.decoder.ffmpeg.b.y(Unknown Source:99)
	at o4.e0.e(Unknown Source:1017)
	at o4.e0.handleMessage(Unknown Source:308)
	at android.os.Handler.dispatchMessage(Handler.java:103)
	at android.os.Looper.loop(Looper.java:264)
	at android.os.HandlerThread.run(HandlerThread.java:67)
Caused by: java.lang.UnsupportedOperationException: Cannot create AudioTrack
	at android.media.AudioTrack$Builder.build(AudioTrack.java:1018)
	at q4.j0.b(Unknown Source:60)
	at q4.j0.a(Unknown Source:4)
	... 10 more

The dump contains:
05-01 12:13:12.471  1000   480  1268 I BufferQueueProducer: [app.symfonik.music.player/app.symfonik.ui.MainActivity#0](this:0xb1663000,id:16642,api:1,p:6695,c:480) queueBuffer: fps=51.44 dur=1010.79 max=92.12 min=11.33
05-01 12:13:12.477 11580  6695  6774 E IAudioFlinger: createTrack returned error -12
05-01 12:13:12.477 11580  6695  6774 E AudioTrack: createTrack_l(-1946121216): AudioFlinger could not create track, status: -12 output 0
05-01 12:13:12.477 11580  6695  6774 E AudioTrack: set(): createTrack_l fail! status = -12
05-01 12:13:12.477 11580  6695  6774 E AudioTrack-JNI: Error -12 initializing AudioTrack
05-01 12:13:12.477  1000   468 11829 W IMGMemtrackHAL: hal_get_memory: memtrack cache rebuild was required
05-01 12:13:12.477 11580  6695  6774 E android.media.AudioTrack: Error code -20 when initializing AudioTrack.

I can ask the user if he allows me to send you the full dump if it's needed.
As a side note I've also reported https://github.com/androidx/media/issues/787 that was completely ignored, but is in fact maybe related, the devices who have that other issues also only have it when using the ffmpeg extension.
Media
Seems to happen on most media for that device.
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

@tonihei not using ffmpeg and not making any other change does fix it for the user.
(I have an option that just change the order of the factory)
This is the same for the other linked issue, not using ffmpeg fixes it.
Seems there's something different the ffmpeg does that can impact BT on some devices but I'm clueless.
If you have ideas of hidden stuff on the extension I can try that last users is more cooperative so I might get him to run tests.

The only reason I can imagine at the moment is that using FFmpeg causes too much memory to be used, leaving not enough memory for the AudioTrack. I'm not sure if this is a reasonable hypothesis, but it could be related to the fact that both need to allocate native memory outside of the Java layer.

That would not explain the other issue that looks similar, I'm currently trying to read code and add some logs on my custom build to help move on those as it's a pain for my users.
I've triple checked and both paths are using the exact same sink:
val audioSink = DefaultAudioSink.Builder(context)
            .setAudioProcessorChain(
                DefaultAudioSink.DefaultAudioProcessorChain(
                    SilenceSkippingAudioProcessor(),
                    SonicAudioProcessor(),
                    replayGainProcessor,
                ),
            )
            .setEnableAudioTrackPlaybackParams(true)
            .setEnableFloatOutput(hires)
            .build()
        if (haveRApi) {
            audioSink.setOffloadMode(offloadMode)
        }

It's just a matter of the order of creation to pass from one case to the other
override fun createRenderers(
        eventHandler: Handler,
        videoRendererEventListener: VideoRendererEventListener,
        audioRendererEventListener: AudioRendererEventListener,
        textRendererOutput: TextOutput,
        metadataRendererOutput: MetadataOutput,
    ): Array<Renderer> {
        return if (preferInternalDecoder) {
            arrayOf(
                getFfmpegRenderer(eventHandler, audioRendererEventListener),
                getDefaultRenderer(eventHandler, audioRendererEventListener),
                MetadataRenderer(metadataRendererOutput, eventHandler.looper),
            )
        } else {
            arrayOf(
                getDefaultRenderer(eventHandler, audioRendererEventListener),
                getFfmpegRenderer(eventHandler, audioRendererEventListener),
                MetadataRenderer(metadataRendererOutput, eventHandler.looper),
            )
        }
    }

Reading internals I think the real cause may be hidden and those devices returns a wrong error code and it's not an issue with memory but just some parameters passed to the sink that will then generate different AudioTrack settings.
Specially since both DecoderAudioRenderer and MediaCodedcAudioRenderer do not create exactly the same the outputFormat passed down.
To be sure I'll add some debug logs at start of DefaultAudioSink.configure() to log the exact full params and see if there's differences with and without ffmpeg for that user.
Do you think about something else that could be useful?
And @tonihei since this is really low level, can you clarify if the AudioTrack itself is somehow related to BT and how? Or it's it's before and then the fact that playing outside BT is working gives another clue. Like a missing event somewhere that prevent releasing the AudioTrack.

To be sure I'll add some debug logs at start of DefaultAudioSink.configure() to log the exact full params and see if there's differences with and without ffmpeg for that user.

That sounds useful to check.

can you clarify if the AudioTrack itself is somehow related to BT and how

AudioTrack is related to BT in that it handles the entire output path of the audio data. So it will behave differently depending the path and also the capabilities of the output device will be different.


Translated text containing \0 is truncated in list view

Poedit 2.0beta3
Windows 10

FWIW, this for once I regard as perfectly reasonable. Poedit as well as gettext tools and the rest of the civilized word use NULs to terminate strings, as the (C) God intended them to be used — it's the normal way of handling strings.
That goes double if the source string does not even use embedded NULs, as in your screenshot. In that case, your translation is quite simply wrong and likely to break things in the software being translated, not just Poedit.
If you meant to write \0 verbatim, then you need to escape the special-purpose character of \ and write it as \\0 (that's what the syntax highlighting is telling you, that the \0 bit is special in some way). Your translation would still be unfaithful to the original then, and thus still wrong, but at least it would not break gcc.

Thanks for this good explanation. I filed a bug against gettext instead: https://savannah.gnu.org/bugs/index.php?50472

I think there's no bug in gettext either. The document I linked to explicitly says it's keeping its options open...

In the gettext bug tracker I commented:
"In fact, the original problem — accidental use of \0 instead of \0 — is due to the backslashing of strings in PO files. I see it as a feature of a good PO editor to be clear about which backslashes are for the string and which backslashes are for the PO file syntax. In other words, it's a quality-of-implementation issue of the PO editor. (And yes, I know that the Emacs po-mode does not have good quality in this sense: it leaves backslashes from the PO file syntax visible to the translator.)"