ERROR: Call to virStoragePoolDefineXML failed: operation failed: pool 'default' already exists

# vagrant up
Vagrant.require_plugin is deprecated and has no effect any longer.
Use `vagrant plugin` commands to manage plugins. This warning will
be removed in the next version of Vagrant.
Bringing machine 'test_vm' up with 'libvirt' provider...
There was error while creating libvirt storage pool: Call to virStoragePoolDefineXML failed: operation failed: pool 'default' already exists with uuid 69266e1a-18f9-4e1f-98ed-d105ad8ba0d5
# vagrant --version
Vagrant 1.7.4
# cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=15.10
DISTRIB_CODENAME=wily
DISTRIB_DESCRIPTION="Ubuntu 15.10"

@pradels: Ping. Any chance this could be resolved?

no chance if you will not sent pull request.

@pronix: hmm... I don't know libvirt enough to fix that. Is there any way you could help me debug that?

you can just rename default pool - it will created, or you should remove existed.

That's interesting! virsh says that the pool isn't there and when I used --provider libvirt instead of exporting that, I got the machine up. Thanks @pronix!

I ran into this issue earlier this morning, it was caused by the default pool being defined but not active:
virsh # pool-list --all
Name                 State      Autostart
default              inactive   no
Downloads            active     yes
...
Activating the default pool resolved the issue:
virsh # pool-start default

Turns out there is a bug in fog-libvirt where it cannot retrieve info of inactive pools.

Hey 8 years later this is still a thing

Hey 8 years later this is still a thing

Same :(


TS error when attempting to invoke .perform() method

When using the addon and attemping to invoke .perform() TS throws the error: Property 'perform' does not exist on type '() => Promise<...>.
The concurrency task is setup as follows:
@restartableTask async foo() {...}
and then invoked with:
this.foo.perform()

This is the same error you would get when using generator functions. TypeScript does know/allow the decorator to change the type of the field. You would have to do a cast ((this.foo as ...).perform()) or better, encapsulate it in a utility function. I'll suggest reading the documentation being added in https://github.com/machty/ember-concurrency/pull/357 and see if that helps (and comment there if it didn't).


Run cp in non verbose mode

for bigger projects cp -v is polluting logs with a lot of useless information

LGTM.


Provide support to localize custom text input via branding & give ability to customize the static content on Identity Server login UIs

Is your feature request related to a problem? Please describe.
ATM, we will have the following problems in IS login UI Branding when the capability is onboarded to IS Console.

Custom text (Copyright, etc.) added via Console's Branding doesn't change when the language is changed. (note that i've selected spanish)

There's no way to customize any static headers or other content in the login UIs.

Ex: If I want to change the Sign In heading to Login, i'm not able to do it easily without applying CSS hacks, etc.

Describe the solution you would prefer
Implement a scalable solution that would support:

Text Customization.
Custom text localization.

@brionmario  can you please verify this checklist

[ ] Is this document impacted?
[ ] Is the end-to-end (e2e) testing complete?
[ ] Are unit tests available?
[ ] Are integration tests available?
[ ] Has this feature been tested on the latest Jenkins build after merging the changes? (Please Attach Proof, Screenshots, Screen recordings)
[ ] Are all the relevant sub-tasks updated as complete (e.g., Cloud readiness tasks and migrations, docs)?
[ ] Have demo sessions been completed? (If yes, please attach the recording link)
[ ] Does this change impact the on-prem environment only or required to be specifically onboarded to Asgardeo?
[ ]
[ ]


new release

Could you make a new release (0.7)? I would not want to add 'dev-master' into composer.json to get current changes/fixes. Latest release does not contain even #59

Should be fixed


Support Sucker Punch

This is an in-memory job queue. As such I don't believe there's value in trying to support this.


Please add license text to the released tarball

I've noticed that the tarball released on pypi is missing the LICENSE file, and while there are references to the MIT license, the only way to know the actual licensing terms is from the git repo.
Also, the code taken from IPython py3compat.py and encoding.py is marked as under the Modified BSD License, but the actual text of that license (plus copyright notice) is not repeated anywhere in the package (as requested by the license: "The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software."). IANAL, but I believe that adding the text of the Modified BSD to LICENSE (including the copyright notice and specifying the parts to which it applies) should be enought.
Thanks in advance

Is it enough if I add LICENSE to MANIFEST.in? I don't mind accepting a PR for this. As far as I'm concerned the source is under MIT (a nominal thing really).
I barely develop Python these days so I'm pretty out of the loop with these things. :+1:


Receive only coherent build from BAR for source-build updates

What we need from arcade: an option in Maestro++ to only create an update PR if a core-sdk build is coherent.
Nice to have: an indicator of coherence in build info.

/cc @mmitche

Depends on https://github.com/dotnet/arcade/issues/1923 and https://github.com/dotnet/arcade-services/pull/128.

@crummel This likely means that you'll only get a PR like...on the last build, and never before.
There is a coherency note in the darc dependency information (get-dependency-graph) today.

Done.  Waiting for updates from DARC.


Convert interface to use Eigen

Please rename on master

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d2d6746). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   31.11%
=========================================
  Files             ?        2
  Lines             ?      180
  Branches          ?      116
=========================================
  Hits              ?       56
  Misses            ?       20
  Partials          ?      104

Flag
Coverage Δ

unittests
31.11% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[Build issue] Facing issues building TensorFlowLiteTaskText

I am seeing the following error when trying to build TensorFlowLiteTaskText:
$ make all
ERROR: /Users/maxwellelliott/Development/PodToBUILD/Examples/TensorFlowLiteTaskText/Vendor/TensorFlowLiteTaskText/BUILD.bazel:273:13: C++ compilation of rule '//Vendor/TensorFlowLiteTaskText:TensorFlowLiteTaskText' failed (Exit 1)
Vendor/TensorFlowLiteTaskText/tensorflow_lite_support/ios/task/text/nlclassifier/Sources/TFLNLClassifier.m:16:9: fatal error: 'GTMDefines.h' file not found
#import "GTMDefines.h"

How can I go about fixing this header import issue? Thanks

Closing, it seems there was an error in GoogleToolboxForMac


Letter of Commendation

This solution supports the mandatory national standard GB18030, providing great convenience for developers and receiving unanimous recognition from the people with rare characters in their names. Ant Group has practiced the spirit of openness, sharing, and cooperation with practical actions, making positive contributions to the development of Chinese information processing technology and benefiting the present and the future.

Thank you, my friend.


NTRs for Dietary Reference Intake terms

This Is a request for a handful of standard Dietary Reference Intake terms, held in this Google Sheet ready for comment.
https://docs.google.com/spreadsheets/d/1-_O51W78nPFCkvfBRufz-XEberdfldOTCbPMkEsUv4kg
Francesco you have edit permission on it. A preview of the terms is below:

parent class
label
definition
definition source

data item
dietary reference intake
A data item which is a quantitative estimate of nutrient intake to be used as a reference value for planning and assessing diets for healthy people.

dietary reference intake
average daily nutrient intake level
A dietary reference intake which is an average of daily nutrient intake values to be used as a reference for planning and assessing diets for healthy people.
https://www.nap.edu/download/10490

average daily nutrient intake level
recommended dietary allowance
An average daily nutrient intake level which is sufficient to meet the nutrient requirement of nearly all (97 to 98 percent) healthy individuals in a group.
https://www.nap.edu/download/10490

average daily nutrient intake level
adequate intake
An average daily nutrient intake level which is recommended based on observed or experimentally determined approximations or estimates of nutrient intake by a group (or groups) of apparently healthy people that are assumed to be adequate with respect to deficiency or disease criteria.
https://www.nap.edu/download/10490

average daily nutrient intake level
tolerable upper intake level
An average daily nutrient intake level which is the highest level that is likely to pose no risk of adverse health effects to almost all individuals in the general population.
https://www.nap.edu/download/10490

average daily nutrient intake level
estimated average requirement
An average daily nutrient intake level which is estimated to meet the requirement of half the healthy individuals in a particular life stage and gender group.
https://www.nap.edu/download/10490

average daily nutrient intake level
Estimated Energy Requirement
An average daily nutrient intake level which is predicted to maintain energy balance in healthy, normal weight individuals of a defined age, gender, weight, height, and level of physical activity consistent with good health.
https://www.nap.edu/download/10490

dietary reference intake
Acceptable Macronutrient Distribution Range
A dietary reference intake data item which is a range of intake for a particular energy source (protein, fat, or carbohydrate), expressed as a percentage of total energy (kcal), that is associated with reduced risk of chronic disease while providing adequate intakes of essential nutrients.
https://www.nap.edu/download/10490

I've linked this in for comment from the JFOW group.

@ddooley proposed classes were created in the pre-release version (ons-edit.owl)
Just a note, the URL for definition source point to a page not accessible to anyone, asking for a login to "MYNAP"
I was not able to open the spreadsheet (file does not exist) to check for other desired annotations or axioms. If any addition to the classes has to be done, let me know.
Keeping this open for further discussion, if there is nothing to add I'll close

Hi,
Ah, somehow an extra "g" got added on to the sheet URL, my bad.  The link should be: https://docs.google.com/spreadsheets/d/1-_O51W78nPFCkvfBRufz-XEberdfldOTCbPMkEsUv4k/edit .  There were just a few other rdfs:comments that could be added from there.
And here is a better URL for the document reference:
https://www.nap.edu/catalog/10490/dietary-reference-intakes-for-energy-carbohydrate-fiber-fat-fatty-acids-cholesterol-protein-and-amino-acids
That is publcily accessable without login, and the pdf is free to download.

Thanks!

p.s. there are abbreviations and one example of usage that should be included too.

Something I see in your ons-edit.owl file - it seems like you have merged the import files directly into it.  Have you considered leaving the import files as separate files, which are linked in ons-edit.owl rather than merged in it?  Then ons-edit.owl primarily holds ons terms and little else.

Actually imports are not merged in the main file, rather they are automatically imported when opening ons.owl (or ons-edit.owl) as they are declared as imports in
<owl:imports rdf:resource="https://raw.githubusercontent.com/enpadasi/Ontology-for-Nutritional-Studies/master/src/ontology/imports/ONS_imports.owl"/>
In Protègè, in fact, you can remove the imports and loose annotations. Isn't that the expected behavior?

https://www.nap.edu/catalog/10490/dietary-reference-intakes-for-energy-carbohydrate-fiber-fat-fatty-acids-cholesterol-protein-and-amino-acids
That is publcily accessable without login, and the pdf is free to download.

Yes, just saw the "download as guest" simply asking for a mail.
Updating!

You should check ons-edit.owl because full FOODON terms do appear to be in that file, and there are no imports.  Same with ons.owl  - that is why they are each above 4.2Mb.  I see ons-edit.owl does have an imports/ontofox_import.owl file, but if you disconnect it I think you will still find lots of FOODON_12345678 terms fully defined in the main file.

p.s. I just learned how to remove them automatically with a ROBOT command that takes out from the main file everything that is present in an import file, so can help with that part.

Yes, some terms are for sure fully defined in main ons.owl

p.s. I just learned how to remove them automatically with a ROBOT command that takes out from the main file everything that is present in an import file, so can help with that part.

that was what I exactly searched for! You mean taking out entire class, or their annotations? that would be useful to also remove double annotation problem #15

This should work:
robot convert --input imports/ontofox_import.owl --format ofn --output temp-import.ofn
grep -v '^Declaration' source-import.ofn > temp.ofn
robot unmerge --input ons_edit.owl --input temp.ofn --output ons-edit-2.owl
Check ons_edit_2.owl to see if that's ok (all classes which are described in import file are now removed from ons_edit_2.owl), and if so, rename it back to ons_edit.owl
You can do the same for foodon_import.owl if you want if its content is different from what is in ontofox_import.owl .  You can see how I got Nico's help to figure that out at bottom of this thread: https://github.com/ontodev/robot/issues/848

Cited the above comment on issue #15 which is dedicated to the problem of multiple annotations.
The requested terms are now in /src/ontology/ons-edit.owl they will be in /master/ons.owl at next release (in few days I suppose)


[FEATURE REQ] Add python 3.9 to test workflow

Is your feature request related to a problem? Please describe.
The tests GitHub actions workflow currently uses python versions 3.6, 3.7 and 3.8. Since 3.9 was released a few months ago and browser-history works well on it, we should add 3.9 to the test workflow to ensure that it stays that way.
Describe the solution you'd like
Add 3.9 to the python-version list here:
https://github.com/pesos/browser-history/blob/42520ac23189907a1c52661b774e3122bc653e78/.github/workflows/python-package.yml#L16-L18

I'm happy to work on it.

You can go ahead and send a PR!


fix footnote ID links in CONTRIBUTING.md

[x] I have followed (at least) the PR section of the contributing guide.

Hey, any updates?


Yarn build fails with [ERR_UNSUPPORTED_ESM_URL_SCHEME]:  in windows

The build is failing on the below error on
OS windows 11 ,
Node 16.14.2
Yarn 3.2.4
@showwhy/discover-app:assets: explore workers bundled!
@showwhy/webapp:assets: cache miss, executing 48d45a860ef8ac5c
@showwhy/webapp:build: cache miss, executing d9e4fc173ef26bc4
@showwhy/webapp:build:     ✔ check types (10156.50ms)
@showwhy/webapp:build: error running webpack build Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only file and data URLs are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'
@showwhy/webapp:build:     at new NodeError (node:internal/errors:371:5)
@showwhy/webapp:build:     at throwIfUnsupportedURLProtocol (node:internal/modules/esm/resolve:1035:11)
@showwhy/webapp:build:     at defaultResolve (node:internal/modules/esm/resolve:1105:3)
@showwhy/webapp:build:     at resolve$1 (file:///C:/softwares/tools/causal/showwhy-main/showwhy-main/.pnp.loader.mjs:1963:14)
@showwhy/webapp:build:     at ESMLoader.resolve (node:internal/modules/esm/loader:530:30)
@showwhy/webapp:build:     at ESMLoader.getModuleJob (node:internal/modules/esm/loader:251:18)
@showwhy/webapp:build:     at ESMLoader.import (node:internal/modules/esm/loader:332:22)
@showwhy/webapp:build:     at importModuleDynamically (node:internal/modules/esm/translators:106:35)
@showwhy/webapp:build:     at importModuleDynamicallyCallback (node:internal/process/esm_loader:35:14)
@showwhy/webapp:build:     at getConfig (file:///C:/softwares/tools/causal/showwhy-main/showwhy-main/.yarn/virtual/@essex-scripts-virtual-f4c528482d/0/cache/@essex-scripts-npm-22.2.0-4e53cad6af-03aa5393a1.zip/node_modules/@essex/scripts/lib/steps/webpack/getConfig.mjs:24:24) {
@showwhy/webapp:build:   code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'
@showwhy/webapp:build: }
@showwhy/webapp:build: 🔥 bundle
@showwhy/webapp:build: Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only file and data URLs are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'
@showwhy/webapp:build:     at new NodeError (node:internal/errors:371:5)
@showwhy/webapp:build:     at throwIfUnsupportedURLProtocol (node:internal/modules/esm/resolve:1035:11)
@showwhy/webapp:build:     at defaultResolve (node:internal/modules/esm/resolve:1105:3)
@showwhy/webapp:build:     at resolve$1 (file:///C:/softwares/tools/causal/showwhy-main/showwhy-main/.pnp.loader.mjs:1963:14)
@showwhy/webapp:build:     at ESMLoader.resolve (node:internal/modules/esm/loader:530:30)
@showwhy/webapp:build:     at ESMLoader.getModuleJob (node:internal/modules/esm/loader:251:18)
@showwhy/webapp:build:     at ESMLoader.import (node:internal/modules/esm/loader:332:22)
@showwhy/webapp:build:     at importModuleDynamically (node:internal/modules/esm/translators:106:35)
@showwhy/webapp:build:     at importModuleDynamicallyCallback (node:internal/process/esm_loader:35:14)
@showwhy/webapp:build:     at getConfig (file:///C:/softwares/tools/causal/showwhy-main/showwhy-main/.yarn/virtual/@essex-scripts-virtual-f4c528482d/0/cache/@essex-scripts-npm-22.2.0-4e53cad6af-03aa5393a1.zip/node_modules/@essex/scripts/lib/steps/webpack/getConfig.mjs:24:24) {
@showwhy/webapp:build:   code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'
@showwhy/webapp:build: }

Hi @san-sankar, we're currently using WSL , could you test with that while we verify why isn't it working on windows?

Hi @dayesouza  i will install WSL , test and get back to you.


Treat uglify-js as empty by default

This fixes the pug build, by always ignoring uglify-js builds. I know it doesn't sound ideal, but that really is the best fix we can come up with here given there isn't an analysis solution with the current technology.
When this happens the following warning is output:
ncc: Ignoring build of uglify-js, as it is not statically analyzable. Build with "--external uglify-js" if this package is needed.

I think this is a good solution for now, considering the popularity of the module

Codecov Report

Merging #307 into master will increase coverage by 1.61%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
+ Coverage   66.76%   68.37%   +1.61%
==========================================
  Files          11       12       +1
  Lines         352      370      +18
==========================================
+ Hits          235      253      +18
  Misses        117      117

Impacted Files
Coverage Δ

src/index.js
69.53% <ø> (ø)
:arrow_up:

src/loaders/empty-loader.js
100% <100%> (ø)

Continue to review full report at Codecov.

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


Updated SearchInputByRole on Generic Defn. config

JIRA link (if applicable)
Change description
Updated SearchInputByRole on Generic Defn. config by Richard Power
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

closing this without merging, as this branch doesn't contain anything on top of what is already available on DefnUpdate-SearchInputByRole.


Switch to GeometryBasics

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines           45        45
=========================================
  Hits            45        45

Impacted Files
Coverage Δ

src/ConstrainedDynamicsVis.jl
100.00% <ø> (ø)

src/convertshape.jl
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


ZEPPELIN-3581. Add "type": "textarea" to "default.statementPrecode" in interpreter-setting.json

What is this PR for?
Fix "interpreter-setting.json" for JDBC interpreter
What type of PR is it?
Bug Fix
What is the Jira issue?
ZEPPELIN-3581
How should this be tested?

Check text area near the "default.statementPrecode" setting.

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

CI green. Merge this to master.


WIP: Network Refactor

Objectives

Support GSM modems
Support Ethernet featherwings
Support Wifi in AP mode

There is still some remaining work to here to investigate whether it's possible to automatically detect the network hardware and switch to the correct network settings.
The benefit of this is that for users building the Frog kit with a cellular modem, wouldn't have to use Curl / a command line tool to change the network config and existing Frogs could be updated with no config changes.
I am imagining for example the priority would be:

Ethernet
Wifi
Cellular

So the algorithm in my head would be:

IF Ethernet hardware is installed

THEN set the config to ethernet

ELSE IF Wifi configuration (SSID and password) are set

THEN set config to WIFI

ELSE Then assume a cellular connection and set config to cellular


Custom report templates (including modified plot & sort) - make them sticky across projects

It would be great if once a "custom report template" was imported, it remained available in the drop down list for all future projects (i.e. did not need to be re-imported every time).  I think that most labs are going to have 1 template (or rather one template per mineral/experiment type) that they use over and over, so not having to re-import it every time would be great.  I'm flexible on how best to achieve this - for example with a "make default" check box or something?
If the same could be done for the "plot and sort" report, that would be great too (so that any edits that are saved are then ported over to subsequent files) however this would be a secondary request.

done in next release


discussion: Should we keep envd login subcommand?

Description

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Can you give more information about the discussion? @gaocegege

We will need envd login to login (and register) in the envd server. Then you can create the environments on envd-server.
But we can also create such a user when users run envd context create.
This issue is to discuss which one is better.

I still think envd login won't affect the core user journey. It will only need to be executed once. And user can disable auth at the server side

/cc @aseaday @kemingy

I think we are talking about several things together.

Shall we provide a way for users that do not want to log in with a username & password? (mainly because there is only one user or it's not necessary for them)
Shall we help users to log in when they create the context?

Yes, and I think it should be configured on the envd-server side to allow anonymous users (but we should generate a token when creating the context to identify the user, the name can be a random string)
No. I agree with https://github.com/tensorchord/envd/issues/1251#issuecomment-1336952859.

Shall we provide a way for users that do not want to log in with a username & password? (mainly because there is only one user or it's not necessary for them)

I think so. So @gaocegege already has no-auth mode in JWT pr, which won't check the user's password when envd login. (any username/password works)

I think we are talking about several things together.

Shall we provide a way for users that do not want to log in with a username & password? (mainly because there is only one user or it's not necessary for them)
Shall we help users to log in when they create the context?

no-auth mode supported for anonymous users and auth mode for registed user lgtm.
I aggre with allen`s reason,my comment can be ignored.

Shall we provide a way for users that do not want to log in with a username & password? (mainly because there is only one user or it's not necessary for them)

We already support this in the auth PR in envd-server.

I think envd login we should keep this command. But the interactive process of envd login shoule be considerated seriously.


cannot find symbol

Hey,
So I've been having this error for a while :
viewsetting\WebviewSetting.java:31: error: cannot find symbol
                    webView.getSettings().setLoadWithOverviewMode(true);
                           ^
  symbol:   method getSettings()
  location: variable webView of type CordovaWebView

For what i've seen by looking it up, it looks like getSettings is no longer available on webView ? Do you know anything about that ? Sorry if it's a noob question :)

Same issue here. Did you find a solution @corentin-gautier ?

Opened and updated cordova in my old project, sadly I have now the same error and no solution. Any idea? Tks in advance.

if anyone else comes upon this, this fork fixed it for me:
https://github.com/RomaRogov/cordova-webview-setting


fix(backfill): handle backwards compatibility case

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Thanks to @chenzl25 for finding the reason and suggesting the fix.

Checklist For Contributors

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[ ] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[ ] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

how about making a PR to the v0.19.0-rc branch too


Adds inline annotation to the compiler

inline_function_calls' inline_heuristic now checks inline metadata and
if inline metadata is set to Inline::Never it returns false and the
function is not inlined.
When inline metadata is set to Inline::Always it will still follow the
default behavior, I left as TODO: check if inlining of function is
possible.
ref #2944

I tried to write a test into test/ir_generation but it is compiling IR without running optimizations should we create an issue to add support for running optimizations as in sway-core/src/lib.rs#L394?

I tried to write a test into test/ir_generation but it is compiling IR without running optimizations should we create an issue to add support for running optimizations as in sway-core/src/lib.rs#L394?

Thank you for this PR! You may want to look at https://github.com/FuelLabs/sway/tree/master/sway-ir/tests/inline :)

This generally seems reasonable to me. Thank you so much for your contribution 😄
I'd like @otrho to review this and give his approval as he's our resident IR expert.

Thanks @esdrubal for looking at this.  There's a couple  of issues I have with the design though...
I'm not sure that adding the inline member to each of those data structures, essentially pairing up with purity, is the best long term approach.  If we were to do this for every annotation we add in the future then those data structures (most of which don't really care about the values, they're just passing them on) will bloat out a fair bit.  I think it'd be better to generalise annotations into a map or something which can be queried when needed.  I'm assuming that would work OK, and then be general enough for further annotations.  But I'm curious what @sezna and @mitchmindtree think.
Also, yep, ideally we'd have fine grained control of which opt passes are run per test, though at the same time I think those tests are more about IR generation and not full compilation, and so they shouldn't have optimisations applied.
But it's tricky, since the unit tests under sway-ir/tests are better suited to testing the actual inlining except they're not supposed to really understand metadata, so confirming that the metadata works is still best suited for the compiler tests.
Hrmm, sorry, I'm not really helping here.  So I guess we need to add inlining to the ir_generation tests, but the ::optimisation-inline:: tag isn't right -- the other two are designed to be boundary markers for FileCheck.  So maybe, like the first line of the sway-ir/tests/inline files, we should have parameters describing which opt passes should be applied.

I think it'd be better to generalise annotations into a map or something which can be queried when needed.

That is certainly a better long term approach. A bit like the generalized metadata map that you created.

I think it'd be better to generalise annotations into a map or something which can be queried when needed. I'm assuming that would work OK, and then be general enough for further annotations. But I'm curious what @sezna and @mitchmindtree think.

Yeah, the TyFunctionDeclaration already has an attributes map. By introducing an inline field we end up storing the same information in two locations.
Perhaps we're better off exposing a method on TyFunctionDeclaration that acts as a short-hand for checking the attribute map like:
    pub fn inline(&self) -> Option<Inline>

This would be similar to how the TyFunctionDeclaration::is_test method works (it just checks the attributes map for a #[test] attribute).
By returning an Option, we can also remove the Inline::Default variant in favour of handling None.

Thank you all for the feedback.
This new version implements an inline method in TyFunctionDeclaration instead of adding inline field to it. This method uses TyFunctionDeclaration's attributes.
The tests no longer check for ::optimisation-inline:: instead they check if there is a comment in the first line with optimisation-inline.

@Centril Thank you for the requested changes, looks much better now.
I applied all the changes except moving module inline into an existing module. Not sure where to put it.


Testing github actions; please ignore

Will close when done.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
460 unchanged lines in 13 files lost coverage.
Overall coverage decreased (-28.9%) to 57.044%

Files with Coverage Reduction
New Missed Lines
%

src/mcmc/emcee.jl
2
90.2%

src/variational/VariationalInference.jl
4
14.29%

src/mcmc/gibbs.jl
4
94.05%

src/mcmc/particle_mcmc.jl
10
87.84%

src/mcmc/gibbs_conditional.jl
12
0.0%

src/mcmc/Inference.jl
18
77.3%

ext/TuringDynamicHMCExt.jl
29
0.0%

src/mcmc/mh.jl
32
57.43%

ext/TuringOptimExt.jl
50
0.0%

src/variational/advi.jl
53
0.0%

Totals

Change from base Build<PHONE_NUMBER>4:
-28.9%

Covered Lines:
911

Relevant Lines:
1597

💛  - Coveralls


Add support for user-defined client JS hooks

Allow users to pass https://hexdocs.pm/phoenix_live_view/js-interop.html#client-hooks-via-phx-hook to the Beacon livesocket.

For now according to the team in the backlog. Could come back in Q4


When will this be published?

Not an issue but a question. I would like to start using this module but I don't see it published in Terraform Modules.
Thanks

@apeabody @anaik91

Hi @bharathkkb - Can you recommend someone?

I mean I know how to use this but i don't wanna have to clone this repo and reference it locally, I wanna use the github source if that makes any sense

v0.1.0 is published.


Create Rules class

Create Rules class to encapsulate operations on rules:

reading
compiling
writing

Simplify compile-rules by using Rules class

Note: In the future I plan to add a check method to this
class to verify that the rules are in the correct format.
Minor:

Add php to example.json

:+1:


CICD does not validate CDDL properly

Seems like the file cannot be found but it's not failing the build. Gotta investigate.

The docker image doesn't fail if the file is not found, only if the file is invalid.

https://github.com/anweiss/cddl/pull/144 will make sure this doesn't happen again.


[SPARK-39545][SQL] Override concat method for ExpressionSet in Scala 2.13 to improve the performance

What changes were proposed in this pull request?
ExpressionSet ++ method in the master branch a little slower than the branch-3.3 with Scala-2.13, so this pr override concat method for ExpressionSet in Scala 2.13.
Why are the changes needed?
Improve the performance
Does this PR introduce any user-facing change?
No
How was this patch tested?

Pass GA
Manual test 1:

microbench as follows and run with Scala 2.13:
    val valuesPerIteration = 100000
    val benchmark = new Benchmark("Test ExpressionSet ++ ", valuesPerIteration, output = output)
    val aUpper = AttributeReference("A", IntegerType)(exprId = ExprId(1))
    val initialSet = ExpressionSet(aUpper + 1 :: Rand(0) :: Nil)
    val setToAddWithSameDeterministicExpression = ExpressionSet(aUpper + 1 :: Rand(0) :: Nil)

    benchmark.addCase("Test ++") { _: Int =>
      for (_ <- 0L until valuesPerIteration) {
        initialSet ++ setToAddWithSameDeterministicExpression
      }
    }

    benchmark.run()

branch-3.3 result:
OpenJDK 64-Bit Server VM 1.8.0_312-b07 on Linux 4.14.0_1-0-0-45
Intel(R) Xeon(R) Gold 6XXXC CPU @ 2.60GHz
Test ExpressionSet ++ :                   Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------------------------------
Test ++                                              14             16           4          7.2         139.1       1.0X

master result before this pr:
OpenJDK 64-Bit Server VM 1.8.0_312-b07 on Linux 4.14.0_1-0-0-45
Intel(R) Xeon(R) Gold 6XXXC CPU @ 2.60GHz
Test ExpressionSet ++ :                   Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------------------------------
Test ++                                              16             19           5          6.1         163.9       1.0X

master result after this pr:
OpenJDK 64-Bit Server VM 1.8.0_312-b07 on Linux 4.14.0_1-0-0-45
Intel(R) Xeon(R) Gold 6XXXC CPU @ 2.60GHz
Test ExpressionSet ++ :                   Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------------------------------
Test ++                                              12             13           3          8.6         115.7       1.0X

Manual test 2:

dev/change-scala-version.sh 2.13
mvn clean install -pl sql/core -am -DskipTests -Pscala-2.13
mvn test -pl sql/catalyst -Pscala-2.13
mvn test -pl sql/core -Pscala-2.13

Run completed in 10 minutes, 40 seconds.
Total number of tests run: 6584
Suites: completed 285, aborted 0
Tests: succeeded 6584, failed 0, canceled 0, ignored 5, pending 0
All tests passed.

Run completed in 1 hour, 27 minutes, 16 seconds.
Total number of tests run: 11745
Suites: completed 520, aborted 0
Tests: succeeded 11745, failed 0, canceled 7, ignored 57, pending 0
All tests passed.

late LGTM

thanks all ~


Add MautiCon India to 2024 page

This adds MautiCon India to the 2024 page as requested!

Thanks  :)


No examples

There is nothing in the example project.

I have not had the time to finish this project, but as I see that there's interest I'll do it soon.
@tecknut expect updates 😄

Count me as interested also. hopefully yours will maintain audio support for filtered videos. I found swift iPad code that I've been trying to get the audio working but I still don't fully understand avassetwriter enough to make it work with audio and video simultaneously.


Use same SLES image as Rancher

In addition to using the SLES image for the aks-operator image, the
dapper image is also switched to using SLES.
A side effect of this is that docker will no longer be installed in the
dapper image. Instead, a new pipeline step is added to build the image
on pull requests.

Should wait to merge other PRs until this one is re-reviewed by others

PR updated to remove docker from the dapper image and use the docker plugin for building on PRs.


DCOS-8058: Add client side resource validation

---
ℹ  You will see these validation errors only on the second click or after switching the tabs. This is know issue which will hopefully be fixed with https://github.com/mesosphere/reactjs-components/pull/303.
/cc @kennyt

Reviewer: @philipnrmn

LGTM!


python312Packages.pylamarzocco: 1.4.1 -> 1.4.3

Diff: https://github.com/zweckj/pylamarzocco/compare/refs/tags/v1.4.1...v1.4.3
Changelog: https://github.com/zweckj/pylamarzocco/releases/tag/v1.4.3

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 368776

x86_64-linux

:white_check_mark: 4 packages built:

python312Packages.pylamarzocco
python312Packages.pylamarzocco.dist
python313Packages.pylamarzocco
python313Packages.pylamarzocco.dist


[Q] How to avoid data averaging when logging to metric across multiple runs?

Hello, I'm trying to log data points for the same metric across multiple runs (wandb.init is called repeatedly in between each data point) and I'm unsure how to avoid the behavior seen in the attached screenshot...
Instead of getting a line chart with multiple points, I'm getting a single data point with associated statistics. In the attached e.g., the 1st data point was generated at step 1,470 and the 2nd at step 2,940...rather than seeing two points, I'm instead getting a single point that's the average and appears at step 2,205.

My hunch is that using the resume run feature may address my problem, but even testing out this hunch is proving to be cumbersome given the constraints of the system I'm working with...
Before I invest more time in my hypothesized solution, could someone confirm that the behavior I'm seeing is, indeed, the result of logging data to the same metric across separate runs without using the resume feature?
If this is the case, can you confirm or deny my conception of how to use resume?
Initial run:

run = wandb.init()
wandb_id = run.id
cache wandb_id for successive runs

Successive run:

retrieve wandb_id from cache
wandb.init(id=wandb_id, resume="must")

Is it also acceptable / preferable to replace 1. and 2. of the initial run with:

wandb_id = wandb.util.generate_id()
wandb.init(id=wandb_id)

hey @a-rich thank you for reporting this! Do you have a code snippet on how you log these data? and is it possible to share a workspace of this chart? there might be a way to change how data are plotted using Weave. Do you want to plot something like the mean loss (y-axis) vs epochs (x-axis) for each experiment, so that would be n experiments line plots?

Hi @thanos-wandb!
I'm not able to share the workspace where this is actually happening, but I made a personal account and can share a workspace from there that demonstrates the issue.

Here's a snippet to reproduce the group in the linked demo workspace seen in the attached screenshot:
import wandb

# wandb run associated with evaluation after first N epochs of training.
wandb.init(project="alrichards", name="test-run-1/job-1", group="test-run-1")
wandb.log({"mean_evaluate_loss_epoch": 20}, step=1)
wandb.finish()

# wandb run associated with evaluation after second N epochs of training.
wandb.init(project="alrichards", name="test-run-1/job-2", group="test-run-1")
wandb.log({"mean_evaluate_loss_epoch": 10}, step=5)
wandb.finish()

Ok, I think my original question is now answered. Using the id and resume fields yield the desired effect. Now I can invest time making changes to support this as a feature.
import wandb

# wandb run associated with evaluation after first N epochs of training.
wandb_id = wandb.util.generate_id()
wandb.init(id=wandb_id, project="alrichards", name="test-run-3/job-1", group="test-run-3")
wandb.log({"mean_evaluate_loss_epoch": 20}, step=1)
wandb.finish()

# wandb run associated with evaluation after second N epochs of training.
wandb.init(id=wandb_id, resume="must", project="alrichards", name="test-run-3/job-2", group="test-run-3")
wandb.log({"mean_evaluate_loss_epoch": 10}, step=5)
wandb.finish()

Hi! Using id and resume definitely works for your use case.
The reason you are seeing a single datapoint is the line chart averages points over runs by default when they are grouped. If you want to turn this off you can do so in the UI in the upper right.

Hi @a-rich thank you for confirming this. I will close the ticket for now, but feel free to re-open it by posting here if you have any further questions and we will be happy to keep investigating!


chore: respect tmux's base-index config in devnet.sh

Motivation
I have a following line in my .tmux.conf
set -g base-index 1

which configures tmux to create windows with indexes starting from 1 instead of the default 0. I use it so that when I want to switch the windows using their indexes, the order of windows matches the order of num keys on keyboard.
This however doesn't play nice with the devnet.sh script, which assumes the indexing of windows from 0, thus failing on creating the second window window1 as the index 1 got taken by the window0.
It would be cool if this tmux setting was taken into account.
I've opened this PR on top of the #2975 as it is kinda required, however I didn't want to include this directly in the previous PR as I  am not sure if you'd like to support this config option.
Test Plan
manual check

Can you verify whether this PR is compatible with this merged in PR/change?
https://github.com/AleoHQ/snarkOS/pull/2972
thanks!

This is also compatible now, I applied index offsetting to client windows too


The IFlashMintReceiver callback should take a sender parameter.

The receiver will always be a contract that needs to know the user that originated the transaction, and only flash.sol can safely say who that is. The callback interface should include a sender parameter, filled by flash.mint with msg.sender.
IFlashMintReceiver(_receiver).onFlashMint(msg.sender, _amount, fee, _data);
Leaving to the sender to fill data with the account to manipulate on the callback would be unsafe.
There are other mechanisms to safely pass the target account, if sender would not be an EOA. Namely you can either store the address of the target account in a state variable in the receiver, or use off-chain signatures embedded in data. Both methods are more complex and expensive than just having flash.sol state who is the sender.

This is an interesting point. Can you re-open this in the dss-flash repo: https://github.com/hexonaut/dss-flash/issues ?
Thanks.

@albertocuestacanada Closing this issue - please see Sam's above comment to continue the conversation.


Initial support for breadcrumbs

Need to:

Write tests for Monolog
Ensure Monolog support isn't going to affect autoloader and older versions of PHP

Alright I think this is generally ok. Will want to squash merge since some of it was broken. I'd def prefer if we could easily do a global handler in Monolog, but the ecosystem is sane enough that it wont be that bad to get people to configure it.


[Event Request] table 1003 "Job Planning Line" OnBeforeFindBin

Please add event publisher OnBeforeFindBin in "table 1003 "Job Planning Line".
    local procedure FindBin() NewBinCode: Code[20]
    var
        WMSManagement: Codeunit "WMS Management";
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeFindBin(Rec, NewBinCode, IsHandled);
        if IsHandled then
            exit(NewBinCode)
        if (Rec."No." <> '') and (Rec."Location Code" <> '') then begin
            GetLocation(Rec."Location Code");
            if Location."To-Job Bin Code" <> '' then
                NewBinCode := Location."To-Job Bin Code"
            else
                if Location."Bin Mandatory" and not Location."Directed Put-away and Pick" then
                    WMSManagement.GetDefaultBin(Rec."No.", Rec."Variant Code", Rec."Location Code", NewBinCode);
        end;
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeFindBin(var JobPlanningLine: Record "Job Planning Line"; var NewBinCode: Code[20]; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 11267.


assess the variation in input variables across sites and possibly use this information to decide which input variables get added to the LSTM v2 (more here).

Assess the variation in input variables across sites and use this information combined with mechanistic hypotheses/understanding to decide which input variables get added to the LSTM (more here).


GPIO: embedded-hal 1.0 compatibility

Could you share a minimum example that's fixed by 64c3af9? Or, could you help me understand why we need this fix?
64c3af9 adds critical sections inside routines that (indirectly) require a &mut Port<...> exclusive reference. By requiring the exclusive reference, we've made it the user's responsibility to avoid this race condition. If they're sharing their Port object across execution contexts, they'll need to wrap it in something like critical_section::Mutex.

You might be right, this might have been an oversight by my part. I might have carried over some thoughts from LPSPI, where it will be necessary that an interrupt disables itself upon getting fired, making this critical section necessary there.
I will revert that change.

@mciantyre Ready for more reviews. Any leftover criticism/comments?
Otherwise I'd like to merge this as a base for the other eh1 implementations. Especially as it's only a non-breaking minor change.

@mciantyre Ping :)

@mciantyre Ready for review again. Now update to eh-1.0 release.

I rebased onto commits that fix clippy warnings in other code, simplified the history, and added a CHANGELOG entry.
I dropped the MSRV recommendation. It's not clear why we should target 1.60, or why it's necessary to adopt an MSRV for this PR. And without a CI job, I'm afraid it would be difficult to maintain the MSRV ourselves. If you have a need for an MSRV, we can discuss in a separate issue.

A agree with all of your points.


Connection proxy

Can we add some tests to ensure the properties are actually available in the API?

That is what I'm doing now.

Any luck with the tests so that we can merge this PR?

@mattheworiordan This PR only has the ARTConnection proxy but I started the spec implementation on #44.


Allies tab not working.

Describe the bug
The allies tab is about 25% of the normal width.
To Reproduce
Steps to reproduce the behavior:

Look at your allies tab.

Expected Behavior
Just the standard allies tab, full width.
Screenshots

Contexts
Please complete the following information:

OS: Windows 10
Browser: Firefox
Browser Version: 74

Also tested on:

OS: Windows 10
Browser: Chrome
Browser Version: 80

I can't reproduce. What's shown in your next challenge section? There's a known issue where <style> in kata description affects how dashboard is rendered.

Also, this is on the dashboard right? Not Allies "tab" in your profile.

Yes, on the dashboard.

I pressed the Skip button and it worked, so it must've been that.

Thanks. Closing as duplicate of #1664.


Update the threat paper with additional threats/commentaries

Some sources:

Dependency Confusion
https://daniel.haxx.se/blog/2021/03/30/howto-backdoor-curl/

[ ] Protestware
[ ] What new threats have been sparked over the past two years?
[ ] What projects (e.g. SLSA, sigstore, S2C2F, A-O, Package Analysis, etc.) have come about that address some of the risks identified?
[ ] Do we feel we've struck the right balance between pointing out risks but not being overly alarmist?

hey, Michael sorry - is there a link to the original paper?
Without having reviewed it, I'd be happy to contribute - I think we want to take a look at however it's organized around "people. process, technology" too. There seems to be more noise as of late about OS licensing problems, as an example.

MFA Fatigue Attack

The Uber Hack’s Devastation Is Just Starting to Reveal Itself (Wired)
Suspected Russian Activity Targeting Government and Business Entities Around the Globe (Mandiant)

Credential stuffing attacks on package index platforms

PyPI Dictionary Attack (PyPI)

Do we feel we've struck the right balance between pointing out risks but not being overly alarmist?

According to Sonatype and ENISA data, we have - probably - identified correct risks in the open-source supply chain, but without spreading awareness to mitigate the impact.

"And according to Mandiant
supply chain compromises were the second most prevalent initial infection vector identified in 2021. Furthermore,
they also account for 17% of the intrusions in 2021 compared to less than 1% in 2020." (Enisa)

Even if OpenSSF is trying to build and spread a set of tools, standards, specifications, and best practices, their adoption is not enough widespread, yet. My2c :)

@jstclair2019 In MD format: Threats, Risks, and Mitigations in the Open Source Ecosystem.md

Perfect @luigigubello Thank you! I'm digging into the PDF ATM.

Think about including AI/ML -specific threats as they relate to the rest of the topics.

So:

[x] Protestware
Missing recommendations for handling this scenario
[x] Dependency confusion
Missing recommendations for handling this scenario
[x] Account hijacking
Added point related to risks for expired domains
I can add a short point also for credentials stuffing attacks (I would like to discuss during the WG meeting)
[x] MFA Fatigue
Probably not important in open-source ecosystem but I have added a point in Account Hijacking session

Next step:

Write recommendations, if we have, and be sure that recommendations are good

In the first version, we wrote recommendations especially for third-party services (e.g. package managers), should we write more recommendations also for developers?

Write a section/chapter for tools (2021 -2023) to improve OSS supply chain security (SLSA, sigstore, S2C2F, A-O, Package Analysis, etc.)

Does the list of threats include all of the threats mentioned in the SBOMit presentation especially the diagram on page 5?
The VCS to source code artifact to build system path is particularly opaque and narrow.  Did the bits from the VCS content truly arrive at the build system unchanged?  Does the signature of the contents in the VCS truly correspond to the signature of the contents in the VCS or adjusted to match the potentially modified contents that will be downloaded?

@edelsohn sorry for my slow response 🙌 I try to reply your questions :) the document "Threats, Risks, and Mitigations in the Open Source Ecosystem" tries to cover as many scenarios as possible in all the steps required to deliver an open-source project. The steps (see this chart) covered by the doc seem to be more or less the same of slide #4 in SBOMit presentation.
Even if this document (v1.1) is not focused only on one specific solution (e.g. SBOM), I think various threats and risks - related to slide #5 of the SBOMit presentation - are described in the section "Package Consumption Phase", and in the updated version (v1.2), we have introduced new threat models (e.g. dependency confusion attack). We have also introduced a new section "Healthy and Integrity Checks" where we suggest some open-source tools, specifications, and standards to streghten the integrity checks. Where a specific check can happen depends on the project's infrastructure and procedures, so in the document - in some particular security topics (e.g. DevSecOps) - we link to external sources. If you think we should focus more on some specific threat models and go deeper, feel free to suggest some examples and I will try to write a new section or integrate an existing one :)


Export TileHandleStyle

Fixed an issue where users can not to set TileHandleStyle due to tile_handle_style.dart is not exported.

I've added code to styles_export.dart to export tile_handle_style.dart.

Thank you for the fix.


[Feature] Extend presets with mixer device and volume control

Allow presets to also specify:
SQUEEZELITE_MIXER_DEVICE
SQUEEZELITE_VOLUME_CONTROL

Available in release 2022-11-24.1


KBV-401 Import API keys from core-infra stack

Proposed changes
What changed
Use API keys from core-infrastructure stack.

Why did it change
Use keys that are less ephemeral.

Issue tracking

KBV-401

Closing as very stale


Make --version show the same value inside package.json

The CLI flag "--version" wasn't showing the right version of the package.
It had a hardcoded version number that needed to be changed along with the one inside package.json and some time ago people stopped updating it.
This implementation ensures the "--version" flag always report the same value inside package.json, without any need for manually updating.
This supersedes #367.
Fixes #350.

Merged and published in version 3.0.0, thanks again!


fix Fix creating hdfs connection coredump within bthread

https://brpc.apache.org/docs/server/basics/
According to the brpc doc, JNI code checks stack layout and cannot be run in bthreads so create a pthread for creating hdfs connection if necessary.


chore(service): Improve error message

Related to https://github.com/platformatic/platformatic/issues/523, this PR changes the error message from Missing config file to:
Missing config file!
Be sure to have a config file with one of the following names: platformatic.service.json
platformatic.service.json5
platformatic.service.yaml
platformatic.service.yml
platformatic.service.toml
platformatic.service.tml
In alternative run "npm create platformatic@latest" to generate a basic plt service config.
Error: TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined

Could you rebase?

Could you rebase?

Done ✅
Tests are failing but for external reasons:

uncovered tested lines 53-54,101-108 on service/index.js
pnpm: EBUSY: resource busy or locked copyfile 'D:\.pnpm-store\v3\files\...' -> 'D:\a\platformatic\platformatic\node_modules\.pnpm\@swagger-api+apidom-parser-adapter-openapi-yaml-3-1<EMAIL_ADDRESS>Error: Process completed with exit code 1


Fix some typos

The intention of this PR is to fix some typos in order to improve reading experience and quality as well :)
Notice: correction on deprecated function testCallsCallbackOnce() may break some calling convention.

Codecov Report

Merging #477 (dcd626b) into master (96146a2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #477   +/-   ##
=======================================
  Coverage   98.87%   98.87%
=======================================
  Files           6        6
  Lines         177      177
=======================================
  Hits          175      175
  Misses          2        2

Impacted Files
Coverage Δ

ReSwift/CoreTypes/Store.swift
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


When accessing an element from a slice via reflection (ie. via the Index method of reflect.Value), the returned reflect.Value's type is the slice's Elem type, not the actual type of the value.
ie.
var x = []any{string("")}

reflect.ValueOf(x).Index(0).Type() // returns "interface {}" instead of "string"

This seems strange behavior, especially since
reflect.ValueOf(x[0]).Type() // returns "string"

See https://go.dev/play/p/YNfd_PClNWe
What did you see happen?
var x = []any{string("")}

reflect.ValueOf(x).Index(0).Type() // returns "interface {}" instead of "string"

What did you expect to see?
I'd expect the Index method to return the actual underlying value of the slice, so its Type method should point to the actual value type.

This is working as intended. For the second example, passing interface types to ValueOf results in passing the underlying type as interfaces don't nest.
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Mention that you can use a method reference in your fabric.mod.json to initialize mixin extras.

Mention that you can use a method reference in your fabric.mod.json to initialize mixin extras.
This may or may not need some rewording

Closing since you don't need to initialize it yourself anymore anyway.


Camera not opening in both of android and iOS

In Ionic v5, i wrote code like below;
import { QRScanner, QRScannerStatus } from '@ionic-native/qr-scanner/ngx';
...
this.qrScanner.prepare()
.then((status: QRScannerStatus) => {
if (status.authorized) {
this.qrScanner.show();
    this.scanSubscription = this.qrScanner.scan().subscribe((text:string) => {
      console.log(text);
    });
  } else if (status.denied) {
    console.log("Camera permission permanently denied");
  } else {
    console.log("Camera permision temporarily denied");
  }
})
.catch((e: any) => {
  console.error('Error', e);
});

...
But it shows blank screen and i was thought its because of background issue and did like this;

in scan
(window.document.querySelector('ion-app') as HTMLElement).classList.add('cameraView');
in stop
(window.document.querySelector('ion-app') as HTMLElement).classList.remove('cameraView');
in app.component.css
ion-app.cameraView, ion-app.cameraView ion-content, ion-app.cameraView .nav-decor {
background: transparent none !important;
}

But still camera not opening and show blank white screen. Actually it`s so urgent and pls help me asap.
Thanks.

same issue .....pls helppp

Up!

Same here. Using cordova-android ^8.1.0, installing by cordova plugin add cordova-plugin-qrscanner and running:
document.addEventListener('deviceready', () => {
  window.QRScanner.scan((err, contents) => {
    if (err) console.error(err)
    else console.log(contents)
  })
})

It asks for the camera permission, but shows no video. All elements on the page has background: transparent applied.
I tried to put a QRCode in front of the camera anyway, just to see if it wold work. It doesn't.

Hi all, I bet you the scanner/camera view IS there - its just hidden behind your app.  Other posts here comment that you must apply background:none transparent to the BODY tag of your main page and also to <ion-content>.
IF that doesn't work for you, see my recent thread on how I had to get it work with my old Ionic v1 app.  You will need to step through your DOM each parent element at a time applying background:none transparent - until your app reveals the scanner view beneath.   Then figure out which of those parents are actually truly need transparency - you may have to code something create as I had to.
NOTE: until you see the camera beneath, I recommend not actually starting a scan or your test devices (phones) will heat up pretty quick while you are stepping through your DOM.  I recommend just doing QRScanner.show() just to activate the camera without activating the scan process.

@rolinger background:none transparent doesn't work actually.
Below code works for me
$("html").hide();
QRScanner.scan(displayContents);

function displayContents(err,text){
  if(err){
	alert("Something went wrong. Please try after some time.");
  }
  else{
	$("html").show();
	QRScanner.hide();
	QRScanner.cancelScan();
	alert(text);
  }
}

QRScanner.show();

Hope it helps, Happy coding!

@cmohanraj10 - its what worked for me.  I use a scanner in two different places in my app and my above technique was the only thing that I could find that would reveal the camera running beneath my app, it also allowed me to put a wrapper on the full camera screen and put in a target scan box giving it all a more professional cleaner look while also still exposing necessary buttons to back out or switch screens.

Does anyone have a demo of this? I have tried everything but still not working. It ask for permission and console shows the status payload:
authorized: true
canChangeCamera: true
canEnableLight: true
canOpenSettings: true
currentCamera: 0
denied: false
lightEnabled: false
prepared: true
previewing: true
restricted: false
scanning: true
showing: true
and on the .show(), console shows:
data:  true

mine was not visible because of DARK MODE in phone setting is enabled


Detailed instruction on submitting model predictions of val/test set to online server for 3D semantic segmentation challenge

Does anyone else know the details instruction for submission ?  And many thanks who could provide the runnable code here!!!

Same question. It seems that the tutorial of 3D semantic segmentation has many flaws and syntax errors.

@alexgorban

Hi, in case you missed it, there has been several improvements to the tutorial_3d_semseg.ipynb in the last few weeks.


RFC: instantiateMVars is unnecessary to use in trace messages.

Proposal
Trace messages automatically come with the current MetavarContext, so there is no need to also call instantiateMVars. For example:
https://github.com/leanprover/lean4/blob/f05a82799a01569edeb5e2594cd7d56282320f9e/src/Lean/Meta/SynthInstance.lean#L374
https://github.com/leanprover/lean4/blob/f05a82799a01569edeb5e2594cd7d56282320f9e/src/Lean/Meta/SynthInstance.lean#L415-L417
In the second example, abstractMVars also doesn't need instantiateMVars, because it already runs it on its input, so here instantiating metavariables is also redundant.
This also appears twice in this file:
https://github.com/leanprover/lean4/blob/f05a82799a01569edeb5e2594cd7d56282320f9e/src/Lean/Elab/Match.lean#L1152

Closing as per reasoning in the PR

Small addendum, in case someone stumbles over this: The Lean.Expr data type has a computed field (Lean.Data) that stores whether the expression has MVars, so an extra instantiateMVar is very cheap and usually not worth the bother to fret about.


Creating a Whack a mole game for website

@harshgupta20  @ridsuteri @991rajat
I am GSSOC21 participant.
I want to add whack a mole game using js
please assign it to me.

Sorry @ayush461 that game already exists in the repo, try improving that game Or try to add any other project
:)

okk


Show total votes and individual votes

Show the total number of votes in a day and the total number of votes for each option more clearly (rather than hovering over the graph).

(issue filed so I can work on it later - currently don't have time)

(issue filed so I can work on it later - currently don't have time)


Loader tool: Filter by statuses

Changelog Description
Added combobox for filtering by statuses in loader tool.
Additional info
Added multiselection combobox for statuses that can be used to filter versions in loader tool. Selection in combobox does show stauts icons in the combobox field. Combobox view also has 3 action items: Select all, Deselect all and Swap which swaps selection. These might be removed if it's too much.
We might add similar combobox for product types, but there would have to be added better implementation of paint for the combobox field as icons are not enough.
Screenshots

Testing notes:

Use loader from tray or use loader tool in host. It would be maybe better to test multiple hosts with Qt Nuke, Maya, Houdini etc. so we validate it looks and works there.

[x] Maya
[x] Blender
[ ] Nuke
[ ] Houdini

Select folder with products.
Change statuses filter > versions or whole products should start to disappear.

EDITED:
I changed behavior of the combobox, when nothing is selected it behaves as everything is selected. Everything is deselected by default, so user is actually checking statuses he wants to see instead of deselecting them.
Maybe with this new behavior we don't need the helper actions to de/select all and swap selection.

owever I have found issue that status diminishes as you re-open or refresh Loader as seen on my scrn capture below...

Good catch, should be fixed.

It definetely is fixed now but here is another issue when adding some more statuses into filter...seems like new issue or undiscovered one too

I couldn't replicate the issue, please DM me when you're available.

This is huge guys, great stuff!


GitLab CI stable/testing branch pattern detection with master/develop branches

Description of Problem, Request, or Question
I'm trying to use CPT from a Giltab CI pipeline where Conan packages are uploaded to an Artifactory instance.
There is one master branch and multiple develop branches.
Each tag shall upload my package either to stable (tag on master branch) or testing (tag on other branches).
My assumption was to use the stable_branch_pattern, but this does not belong to $CI_BUILD_REF_NAME.
If I tag on master branch CPT detects only the tag:
 >> Branch detected
   >> 1.10.0

Obviously this tag does not match the pattern.
Steps to reproduce (Include if Applicable)
My expected GitLab stage:
.deploy:
  only:
    - tags
  variables:
    CONAN_USERNAME: "user"
    CONAN_STABLE_BRANCH_PATTERN: "master$ main$ release.* stable.*" # all tags on master branch will be deployed to "stable" channel (all others: "testing")
  script:
    - python3 build.py

My CPT output if i tag on master
[local_vars]
+---------------------------+------------------------------------------------------------------+
| Configuration             | value                                                            |
|---------------------------+------------------------------------------------------------------|
| username                  | user|
| upload_only_when_stable   | False                                                            |
| upload_only_when_tag      | False                                                            |
| stable_branch_pattern     | master$ main$ release.* stable.*                                 |
| stable_channel            | stable                                                           |
| channel                   | testing                                                          |

Is there an easy way to use the gitlab only: -tags feature with stable/testing detection?
My current solution always uploads to testing, due to misleading branch detection.
Environment Details

Conan Package Tools Version: 0.23.0
Conan version: conan 1.43.0
Python version: python 3.7.0

I think you can close this issue, my understanding of git + tags was a little bit confusing.
A tag is not related to a branch but we create our packages for each tag commit.
Now I´m solving my problem with a tag convention.
  rules:
    - if: '$CI_COMMIT_TAG && $CI_COMMIT_TAG =~ /^\d+\.\d+\.\d+$/'
      variables:
        CONAN_CHANNEL: "stable"
    - if: '$CI_COMMIT_TAG && $CI_COMMIT_TAG =~ /^(\d+\.\d+\.\d*-rc.+$)/'
      variables:
        CONAN_CHANNEL: "testing"

Maybe this helps other people which are a bit slow on the uptake :)


Neogit slowing down my neovim

Description
a couple of days ago I've noticed my nvim workflow was a little bit slow, doing simple actions. After an investigation (updated my neovim to the latest to see if it was a problem too), I've started to remove my plugins to see if there was any difference. Found it was neogit, let only neogit and packer to see if the problem continued or if it was a compatibility issue with other plugins.
Neovim version
NVIM v0.7.0-dev+1341-g79dcd045d
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Operating system and version
ubuntu 18.04
Steps to reproduce
any action
Expected behavior
not to gag
Actual behavior
stutter a lot even to open neogit, or navigate through the file with jklh. Even without setting up, just by letting installed stutters my editor
Minimal config
return require("packer").startup({
	function(use) -- plug manager
		use("wbthomason/packer.nvim")
		use({ "TimUntersberger/neogit", requires = "nvim-lua/plenary.nvim" })
        end
})

I have the same thing. Disabling auto_refresh fixed it for me, it seems like it hapeo big repositories. Perhaps the logic that runs is sync and that's why we feel the stutters.

I dont know what it fixed, but I've tried now after months without using it and it fixed. Dont think its the repo size, because by just installing it was stuttering my vim, opening vim on any directory of any size.... So far, its ok.
For the record this is my current :version
NVIM v0.8.0-dev-775-g27ce21ac8                                                                                Build type: RelWithDebInfo                                                                                LuaJIT 2.1.0-beta3                                                                                Compilation: /usr/bin/gcc-11 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -DNVIM_TS_HAS_SET_ALLOCATOR -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wdouble-promotion -Wmissing-noreturn -Wmissin g-format-attribute -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/home/runner/work/neovim/neovim /build/cmake.config -I/home/runner/work/neovim/neovim/src -I/home/runner/work/neovim/neovim/.deps/usr/include -I/usr/include -I/home/runner/work/neovim/neovim/build/src/nvim/auto -I/home/runner/work/neovim/neovim/build/include                                                           Compiled by runner@fv-az242-505


Current version doesn't allow to adapt symlink labels

For that -f -l (of -f l for RHEL>7) is needed.

Hey @daften
Thanks for letting me know. Would you like to suggest a PR for that?

Jup, I will try to create one over the course of the next week. :)
Thanks for the quick reply!

Added a PR: #52
Still working on it, but feel free to comment

The PR is finished as far as I'm concerned, feedback welcome.
I also noticed that one of the tests through kitchen fails, since the relabel's aren't always done immediately. I've re-enabled the faulty test in my last commit, couldn't figure out why it was failing exactly, or how to fix it, besides doing an extra relabel just before, or setting the relabel notification on the modify action to :immediately.

Thanks for the contribution. Let's continue further discussion in the PR.
CC @equick if you wanna take a peek

fixed by 394cd97c08bb27067db9085a63ccd950121c4184


Deprecated hint is not reported when the deprecated formal parameter is in Null Safe library, and the use in in legacy

I had to update DataDrivenFixProcessorTest.setPackageContent with language 2.9 to make test_cupertino_CupertinoTextThemeData_copyWith_deprecated in FlutterUseCaseTest to pass.
The change will probably land as https://dart-review.googlesource.com/c/sdk/+/169141
@bwilkerson this might be an issue for the actual Flutter migration.

Yes, it probably will be.
That begs the question of why the hint isn't being reported; that feels like a bug, and I'd rather see the bug fixed than to have the test changed.

Yes, definitely the bug should be fixed, and the hint should be reported.
I just wanted for you to know that there is a problem, and put some record for whoever will fix it.

https://dart-review.googlesource.com/c/sdk/+/169256


Fix client-rest-high-level tests for ppc64le.

Description
Fixes tests for ppc64le mentioned in #12313
Related Issues
Resolves #12313

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Closing this PR in favour of #16089


Investigate the possibility to rescan with a pruned node

I need to investigate. I wonder if we could try to be smart.

yes please. But i am not aware of anybody doing this for now :/

I don't think it's actually doable.  In short, you'd have to know which blocks to request from a peer.
An alternative could be that we relax the requirement we have to know all transactions associated with our coins. This would allow for partial recovery (basically just scan the current utxo set and locate your current coins). But this would:

Be a very invasive change
Increase the complexity of the wallet

Therefore i don't think the costs are outweighed by the benefits here. Leaving this open if anybody else has an idea on how to do a full rescan in prune mode (missing historically spent coins and their associated txs would be fine, missing transactions associated with current coins isn't).

can you had a bit of context there, i missing smth, i guessed it's already what we done w/ a liana managed bitcoind in the context of a restore flow?
I guess in this context bitcoind rescan the (not pruned) blocks + UTXO set?
so your point is about rescanning in order to even get an history of the spent coins before the 'prunning point'?

@pythcoiner i can't parse your questions. Upon rescan of a wallet bitcoind goes through the block chain and matches against transactions in which our wallet is involved. You can't do that if you don't have the block chain (i know you know that, but i'm not getting what the question is about).


error: attribute 'inputs' missing.

Platform

surface pro 8

[x] Linux:

Additional information
I try to install hyprland plugins in HM, the instruction is:
wayland.windowManager.hyprland.plugins = [
inputs.hyprland-plugins.packages.${pkgs.system}.hyprbars
];
I get the
error : attribute 'inputs' missing.
But I have passed the inputs to HM:
modules = [
./hosts/surface
      nixos-hardware.nixosModules.microsoft-surface-pro-intel

      home-manager.nixosModules.home-manager {

        home-manager.useGlobalPkgs = true;
        home-manager.useUserPackages = true;
        home-manager.extraSpecialArgs = inputs;

        home-manager.users.ventus = import ./home;
      }
    ];

it doesn't work.
and then I tried to add these inputs manualy:
{pkgs, inputs, lib, hyprland-plugins, hyprgrass, ...}:
then it is the new error:
Output
Output

-- Setting precompiled headers
-- Setting link libraries
Could not open input file: No such file or directory
Could not open input file: No such file or directory
Could not open input file: No such file or directory
Could not open input file: No such file or directory
Couldn't load proto
-- Checking for module 'tomlplusplus'
--   Found tomlplusplus, version 3.4.0
-- Configuring incomplete, errors occurred!

Priorities
Add :+1: to issues you find important.

Hey, I had this same issue as you where I tried to pass inputs to extraSpecialArgs. This comment helped though: https://github.com/nix-community/home-manager/issues/1698#issuecomment-832545025
Basically, instead of passing inputs broadly, just inherit the specific inputs you would like:
{
	inputs = {
		apple-fonts = { #... };
	};
	outputs = {self, nixpkgs, home-manager, apple-fonts, ...} @ inputs: nixosConfigurations = {
		user = nixpkgs.lib.nixosSystem {
			modules = [
				home-manager.nixosModules.home-manager {
					home-manager.extraSpecialArgs = {inherit apple-fonts};
				}
			];
		};
	};
}

Sorry, this seems to be a skill issue, not an issue with nixpkgs or home-manager. In short, specialArgs in inputs.nixpkgs.lib.nixosSystem defines which arguments you want to pass to each separated modules. You may need something like
{
	inputs = {
		nixpkgs.url = "...";
		home-manager.url = "...";
	};

	outputs = inputs: {
		nixosConfigurations = {
			nixos-machine = inputs.nixpkgs.lib.nixosSystem {
				system = "x86_64-linux";
				# ...
				specialArgs = {
					inherit inputs;
				};
			};
		};
	};
}

So you can write
{ inputs, ... }:

{
	# ...
}

in modules.
It is worth noting that here or the nix repo issues are not very suitable places to ask questions. You can go to https://discourse.nixos.org or https://matrix.to/#/#users:nixos.org to get more help.


License?

Hello. I am wondering if there is a license for this project? Is one able to use the code as a base for an app that will be released to the app store that will not make money?
Thanks.

sure, MIT - added in e673f867f24047201f6


Sorting address book by name

add an icon next to the Name column: A and Z next to an arrow
prompt "Sorting address book by name will overwrite custom order. Are you sure?"
don't show prompt if already ordered alphabetically or reverse alphabetically
flip order if already alphabetical or reverse alphabetical

Is anyone working on this?


Bridge Hub Integration: View bridged assets from in Asset Hub in Wallet

Is your feature request related to a problem? Please describe.
Asset Hub now supports foreign assets bridged through Bridge Hub. A user should be able to see any bridged assets that they own in their wallet.
Describe the solution you'd like
Simply display any bridged assets that the user owns in the wallet UI.
Describe alternatives you've considered
Users would potentially have to query chain state manually.
Additional context
Asset Hub now supports foreign assets bridged through Bridge Hub. These assets are stored in a different instance of the assets pallet called ForeignAssets from version 1.7 release of Asset Hub.
There are two bridges currently deployed in Rococo

Snowbridge (bridges to Sepolia Ethereum testnet)
Westend Bridge (bridges to Westend substrate based testnet)

For Snowbridge the token that is Bridged is WETH (Wrapped Ether). Please see docs for more information on how to interact with the bridge and query asset balances.
We have plans for launch in Kusama and Polkadot (bridging to Ethereum mainnet) by the end of March.

Snowbridge Team

Thanks @alistair-singh for creating this issue. We'll work on this asap

Account has WETH: 5Eg2fnsjAAr8RGZfa8Sy5mYFPabA9ZLNGYECCKXPD6xnK6D2

Extrinsic send WETH from Asset hub Rococo -> Sepolia (Ethereum testnet): https://assethub-rococo.subscan.io/extrinsic/3879712-2

Resolved in https://github.com/Koniverse/SubWallet-Extension/issues/2658


Azure Storage Connection String Parsing doesn't parse format exported by Storage Explorer

Which service(blob, file, queue, table) does this issue concern?
Blob
Which version of the SDK was used?
12.16.0
Which platform are you using? (ex: .NET Core 2.1)
.Net 7
What problem was encountered?
Azure.Storage.Blobs does not correctly handle creating a BlobServiceClient with a container-specific connection string obtained from Azure Storage explorer
How can we reproduce the problem in the simplest way?

Retrieve a connection string from Azure Storage Explorer, e.g. SharedAccessSignature={sas};BlobEndpoint=https://{account}.blob.core.windows.net/mycontainer;
Create a BlobServiceClient with that value: var client = new BlobServiceClient(conStr);
Retrieve a reference to the container: var cont = client.GetBlobContainer("mycontainer");

That will end up appending "mycontainer" to the URL even though it's already included.
For this particular case we want one code path that can handle all kinds of connection strings - those using an account key, those with a SAS for the whole account, and those with SAS for a specific container in that account. But this path fails if the connection string already includes the container, which is what Storage Explorer suggests.
Have you found a mitigation/solution?
You can remove the /mycontainer from the connection string - and any methods that operate at the account level would need to be omitted (and I haven't found the perfect solution for detecting that yet)

Adding Service team to look into this.

Sounds like a bug. I'm going to try to reproduce this and if it does reproduce I will fix it by correctly passing the SAS over to the child clients.


[Feature]: Custom at_edge behaviour (user-defined function)

Similar Issues

[X] Before filing, I have searched for similar issues.

Description
I just had a (weird?) idea to open nvim-tree automatically when I try to move left from the leftmost window, similar for the other edges. It might be cool to allow the user to pass a function as the at_edge parameter and call it with some information like which edge we hit. The function should return wrap, split or any other future behaviour, or 'none'/nil for which smart-splits wouldn't do anything (but from inside at_edge I would call NvimTreeOpen and focus it)

That sounds perfectly usable too


fix: clear unreached commits when commits are pushed successfully

fix #603
Overview
Clear unreached commits when commits are pushed successfully in middlewareHandler.commitAndPush and middlewareHandler.pushAndCommit
In middlewareHandler.push, not clear unreached commits because commits are pushed by the number specified in the action payload, and all unreached commits may not be pushed.
see e5d67ff for fix
see 46499be for test

@1natsu172
Thank you for your fix of test


Shouldn't automatically generated indexes for references in derived entities be filtered?

Let's assume we have the following simple case:
*) Entity AA derives from Entity A. We have the whole type hierarchy in one table.
*) Entity AA has a property BID being a foreign key for the ID column of Entity B.
Entity Framework will now make a non-clustered index for AA.BID which is perfectly reasonable, but shouldn't this index be filtered on the discriminator value for Entity AA so only rows in the table which denote instances of AA will make it to the index. As it is today, if another derived class of A, i.e. AB, will also have instances, null values (for BID, which is not a property of AB, but of AA) go to the index for each AB-row as the index is not filtered. So there will be plenty of useless performance-reducing null values in these automatically generated indexes.
Isn't that correct?

EF Team Triage: This issue is not something that our team is planning to address in the EF6.x code base.  This does not mean that we would not consider a community contribution to address this issue.
Moving forwards, our team will be fixing bugs, implementing small improvements, and accepting community contributions to the EF6.x code base. Larger feature work and innovation will happen in the EF Core code base (https://github.com/aspnet/EntityFramework).
Closing an issue in the EF6.x project does not exclude us addressing it in EF Core. In fact, a number of popular feature requests for EF have already been implemented in EF Core (alternate keys, batching in SaveChanges, etc.).
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Release

There are issues that are already solved in main, but there is no new release of docker containers.
Important changes are for example #17 and #8.
Will there be a release in the near future?

AFAIK we do not really have a communicated release plan for the KUKSA components, but now when CAN Provider has been moved to a separate repo it should be quite easy to create a new release just for KUKSA CAN provider. I will check if i can get time to make a new release.
FYI: @mikehaller @SebastianSchildt

@fpetry - I created a new 0.4.3 release and docker containers. Basic testing performed. Feel free to try it out and report any problems.

No comments so closing it, please reopen if any problems are observed


python release v0.0.2

Codecov Report

Merging #305 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #305   +/-   ##
=========================================
  Coverage     95.55%   95.55%
  Complexity       21       21
=========================================
  Files             7        7
  Lines           270      270
  Branches          8        8
=========================================
  Hits            258      258
  Misses           10       10
  Partials          2        2

Flag
Coverage Δ
Complexity Δ

#go
96.93% <ø> (ø)
0.00 <ø> (ø)

#java
96.42% <ø> (ø)
21.00 <ø> (ø)

#node_js
95.91% <ø> (ø)
0.00 <ø> (ø)

#python
92.53% <100.00%> (ø)
0.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ
Complexity Δ

...al/python/alibabacloud_ccp_credentials/__init__.py
100.00% <100.00%> (ø)
0.00 <0.00> (ø)

Continue to review full report at Codecov.

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


[stable/elastalert] fix scope of realertIntervalMins

What this PR does / why we need it:
realertIntervalMins is not an elastic search properly, it is documented and in the default values.yaml correctly, but the template looks for it as a property/sub element of 'elasticsearch'
Special notes for your reviewer:
@jertel

/ok-to-test

/lgtm

/lgtm


Unsubscribe links in user emails

For transactional emails needed to make things works, we don't need this. But for "publicity" emails we should probably have it.

I think this can be done via mailgun if we have a "mailing list". Not sure if it's easier to do that, or just via our own system. I'd like to be able to track our unsubscribe rate quite closely.


[GUI] Change 'card' background color when and adress have been copied

Here i often generate few  addresses and forgot wich one i've copied, it should be nice to change the card color in order to notify user which have already been copied.

Yeah, or have a tick or something.


Remove Carthage-specific paths, remove Box and Result from copy phase

This change enables Pipes to be used as a subproject or workspace project.

Awesome! Thanks @bwhiteley :+1:


unexpected `end', expecting end-of-input

rails -v
> Rails 7.0.1
ruby -v
> ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x86_64-linux]

I do gem add syntax_tree, than rails s, and then I receive a ton of the following (and similar) in logs:
/home/staleo/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/syntax_tree-1.2.0/lib/syntax_tree.rb:14006: syntax error, unexpected end', expecting end-of-input
Any ideas how to solve this? Thanks!

Yeah, I'm using argument forwarding which was available in 2.7.3. Sorry I haven't tested on old ruby versions yet. Can you try on 2.7.3 and see if it works?

Hi Kevin, right, switched to 3.0, and it's back to business now.


doc fix for effective_sample_size

Summary:
@avehtari pointed out on our formus that the doc for the ESS code doesn't match its implementation:
http://discourse.mc-stan.org/t/n-eff-bda3-vs-stan/2608/2
Fix so that it matches.
P.S.  It'd also be nice to fix that Eigen Matrix of Eigen Matrix data structure.
Current Version:
v2.17.0

Is there somewhere support for the comment in the doc for the current ESS implementation? If not, then it would be better to change the code to match BDA3 as there is theory behind the Geyer's approach.

I thought our implementation matched the BDA definition.
I don't mind if matches or not or what the implementation is---@betanalpha and @andrewgelman would be the ones to ask (don't know if Andrew gets GitHub messages).

I asked @betanalpha, and he was in favor of BDA3 definition. @andrewgelman accepted definition in BDA3, when BDA3 was written.

I added a branch https://github.com/stan-dev/stan/tree/bugfix/issue-683-effective_sample_size
Now it's bit confusing that this issue is is in stan-dev/math, but the chains.hpp is in stan-dev/stan,
and I didn't notice this before pushing the branch. I guess I should make a new issue in stan-dev/stan?

What you have on the branch linked isn't a documentation fix, so this issue should be updated to reflect what's really going on.
If this requires fixes to math and to stan itself, it'll need two issues.  If it's only doc on stan, then you can just add that to the ongoing doc issue.

Both the code and doc need fixing. I'll add another issue for the code in stan-dev/stan.
I assigned myself to fix the doc, too.

Functionality changes and doc should go together in one issue and one pull request.

Ok. I close this issue as the correct issue should be in stan-dev/stan (not in math).


chore: check test running

Hi @ssa-wtag seems that spec/xapian_db/railtie_spec.rb is the culprit (line 2). railtie.rb requires Rails but Rails is not part of the Gemset. Maybe we should just dropspec/xapian_db/railtie_spec.rb?

Hi @ssa-wtag seems that spec/xapian_db/railtie_spec.rb is the culprit (line 2). railtie.rb requires Rails but Rails is not part of the Gemset. Maybe we should just dropspec/xapian_db/railtie_spec.rb?
Another option would be to move the method that you want to test to a new module and require that module in the railtie.

yes, i am moving to a new method


Write to parquet file?

Is it possible to write to parquet file using this library? (quickly checked the code, didn't see any write function).

No plans for writing parquet files at this time.
I could be convinced otherwise, but generally I feel that if you are creating parquet files, you are more likely to be in a backend environment so it makes sense to use existing parquet libraries in like python, C++ or Rust.
What I really want with this library is to make it easy to view parquet data in the browser, since there was no good library for decoding parquet files in javascript that was lightweight and could handle remote files efficiently.
You might like the work of @kylebarron on parquet-wasm. Hope you find what you need!

lightweight and could handle remote files efficiently.

Generally agree that "webassembly" and "lightweight" are not synonyms, but there's no technical blocker to handling remote files efficiently in parquet-wasm. In the latest release you're able to fetch individual row groups or columns from a Parquet file without downloading the entire file. And we could implement something like pyarrow's filters param, I just haven't taken the time to fully implement that yet.

No plans for writing parquet files at this time.
I could be convinced otherwise, but generally I feel that if you are creating parquet files, you are more likely to be in a backend environment so it makes sense to use existing parquet libraries in like python, C++ or Rust.
What I really want with this library is to make it easy to view parquet data in the browser, since there was no good library for decoding parquet files in javascript that was lightweight and could handle remote files efficiently.
You might like the work of @kylebarron on parquet-wasm. Hope you find what you need!

I'm creating an offline application (with local server and web application using Electron) that stores transactional data locally in the backend/server, and then uploads it to S3 to be analyzed with cloud-native tools such as Athena, QuickSight etc.
I'm looking for a lightweight library to read/write to Parquet file, and your library ticks all boxes except for the write function.
I've checked other libraries but most aren't maintained.

You can use WebAssembly in Electron, so parquet-wasm should work out of the box.

I know, all the data handling hairband in the server which will take care of persisting files and such. The client will be very "stupid" and just display stuff.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Kyle Barron @.>
Sent: Monday, April 29, 2024 4:39:59 PM
To: hyparam/hyparquet @.>
Cc: Erik Norman @.>; Author @.>
Subject: Re: [hyparam/hyparquet] Write to parquet file? (Issue #6)
You can use WebAssembly in Electron, so parquet-wasm should work out of the box.
—
Reply to this email directly, view it on GitHubhttps://github.com/hyparam/hyparquet/issues/6#issuecomment-2082927098, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJZL7L5KDYWUWO3MIFN2QALY7ZLT7AVCNFSM6AAAAABG44IV7KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSHEZDOMBZHA.
You are receiving this because you authored the thread.Message ID: @.***>

Generally agree that "webassembly" and "lightweight" are not synonyms, but there's no technical blocker to handling remote files efficiently in parquet-wasm.

parquet-wasm has 5+ megabytes of wasm file, hyparquet is sub-100k of javascript. Loading can be much faster especially for time to first render.
Because hyparquet is not a compiled wasm blob, there is no need for transferring data across the wasm boundary, and no cold-start time for loading the wasm vm. Also I've done some optimizations for the web like if you are fetching a bunch of columns in a rowgroup, it will fetch the data in just one http request instead of multiple round trips. I'm guessing that parquet-wasm, if you can implement ranged-gets, probably doesn't coalesce the requests to save round trip time?
Huge respect for your work Kyle, I love reading your blog about parquet stuff. Definitely not knocking parquet-wasm! Just pointing out the reasons I built hyparquet. :)

Just pointing out the reasons I built hyparquet. :)

That's very fair! I think it's valuable to have a pure-JavaScript implementation!
My own bias is that Parquet is an absolutely perfect place for WebAssembly, because Parquet is such a complex spec with such a long tail of complexities. It's not that I don't want a pure-JS implementation; rather my own conclusion was that implementing a stable pure-JS Parquet implementation that supports all encodings and compressions would be an absolutely massive engineering effort. Most previous JS Parquet implementations were eventually abandoned.
Whereas there are a ton of people building databases in Rust, so the Parquet implementation is stable, fast, and loads into a binary representation. Perhaps it's a use case where the benefits of WebAssembly outweigh the costs.
So take encouragement with a hint of skepticism 🙂. If you're able to implement a stable pure-JS Parquet reader, it'll be really impressive!

parquet-wasm has 5+ megabytes of wasm file, hyparquet is sub-100k of javascript. Loading can be much faster especially for time to first render.

1.2MB brotli-compressed 😉 , but yes. We might have alternate use cases; you might care more about time to first render whereas I'm more focused on handling large datasets where Parquet 1.2MB is very small compared to the data savings from Parquet.

I'm guessing that parquet-wasm, if you can implement ranged-gets, probably doesn't coalesce the requests to save round trip time?

It does. Multiple ranges are coalesced by default. The coalesce size is currently 1MB and not configurable though.

Also note that the people in loaders.gl are also building a pure-Typescript Parquet implementation, which I think was forked from parquets. It might be worth reaching out to them

Thanks for the additional information.
The application server - the only project managing data - doesn't know anything about any UI or client libraries, that's why I'd rather stick with a JS/TS library to read/write Parquet files.

it's particularly valuable when we're interested only in reading the metadata.

Your use case involves reading the metadata only... but not the data?

Oh if we're talking compressed size, then hyparquet is 24.1kb compressed :wink:

Your use case involves reading the metadata only... but not the data?

Yes, we just launched a Parquet metadata viewer: https://huggingface.co/datasets/HuggingFaceFW/fineweb/tree/main/data/CC-MAIN-2013-20?show_file_info=data%2FCC-MAIN-2013-20%2F000_00000.parquet
It's powered by hyparquet!