Create a edit component to edit the subcategory

Is your feature request related to a problem? Please describe.

We need to able to edit subcategory after created.

Describe the solution you'd like

Reuse AddSubcategory component as the edit subcategory

I'm working on this.


Add links and removed dead links

I have added new links for angular,clojure,r programming and ruby.
I have corrected the link for lua
I removed certain links of android which were dead or outdated

@vhf Can codeacademy or codeschool courses be included in this file or should it be in free-programming-interactive-tutorials.md

@MathewtheCoder I'm not sure, I'm not so familiar with these websites. If it's mainly "classic" educational content or if they are video lectures, it's more of a "course". If it's really interactive, for example if you have to actually type code to progress, then it's interactive. If it's a book, then we also have files for these. :)


How can I install Microweber on my localhost

I uses Xampp to run projects, so I wanna install mw on the localhost.
After cloning mw, I started xampp and run this url : "localhost/my_microweber/install.php" (my_microweber is my cloned project).
Browser displays "Warning: require(D:\xampp\htdocs\my_microweber\bootstrap/../vendor/autoload.php): failed to open stream: No such file or directory in D:\xampp\htdocs\my_microweber\bootstrap\autoload.php on line 17"
Thanks for your help ^^

you need to run composer install   or download the zip file from here  https://microweber.com/download

oh thank sir, I succeeded
But why I can't use a cloned mw project?

from github you need to use composer to install the dependent libraries
https://getcomposer.org/download/
you clone the repo and then you go in the repo folder and run command composer install

ohhh thank sir so much ^^
I understood. I'll try both of way
Thank sir <3


Invisible history records

This PR provides ability to replace current events change capturing system, currently based on writing resource keys for changed resources to user tables, by querying Resource table based on processed transactions (old events system is still not removed and works AS-IS).
To enable above we need to hold resource versions for the event retention period, so events system can query Resource table and data is not missed. This works out of the box with versioned data policy and absence of hard deletes.
To cover no-version policy and hard deletes PR code delays physical deletes. It marks historical resource version as "invisible", i.e. sets raw resource to 0xF. As far as data size, setting raw resource to single byte is equivalent to retaining only resource keys in the old events tables, but without actual writes (other name of this feature is "events with no writes"). Note that these updates are scoped only to no-version & events enabled case. This magic 0xF value allows to filter out these resources from search (hence "invisible" name).
To finally remove these invisible historical resource keys, a new watchdog "InvisibleHistoryCleanupWatchdog" was added.
Main logic to set raw resource to 0xF is in the couple of lines added to MergeResources and HardDeleteResource stored procedures.

/azp run

/azp run

/azp run

/azp run

@SergeyGaluzo Just a general question Sergey, is there any problem with this change and the constraint we have on the Resource table to detect empty resources?

No. Resource is technically not empty (one byte 0xF), and it passes constraint check.

In reply to:<PHONE_NUMBER>


Build Error: Does the ember-cli-babel version 7 and above provide no support for babel compact option?

I was using ember-cli-babel version 6.16.0 and in my ember-cli-build.js I specified a config
babel: {
 compact: false
}

At this version during build I faced no errors, Recently I upgraded my ember-cli-babel version to 7.1.2 and faced an error during build Invalid Option: compact is not a valid top-level option. I presume this is thrown from<EMAIL_ADDRESS>and above. Can't we specify compact option for babel for<EMAIL_ADDRESS>onwards(is it handled internally) or it is not supported? I am pretty sure babel provides support for this option.

Try passing it as a generator option:
babel: {
 generatorOpts: { compact: false }
}

@nicolo-ribaudo It fails again with [BABEL] unknown: Invalid Option: generatorOpts is not a valid top-level option. I presume the valid top level options are these If some invalid options are provided error is thrown by babel-preset-env.

@stefanpenner @rwjblue We have deleted invalid properties added to @babel/preset-env here Should we also include compact?

The reason the suggestion from @nicolo-ribaudo above doesn't work here is that ember-cli-babel passes the options from the babel config key directly in to @babel/preset-env and there isn't currently a way to disambiguate options destined for @babel/preset-env and top level configuration (e.g. parserOpts/generatorOpts).

@rwjblue So do you mean that options other than the one destined for @babel/preset-env will not work with ember-cli-babel

So do you mean that options other than the one destined for @babel/preset-env will not work with ember-cli-babel

Correct. That is the current implementation. I think we can fix this in general, but that is where things are at the moment.

What is the progress on this issue? Not being able to specify compact: false in development is a slight issue for me as my build time has increased. This is because some of the node module like lodash and sinon once installed have sizes greater than 500KB and will emit a warning saying styles have been deoptimized.

Have this same issue as well, build times are unacceptably longer because this option is no longer available for babel. Is there a workaround that can be implemented in the interim?

+1, build times are ~5mins due to firebase

Any updates on this we could greatly benefit from this as well

Pinging on this as we could also make use of it and we don't seem to be the only ones.

Near as I can tell the deoptimization warnings that people are likely seeing are happening after this plugin and are due to ember-auto-imports and its webpack configuration. I currently have a fork of this repo that allows specifying a babelConfig setting on the root level babel configuration. This setting is sent directly to the Babel transpiler, but it only winds up affecting ember source and your application/addon code.
You can try it at this branch: https://github.com/yads/ember-cli-babel/tree/fix-for-302 and see if it helps anybody, I can look at submitting a PR if this is something that would be worthwhile.
Effectively you would do:
var app = new EmberApp(defaults, {
  ...
  babel: {
    babelConfig: { generatorOpts: { compact: false } }
  },
  ...
});

I am still trying to figure out if ember-auto-imports can be configured to allow for improving the build times.

Ok so I think I found the issue in ember-auto-importer. The default rule for the babel loader is here if we could inject options into the options parameter there it could fix the deoptimized style warning, however, near as I can tell there's no way currently to update a config setting that is in an array, since the config merging will always concatenate arrays.

FWIW, you can specify any arbitrary Babel configuration now in the beta release (if you use a babel.config.js).
Doesn't resolve the ember-auto-import angle, but does unblock access to all other (previously hidden) Babel configuration options.

Yes I saw that code path. Near as I can understand though you have to recreate the entire config that this plug-in would generate automatically. There is an issue where slow build times are acknowledged for large projects https://github.com/ef4/ember-auto-import/issues/270

Near as I can understand though you have to recreate the entire config that this plug-in would generate automatically

Hmm, not sure what you mean. I would expect folks to be able to end up with a Babel config like mentioned in our README.
Something like:
// babel.config.js

const { buildEmberPlugins } = require('ember-cli-babel');

module.exports = function (api) {
  api.cache(true);

  return {
    presets: [
      [
        require.resolve('@babel/preset-env'),
        {
          targets: require('./config/targets'),
        },
      ],
    ],
    plugins: [...buildEmberPlugins(__dirname)],
  };
};

Most of the plugins are still handled automatically in that ...buildEmberPlugins(__dirname) invocation.

Ah that makes a lot more sense, yes this seems pretty easy to use.

Hi there -- I wanted to continue the discussion here because I'm still not sure we're able to pass babel configuration keys other than presets and plugins if we write our own babel.config.js.
In the source, getBabelOptions seems to be the function which looks for the configuration for babel but it appears to only check for presets and plugins. If I wanted to pass another property (e.g. compact), how does that propagate accordingly?

Hmm, I don't think so. getBabelOptions should not be used if you opt-in to using babel.config.js setup.
See this branch:
https://github.com/babel/ember-cli-babel/blob/4c3b9091d7c711ecb804a52226409b409a702d82/index.js#L67-L85
It directly uses Babel API's to load the configuration from that file, it should support 100% of what Babel supports.

Interesting -- I'm explicitly passing compact: false for my configuration but I'm still seeing a warning around deoptimised styling.
Thank you for the quick response.

@justinzelinsky see my comment that warning is actually coming from ember-auto-import and has no good way to override it.

Understood! Thanks for the help!

Hey guys, I'm also trying to use this configuration to set compact: false, but I'm having difficulties in getting it right.
I'm using ember-cli: 3.15.2 and ember-cli-babel: 7.26.6. I'm setting the options as explained in ember-cli-babel README. However, if I use the original babel.config.js as it is in README, I get a lot of errors like these:
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'node_modules/@sentry/browser/esm/exports' in '/home/micael/code/weldnote-frontend/node_modules/@sentry/browser/esm'
<EMAIL_ADDRESS>1:0-34:2
 @ /tmp/broccoli-96956DVwtAwZ2klu2/cache-891-bundler/staging/app.js
 @ multi /tmp/broccoli-96956DVwtAwZ2klu2/cache-891-bundler/staging/l.js /tmp/broccoli-96956DVwtAwZ2klu2/cache-891-bundler/staging/app.js

ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'node_modules/@sentry/browser/esm/integrations' in '/home/micael/code/weldnote-frontend/node_modules/@sentry/browser/esm'
<EMAIL_ADDRESS>1:0-34:2
 @ /tmp/broccoli-96956DVwtAwZ2klu2/cache-891-bundler/staging/app.js
 @ multi /tmp/broccoli-96956DVwtAwZ2klu2/cache-891-bundler/staging/l.js /tmp/broccoli-96956DVwtAwZ2klu2/cache-891-bundler/staging/app.js

(...)

I noticed that there is an disableModuleResolution option and indeed setting it to true will allow for the server to run. But accessing the page, it crashes with the following JS error:
Uncaught Error: Could not find module `ember-classic-decorator` imported from `weldnote/routes/application`
    at missingModule (vendor.js:259)
    at findModule (vendor.js:270)
    at Module.findDeps (vendor.js:180)
    at findModule (vendor.js:274)
    at requireModule (vendor.js:36)
    at Class._extractDefaultExport (vendor.js:288927)
    at Class.resolveOther (vendor.js:288587)
    at Class.resolve (vendor.js:288650)
    at resolve (vendor.js:25607)
    at Registry.resolve (vendor.js:25147)

I tried debugging get-babel-options.js to see if there were any differences between the default configuration (no external file) and providing the babel.config.js file. I found that ember-classic-decorator was missing from the plugins list in the later solution. I tried to add it like this:
// full `babel.config.js` i'm using

const { buildEmberPlugins } = require('ember-cli-babel');

module.exports = function (api) {
  api.cache(true);

  return {
    presets: [
      [
        require.resolve('@babel/preset-env'),
        {
          targets: require('./config/targets'),
        },
      ],
    ],

    plugins: [
      // if you want external helpers
      [
        require.resolve('@babel/plugin-transform-runtime'),
        {
          version: require('@babel/plugin-transform-runtime/package').version,
          regenerator: false,
          useESModules: true,
        },
      ],
      ...buildEmberPlugins(__dirname, {
        disableModuleResolution: true,
      }),
      'ember-classic-decorator/lib/classic-decorator-transform-v4.js',
    ],
  };
};

But to no avail, the returned options in get-babel-options.js is exactly the same as without my edit. Moreover, nothing I do in plugins seems to affect what ends up in returned options. Maybe I'm looking in the wrong place?
Can somebody provide me some guidance on this? Or better yet: is there an easier way to tell ember-cli-babel to use auto-generated configuration file and then just change the compact property?
Thanks for any help!

If you came across this post because you have big files that are being compacted and you need to see the source (as was my case), check out the solution that worked for me in this stack overflow question.


Makefiles

Updates to the makefiles, to remove the need for "smake setup", now Debug and NoDebug-directories get's created if they don't exist instead. Also removes NOALVS to make it compile with SAS/C 6.58.

Weird, I can't see your comment about DEFINE=NIKRELEASE=$(NIKRELEASE) in Server/makefile here on github, but I have removed it and pushed the change to my makefiles-branch now!

On my side once you had pushed the fix to the branch the diff became outdated. In the conversation tab I can click on "Show outdated diff" to see the comment. It doesn't seem to be visible in the "Files changed" tab anymore though.


osd memory target calculation seems incorrect

On mimic using stable/3.2 on commit bf8bd4c0f1337d0867b319139a5f701758078df3. I didn't see modifications to the relevant lines in the master branch so I believe this issue should still persist.
The calculation used to set osd_memory_target in the line below seems to be incorrect. With several machines, each with 64GB of ram and 1 OSD in each, this calculation set osd_memory_target to 48GB of RAM and caused massive resident memory in my OSD daemon. I had to manually edit the .j2 to use the osd_memory_target in all.yml to set it to the reasonable 4GB default.
https://github.com/ceph/ceph-ansible/blob/c878e99589bde0eecb8ac72a7ec8bc1f66403eeb/roles/ceph-config/templates/ceph.conf.j2#L103
I haven't investigated how the line below calculates the value but it results in a memory value that is way too large in my case.

The calculation used to set osd_memory_target in the line below seems to be incorrect.

It's no incorrect. It's just not adapted to your setup.
On non HCI deployment the memory target of all OSD on the same node is 70% of the RAM.
This is control via the non_hci_safety_factor variable (default to 0.7).
If this value is too high for you then you can still override it in the group_vars [1]
Setting this value to 0 will fallback to the 4G RAM configuration or alternatively reduce the ratio to 0.1 or 0.2.
non_hci_safety_factor: 0

If this value is still too high (4G) for you then you can also set the osd memory target value via the ceph_conf_overrides variable.
ceph_conf_overrides:
  osd:
    osd memory target:<PHONE_NUMBER>

[1] https://github.com/ceph/ceph-ansible/blob/master/group_vars/all.yml.sample#L375


[Sysman] Add GPU Effective frequency

Add new frequency type - effective frequency to for BMG ARC control GUI.
Add new frequency type struct  - zes_freq_state_t.effective

Add new frequency type struct  - zes_freq_state_t.effective

Defer this API for now since its a client only feature


Windows: Fix wrong initial size passed to rendering driver

DisplayServerWindows was passing the window size including the window frame. Pass the client size instead.
I am not sure if this has any visible effects, probably just a very minor thing.

Thanks!


Fixing setup.py

Was getting errors running setup.py because AttributeError: type object 'zip' has no attribute 'seek'. I think this fixes the problem, although I'm not seeing any changes to timestamps, so maybe I'm not doing something right.

Thanks! I will check again locally that timestamps work and will merge your PR or rollback my commit entirely.

Yup, timestamps are restored correctly.


Login page error

Hi volkanceylan
I have deployed a Serenity app in IIS8 and login page with error.

Check your server errors, sql, connections, settings, users, i can't help you with deployment beyond what is written in FAQ, Troubleshooting.

Application pools setting integrated mode and enable 32-bit application is true fixed it.
Thank you.


[Demangle] Abort less eagerly to allow de-mangler recognizing global protocols. rdar://29188832

@swift-ci please smoke test

@jckarter mind reviewing this? This patch fixes a de-mangle failure in SourceKit.

@swift-ci test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 2b590ebd66e0f43fc88147c7bf53243abaa732f5
Test requested by - @nkcsgexi

@swift-ci Please smoke test

@swift-ci please smoke test

Joe is on vacation. @rjmccall mind reviewing if this change makes sense?

Since no field experts raise any objections and all tests pass, I'm merging this to fix SourceKit.


Problems with subscribing to announcements

I've tried to subscribe to the announcements several times using email addresses from different provides such as Gmail and GMX. I checked my inbox and spam folder. But unfortunately I've never received a confirmation email.

@BrownBurger yes, unfortunately, the announcements list is known to be dead.  PR #610 redirects people to following an RSS feed, but that PR hasn't received any review yet.  If it takes much longer to get review, perhaps I'll just delete the announcements text in the meantime so we aren't wasting the time of people such as yourself (sorry about that).
Closing this issue as a duplicate of a known issue, but thanks for reporting!


Symfony's function import has different signatures, it is not fixed when transpiling to 7.1

Symfony has the same function import from class FileLoader with different signatures, and these are not harmonized when transpiling to PHP 7.1.
In Symfony\Component\Config\Loader\FileLoader it has bool $ignoreErrors = false (source):
public function import($resource, string $type = null, bool $ignoreErrors = false, string $sourceResource = null, $exclude = null)

In its extending class Symfony\Component\DependencyInjection\Loader\FileLoader it has $ignoreErrors = false, so bool is missing (source):
public function import($resource, string $type = null, $ignoreErrors = false, string $sourceResource = null, $exclude = null)

When running this in PHP 7.1 it shows a warning (first lines of this screenshot):

I guess that, when transpiling, the first function should have the bool typehint removed.

This seems like a missing contra/covariance substitution, will look into this, it should work fine.
Just in case, would you send here your composer.json?


Chef client does not accept relative dir

as normal user, If I invoke
chef-client --local --config config_dir/file.rb

I get
[2015-01-25T18:32:19-06:00] ERROR: Permission denied @ dir_s_mkdir - /config_dir

And if I use a abs path it works!

This needs a thorough review of all config loading code to ensure it all DTRT.

Have tested with chef-15 seems it working as expected.


Support for Image resource.

:tada: This PR is included in version 1.25.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Ability to give names besides "tabs" to a container which holds sub-tabs

AFAICT (apologies, I'm very new to the egui ecosystem and feeling my way around and possibly missing things) if I have nested tabs eg.
let top_level_tab1: TileId = tiles.insert_tab_tile(tabs1);
let top_level_tab2: TileId = tiles.insert_tab_tile(tabs2);
let top_level_tabs = tiles.insert_tab_tile(vec![top_level_tab1, top_level_tab2])
Container::new_tabs(top_level_tabs)

There's no function to assign a name to top_level_tab1 and top_level_tab2. They're both just labelled "Tabs"

https://github.com/rerun-io/egui_tiles/blob/f835c4df1cc260a58122a8d37c7c3738902b9643/examples/simple.rs#L7-L12
Does this fit your use case?


Consider factoring of ModelMetadata

ModelMetadata contains data today used for the following purposes:

Structure (properties/types)
Display (data-annotations)
Validation (data-annotations, requiredattribute)
Model Binding (bindattribute, frombodyattribute)

To extend the set of provided data or customize discovery, users must author an IModelMetadataProvider, which is very complex.
We should think about the factoring of what data needs to be part of ModelMetadata and what can be moved to a higher level. This could make authoring an IModelMetadataProvider easier, increase cacheability, etc.

90cef3b9ca86f56bd49700f632474e7db068816a


Prepopulation of manuscript data

Adds three main chunks of functionality:

Creates fields on the Describe Dataset page for entering the journal and manuscript number, if available. These get added as internal data fields with data_type "publicationISSN" and "manuscriptNumber."

Sets up autocomplete on the journal field, so that you can type in a journal name instead of an ISSN and a list of matching journals with ISSNs is pulled from Crossref, and while the name is displayed to the user, the ISSN is internally stored in the internal data field.

This feature is a bit slow because Crossref's API is really minimal for the /journals query endpoint. It doesn't give you any good way to discover the match score or tweak the result order in any way, so basically I had to set it up to make two calls: one to find out how many results there are, then another to return all the matched results. The resulting list can be very long for a short search string like "Cell," but there's no other way to guarantee that a short journal name will be in the results. Someone else can hook into this later and do some kind of sorting to return a smaller list if necessary.

Added the "Add Manuscript Metadata" button. If the ISSN/Manuscript Number combination is available in the Dryad Classic journal integration system, the dataset is prepopulated with the metadata from that manuscript, overwriting whatever was there before. This works via PUT, so even if you change the manuscript number and journal name and then redo it, you'll repopulate the metadata over again. It doesn't change anything if the metadata isn't available.

I have everything working on the dev server:

Go to dryad-dev and log in
Create a new dataset
Choose “Applications in Plant” and watch the autofill (it takes a sec) and choose “Applications in Plant Sciences”
Enter APPS-D-17-00113 as the manuscript number
Hit the “Add Manuscript Metadata” button

The package page should refresh with the new metadata prepopulated.

Okay, I've redeployed my Dryad Classic server with the new changes that do the user assignment. I tested it with your other login, and it works again.


Replace the way logs get mocked

Take a look to this and use it on our tests : https://docs.pytest.org/en/7.1.x/how-to/logging.html#caplog-fixture

might be the cause of #270


a318 template

idk if there is one already but i made one

update

idkif other one worked but
most recent^


fs2-3.0.0-M3

Gets us back on milestones.

This shouldn't be controversial, and it's not a production branch yet, so I'm going to keep things moving.


Add ability to provide cluster name to an instance instead of cluster arn

Is your request related to a problem? Please describe.
Unable to use the module to import existing cluster instance due to that instance having the cluster_identifier set as the cluster name and not the arn which this module uses.
Describe the solution you'd like.
Add an override to allow using the cluster name for the cluster_identifier

We're trying to use the module to import a postgres cluster and cluster instances that was initially created through the AWS console.
The cluster imports successfully and we can then manage in code.
However when we import the cluster instances, whilst it allows an import there is a mismatch on the cluster_identifier on any subsequent plans requiring a resource replacement. This mismatch is highlighted below
~ cluster_identifier                    = "cluster-qa" -> "arn:aws:rds:us-east-1:9999999999:cluster:cluster-qa" # forces replacement

This pull request allows us to populate cluster_identifier with the value that the AWS API is passing back, upon our tests we just pass in the cluster name and then it no longer forces a replacement.

Hope that makes sense, if not please let me know and I can clarify further :)

@bryantbiggs please could we get some feedback on this? 🙏

I haven't had time to look into this but I don't think a module change is required due to importing resources. This sounds like there is an issue with the provider or an issue with how the resource was imported

@bryantbiggs thanks for your advice - managed to get to the bottom of it in the end.
Problem was with the terraform import being done with the arn and not just the identifier. When we changed this all lined up
Closing pull request now, thanks anyhow though 🙏


Problem in code understanding

Hi Rakhlin,
I have doubt in your code written kept in the link "https://github.com/alexander-rakhlin/CNN-for-Sentence-Classification-in-Keras/blob/master/sentiment_cnn.py"  in sentiment_cnn.py file file at line number 81.In this embedding_weights which is a list and embedding_weights[0] is np.narray, how you can pass np.narray as index to the array index. It should technically give error something like "arrays used as indices must be of integer (or boolean) type".
Kindly help me I am stuck at this step. It will help me a lot in my college project. Attaching screenshot for you reference.

Hi,
x_train, x_test are integer

Can you please see initial code. x_train is not integer. It is numpy array. You loaded imdb data in x_train and x_test.

They are arrays of integer type and therefore can be used as indices.

Its a n*m np.array of shape (25000, 400). You can't even give normal array as index too

I suggest you to run the code in debugger and see how this indexing works.


Method 'facets' not found when updating diamond

I get the following error after running deploy the second time on a diamond. It seems like the Loupe facet is not registered / deployed in the initial deployment of the diamond.
buidler: 1.4.3
buidler-deploy: ^0.5.11
Error: ERROR processing /Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/deploy/3-deploy-contracts.js:
Error: call revert exception (method="facets()", errorSignature=null, errorArgs=[null], reason=null, code=CALL_EXCEPTION, version=abi/5.0.3)
    at Logger.makeError (/Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/node_modules/@ethersproject/logger/src.ts/index.ts:205:28)
    at Logger.throwError (/Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/node_modules/@ethersproject/logger/src.ts/index.ts:217:20)
    at Interface.decodeFunctionResult (/Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/node_modules/@ethersproject/contracts/node_modules/@ethersproject/abi/src.ts/interface.ts:326:23)
    at Contract.<anonymous> (/Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/node_modules/@ethersproject/contracts/src.ts/index.ts:291:44)
    at step (/Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/node_modules/@ethersproject/contracts/lib/index.js:46:23)
    at Object.next (/Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/node_modules/@ethersproject/contracts/lib/index.js:27:53)
    at fulfilled (/Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/node_modules/@ethersproject/contracts/lib/index.js:18:58)
    at process._tickCallback (internal/process/next_tick.js:68:7)
    at DeploymentsManager.runDeploy (/Users/johannes/Documents/dev/actus-protocol/ap-monorepo/packages/protocol/node_modules/buidler-deploy/src/DeploymentsManager.ts:851:19)
    at process._tickCallback (internal/process/next_tick.js:68:7)
error Command failed with exit code 1.

Could you try with the latest version of buidler-deploy (0.6.0-beta.36) ?
Or at least version 0.5.18
Also if you have a reproduction repo, that will be very helpful.
Note that the version of Diamond that buidler-deploy uses has the Loupe facet builtin. so not sure what causing the error

Bumped to 0.6.0-beta.36 but experiencing the same error.
Here is a link to the buidler-deploy script in the example repo: atpar/protocol

do you reset ganache each time ?
If so this is normal behavior. --reset is needed to ensure past deployments are not considered

Ok that makes sense. But I would expect for the non-diamond contracts to throw the same error.

for other contract, they would not fail on deploy, but if you try to call a function on them (like .facets() for diamond) it will also fails
unless the contract code changed which would then trigger a deployment

actually I forgot, maybe the deployment is always executed because the tx hash for the previous deployment is not found.

actually I forgot, maybe the deployment is always executed because the tx hash for the previous deployment is not found.
Yes. We do call some methods in the deployment workflow and they all succeed.

Thanks. You can close the issue.


Add --releaseType flag to update

An optional update flag has been added, allowing you to specify which latest version of the type you want to update to. By default, the latest version will be used, which is the same behavior as before. This will only work with modrinth mods.
For example if you want to update to the latest stable release of the mod just add --releaseType release to your update command.

Yes, that would be great! Maybe an option to specify that you want to download a release version, but if none are available, download the newest one (even if it's beta/alpha) could be nice here.


unbound: 1.12.0 -> 1.13.0

https://lists.nlnetlabs.nl/pipermail/unbound-users/2020-December/007102.html
Fixes: CVE-2020-28935

Motivation for this change
New upstream release.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I don't think we are affected by CVE-2020-28935, so I don't think this warrants a backport to stable.

This should have been backported: https://github.com/NixOS/nixpkgs/pull/110673

This should have been backported: https://github.com/NixOS/nixpkgs/pull/110673


Added Colab Notebook Link

FFmpeg and FFprobe is already installed on Colab. No need to add again.

@athenasaurav, thanks for this. I'm closing this PR in light of #64. I couldn't find a way to edit permissions on your collab, so decided to make a new one. The notebook also supports inference on t4 GPU :)
Thanks for the initial setup & PR


openssl_probe::init_ssl_cert_env_vars

openssl_probe::init_ssl_cert_env_vars() now returns a bool which fails to compile at this line:
https://github.com/sfackler/rust-native-tls/blob/master/src/imp/openssl.rs#L94
   |
94 |     ONCE.call_once(|| openssl_probe::init_ssl_cert_env_vars());
   |                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `()`, found `bool`

That is a break in openssl_probe, and the release has been yanked.


Add AAD Error Handling

🗣 Description

Updated all of the ConvertTo-Json calls to use the current syntax
Wrapped all API calls inside the TryCommand function
Updated Rego so that it always expects provider output items to be lists

💭 Motivation and context
Closes #83, #68, and #19.
🧪 Testing

Ran code against Test Tenant 1 (G5) on both main and this branch, closely comparing output and ensuring that all differences are expected.
Ran code against Test Tenant 2 (G3) on both main and this branch, closely comparing output and ensuring that all differences are expected.
Ran code in Test Tenant 4 (E5) as an unprivileged user to generate errors. Ensured that all errors generated were handled correctly.
Ran code in Test Tenant 4 (E5) as an admin after deleting all conditional access policies. Was able to reproduce the error described in #19 in main; bug is not present on new branch.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Add a tag or create a release.

@nanda-katikaneni Please regression test this using the new semi-automated test script against Test Tenant 5 (E3) for AAD only.

[ ] Generate golden copy of provider and test results using main branch code
[ ] Run semi-automated test script in the "add-error-handling" branch using Rego test option to make sure outputs are the same
[ ] Run ScubaGear using the "add-error-handling" code and make sure the provider functions as expected

I compared the report results for my CISA account, my test account, and an account with no permissions on the main branch and on this branch.
This account correctly displays Errors on the report in all cases, all unit tests pass, and the JSON outputs are correct as well without redundancies.
I approve.

@tkol2022 I discovered a bug. You can see if by running Disconnect-MgGraph then the tool with -Login $false
The issue is with how I handled $PrivilegedUsers.  Everything said in the comment is correct...but on error, $TryCommand returns an empty list, not a dict like the function normally does, resulting in invalid json.
I don't want to interrupt your testing. Should I fix this now or wait until you finish?

You can fix it now. Should not affect my testing. Thanks!

Not sure if this is a bug?
So I know you mentioned at today's meeting that there are some cases where a policy will be missing from the report if its respective provider Cmdlet fails but I can't remember the full context of what you said, so you may know about this already.
When the Get-MgDirectorySetting errors, the Baseline 2.7: Policy 3 "Group owners SHALL NOT be allowed to consent to third-party applications" gets excluded from the report.

Might need to add logic to handle the scenario where Get-MgSubscribedSku fails
If Get-MgSubscribedSku fails, we should not call Get-PrivilegedUser and Get-PrivilegedRole because those functions depend on the list of SKUs to determine whether or not they should execute PIM-specific logic. Both of those functions have the line of code below:
  if ($TenantHasPremiumLicense) {
You might need to add an if block to ensure that those functions are only called if Get-MgSubscribedSku succeeds. See my example below.
    $ServicePlans = $Tracker.TryCommand("Get-MgSubscribedSku").ServicePlans | Where-Object -Property ProvisioningStatus -eq -Value "Success" -ErrorAction Stop
    if($ServicePlans){
       ### Put all code that calls Get-PrivilegedUser and Get-PrivilegedUser HERE
    }

Not sure if this is a bug?
So I know you mentioned at today's meeting that there are some cases where a policy will be missing from the report if its respective provider Cmdlet fails but I can't remember the full context of what you said, so you may know about this already.
When the Get-MgDirectorySetting errors, the Baseline 2.7: Policy 3 "Group owners SHALL NOT be allowed to consent to third-party applications" gets excluded from the report.

Good catch. I thought this was worth fixing, so I just pushed a patch for it. We should fix all cases that we know cause something to be missing from the report. What I was saying in the meeting you referred to was that if there ever were something missing from the report (e.g., due to a Rego bug), we don't currently have code in place to flag that for the user, so unless catches it like you did, it will go unnoticed.

Added the logic you suggested.
@schrolla All requested changes have been made.


fix a word

No, this is the intended word. Although I could see an argument for either working.

No, this is the intended word. Although I could see an argument for either working.

ok.


Modularize the functions in the generate_simulated_data() function

This PR closes #3 and attempts to make the generate_simulated_data() function a series of calls to other functions that represent the critical model components, with the goal being that you can swap in and out different functions for different models (for example how to get the subpopulation R(t) estimates).

@dylanhmorris This is ready for re-review. I could combine more things into larger wrapper functions, but my preference is that each individual component have its own function. It does still mean that the generate_simulated_data() function is large and verbose, but I don't think there's a great way to avoid this in a complex model.

@dylanhmorris Let me know if this is good to merge, I made a separate issue for specifying all inputs https://github.com/CDCgov/ww-inference-model/issues/61

@dylanhmorris This addresses #65 and updates the package data accordingly.

@damonbayer This PR adds the true_global_rt dataframe to the package data. This contains the date and time columns and the unadj_rt and the realized_rt.

@dylanhmorris @gvegayon Ready for re-review.
I think it makes sense to make a separate issue to look into the back-calculation of R(t) and see why we are not able to reproduce a flat repeated value each day. I think it might have something to do with indexing/initialization but I'd want to look into it. Should be easier to do with a more functionalized version of the generate_simulated_data() function.

Ok I think this fixes the back-calculation of R(t), by padding a 0 to the generation interval which replicates the forward

@dylanhmorris Let me know if there are small changes needed to get this through review. Would be ideal to fix some of this before we have people using it, even if just prototyping. Apologies that it became such a large PR, but I think most of it has been reviewed by now :(

LGTM, thanks @kaitejohnson. Can you make a separate issue to reconsider the R(t) additional noise in the simulated data?

Sure, you mean to consider the noise being generated in log scale?

LGTM, thanks @kaitejohnson. Can you make a separate issue to reconsider the R(t) additional noise in the simulated data?

Sure, you mean to consider the noise being generated in log scale?

But also whether to have a user-specified timeseries with added noise at all (versus either/both of generating the $\mathcal{R}(t)$ from the underlying stochastic process or taking a user-specified timeseries as is (which the user can generate with whatever process they like)

LGTM, thanks @kaitejohnson. Can you make a separate issue to reconsider the R(t) additional noise in the simulated data?

Sure, you mean to consider the noise being generated in log scale?

But also whether to have a user-specified timeseries with added noise at all (versus either/both of generating the R ( t ) from the underlying stochastic process or taking a user-specified timeseries as is (which the user can generate with whatever process they like)

Yeah, I guess the "added noise" could certainly happen outside the function in any simulation analysis.


Fixing sprockets wysihtml5 problem (2)

This works but labels are missing from some toolbar buttons. I guess this is another issue to look into.

I'm experiencing this problem as well. Good job. :+1:

Asset alias check has been reverted by rails/sprockets-rails#251.
Using the master of sprockets-rails works now.
gem 'sprockets-rails', github: 'rails/sprockets-rails'

Thats good. resolved my problem. Thanks :+1:

Does anybody have another solution? Because It crashes with using compass-rails gem
Bundler could not find compatible versions for gem "sprockets":
  In Gemfile:
    compass-rails (>= 0) ruby depends on
      sprockets (< 2.13) ruby

    sprockets-rails (>= 0) ruby depends on
      sprockets (3.0.0)


Implement groundspeed_undershoot for flying without airspeed sensor

Currently groundspeed undershoot changes the target airspeed
https://github.com/PX4/Firmware/blob/master/src/modules/fw_pos_control_l1/fw_pos_control_l1_main.cpp#L1015
When flying based on throttle setting this would be bypassed and risk fly-aways

Isn't TECS only using potential energy with airspeed disabled?

@sanderux any thoughts on how you'd like to implement this?

Not yet, do you have any idea's?

Just an idea that it's not an obvious addition :wink:
Perhaps a part of TECS in no airspeed mode to maintain a minimum groundspeed.

why not obvious? there is a groundspeed_undershoot detection for vehicles with an airspeed sensor, so wh not for vehicles without one?

Not obvious how to implement. As you know groundspeed undershoot just adjusts airspeed. The rest of the controller is oblivious.

Yes, the way to scale it might be tricky. perhaps it should add some throttle, wait to seewhat the ground speed does, add a little more, etc

Is this done since then?

Not yet

" > perhaps it should add some throttle, wait to seewhat the ground speed does, add a little more, etc "
@sanderux Something like a simple integrator controller?

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)


Type checks

Type checks: The updateList.js code does not have any explicit type checks. It would be beneficial to add type checks, especially if the inputValue parameter is expected to be of a specific type. Eg: A type check to ensure that inputValue is a string before assigning it to label.textContent.

Javascript is not good at handling type checks. Typescript would have worked better for this implementation.


Remapping (almost) all RPC status codes to our exceptions in datastore.

Fixes #2497.
(Relevant to #2583 as well.)
FYI @jonparrott

@jonparrott @daspecster PTAL

I like this better IMHO.

FYI I am toying with Circle CI. Looks like Travis is being helpful for the time being though

Sorry to be late to the party.  Wouldn't the _GRPC_ERROR_MAPPING and _grpc_catch_rendezvous bits logically live in cloud/google/cloud/exceptions.py?  I presume we might want to use them in other APIs.

@tseaver I intentionally didn't do that, mindful of two things:

it may be premature to lift into core (i.e. YAGNI, but not quite YAGNI, more like YAGNI-yet)
changes in core have different implications release-wise than in the service packages

@dhermes FWIW, anywhere we are currently sniffing at StatusCode STM to be a candidate:
core/google/cloud/_testing.py
core/google/cloud/streaming/exceptions.py
core/google/cloud/streaming/http_wrapper.py
core/unit_tests/streaming/test_http_wrapper.py
datastore/google/cloud/datastore/_generated/datastore_grpc_pb2.py
datastore/google/cloud/datastore/connection.py
datastore/unit_tests/test_connection.py
docs/error-reporting-usage.rst
error_reporting/google/cloud/error_reporting/client.py
error_reporting/unit_tests/test_client.py
logging/google/cloud/logging/_gax.py
pubsub/google/cloud/pubsub/_gax.py

OK.


Remove 'success' and 'reason' as outputs in claim commands + Wallet unit test

Replacement for https://github.com/lbryio/lbry/pull/306 (branch from lbryio/lbry instead of kaykurokawa/lbry)
Remove 'success' and 'reason' as outputs in Daemon claim commands, as we throw an exception instead for failed claims instead.
Have support and abandon claim commands do the same.
Added test_Wallet.py under tests to test these features, and to implement more tests for Wallet.py in the future.

Ha, wow.  Those simple tests increased coverage by 10 percentage points!

a couple of nits, otherwise LGTM

As per comments, moved around arguments in assertEqual, and use assertFailure instead of custom exception checking function.

Closing as this was merged with https://github.com/lbryio/lbry/pull/339/


fix(forge): rm already set gas field before estimating gas

Motivation
Ref: https://github.com/foundry-rs/foundry/issues/2941
this ensures the gas field is removed before estimating gas via estimateGas

Solution

merging as confirmed proper fix


Microsoft.CodeCoverage is missing

Package Microsoft.NET.Test.Sdk version 16.3.0-preview-20190828-03 is depends on the Microsoft.CodeCoverage which is broken

Sorry for the trouble. Microsoft.CodeCoverage 16.3.0-preview-20190828-03 has been uploaded here https://www.nuget.org/packages/Microsoft.CodeCoverage/16.3.0-preview-20190828-03.
This should fix it.


[Doc Feature][Updated what's new]

Docs Build status updates of commit 502e8de:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/whats-new.md
:warning:Warning
View
Details

msteams-platform/whats-new.md

Line 61, Column 97: [Warning: file-not-found - See documentation] Invalid file link: 'toolkit/TeamsFx.md'.
Line 62, Column 123: [Warning: file-not-found - See documentation] Invalid file link: 'Add/api.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/whats-new.md
92
33
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


The CircleImageView is taking up too much memory

I found that it was better to crop the Bitmap. Because setting up images with CircleImageView takes up too much memory. This may be the reason for the onDraw () method. The author can optimize it

I'd be happy to talk about memory optimizations but this lacks substance.


fix: dialyzer errors on Elixir 1.16

Even though the code still works, dialyzer fails when using Elixir 1.16.
The fix is the same as described in this comment: https://github.com/elixir-lang/elixir/issues/13212#issuecomment-1871339145. I’ve removed the :read_ahead option as I understand Elixir will add it with a reasonable value by default. Let me know your thoughts about this.

This is the same as https://github.com/breakroom/multipart/pull/18 but keeps support for older Elixir versions.

Do you know why does dialyzer fail on this ? The read ahead option seems to be declared in the File.stream_mode type.

Ah ok it's about the order of arguments.
File.stream!(path, 1024, [{:read_ahead, 4096}]) should work.

Ah ok it's about the order of arguments.

that's correct (sorry for taking this long to get back to you, I somehow missed the notification 🙈)

File.stream!(path, 1024, [{:read_ahead, 4096}]) should work.

indeed, that would work as long as you are happy breaking compatibility with older elixir versions (< 1.16). That's the reason behind the Version.compare(System.version(), "1.16.0") expression.

I am not sure why dialyzer is picking that since the if expression is evaluated at compile time.


Update MacVim to 7.4-84

Update MacVim to 7.4-84.
Binary targets OS X 10.8(Mountain Lion), 10.9(Mavericks), 10.10(Yosemite), and 10.11(El Capitan).
Release page: https://github.com/macvim-dev/macvim/releases/tag/snapshot-84

@vitorgalvao Should we be dropping support for older OS versions?

Thank you for the contribution. It was merged directly as 01230483ebc2a717a4d813717874da297c41db11 to keep commit history cleaner. Your contribution is still credited to you.


Fix rare 'file already closed' during restore

What is the purpose of this change? What does it change?
Reworked cache of open target files used by restorer to fix a possible race between two concurrent writeToFile requests.
Was the change discussed in an issue or in the forum before?
See #2183
Checklist

[x] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[ ] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Codecov Report

Merging #2189 into master will decrease coverage by 4.44%.
The diff coverage is 92.3%.

@@            Coverage Diff             @@
##           master    #2189      +/-   ##
==========================================
- Coverage   50.76%   46.32%   -4.45%
==========================================
  Files         176      172       -4
  Lines       14251    14227      -24
==========================================
- Hits         7234     6590     -644
- Misses       5964     6650     +686
+ Partials     1053      987      -66

Impacted Files
Coverage Δ

internal/restorer/filerestorer.go
76.05% <100%> (ø)
:arrow_up:

internal/restorer/fileswriter.go
83.33% <92%> (+3.84%)
:arrow_up:

internal/backend/b2/b2.go
0% <0%> (-80.69%)
:arrow_down:

internal/backend/swift/swift.go
0% <0%> (-78.45%)
:arrow_down:

internal/backend/gs/gs.go
0% <0%> (-74%)
:arrow_down:

internal/backend/azure/azure.go
0% <0%> (-69.46%)
:arrow_down:

internal/backend/swift/config.go
36.95% <0%> (-54.35%)
:arrow_down:

cmd/restic/cmd_mount.go
13.75% <0%> (-42.5%)
:arrow_down:

internal/archiver/file_saver.go
81.57% <0%> (-7.02%)
:arrow_down:

internal/restic/progress_unix_with_siginfo.go

... and 4 more

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 7b8d1dc...2532218. Read the comment docs.

Sadly, I was not able to reproduce the race locally, at least not without adding artificial time.Sleep to writeToFile. My working theory this is due to the difference between macos and linux schedulers, but could also be due to my misunderstanding of how goroutings work. Regardless, I think the change simplifies coordination between concurrent writeToFile invocation and should make things more reliable.


Providing stack trace in messages when validation is delayed

We know that some validation can not be done immediately when recording a command (e.g. tracking image layout). So some of them are deferred at submission time. However, if we hit an error (or a warning) there, then we lose the context from where the offending command comes.
So, a nice (and non-intrusive) feature that would be very practical is if, for such deferred validation, information about the call stack could be saved (at record time) and displayed along with messages. This should be possible to implement at least on some platforms.

Thanks @fduranleau-gl!

There certainly is a cost. Maybe it could be an opt-in feature?


⚠ Jenkins Plugins Artifactory has degraded performance

In fb2644d, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1333 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 3f7fe48 after 12 minutes.


Support cluster scoped deployments of kubefed-operator on Kube (operatorhub.io)

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?

Updates to existing Operators

[ ] Is your new CSV pointing to the previous version with the replaces property?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?

Your submission should not

[ ] Modify more than one operator
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

@abhat it failed on this error: dial tcp <IP_ADDRESS>:8889: connect: connection refused

@J0zi can this be merged?


Can you add conditional fields?

I was wondering if you could add conditional fields like... in a radio-list if "option 2" is selected, you need to provide Passport ID.
If this was just plain angular I would add a ng-if="model.value==''option_2'" and that would be it. How can I do things like this with angular-xeditable?

You probably could do this with a flag that gets flipped by an ng-click on the radio button and the ng-if on the second field to look for that flag.

@kas84 are you still having an issue?

@eugef Please close due to lack of response


Modify socket samples to use user mode

Since user mode support was enabled in Zephyr for sockets (See https://github.com/zephyrproject-rtos/zephyr/issues/8203) at least some of socket-based samples need to be modified to support it.

Ok, so as this was mentioned at yesterday's networking call, let's go forward with this, and let me record response to this ticket as a kind of walkthru.

Let's choose a random socket sample, e.g. samples/net/sockets/http_get , and build it for the default BOARD, which is qemu_x86.
Grepping outdir/qemu_x86/zephyr, there's CONFIG_USERSPACE=y. That fully corresponds to my expectations, as I "know" that qemu_x86 runs with userspace mode enabled for quite a while. That's why we have another, lesser known BOARD qemu_x86_nommu.
Let's how verify that "knowledge" and see where CONFIG_USERSPACE=y comes from:

zephyr/boards/x86/qemu_x86$ grep USERSPACE *
grep: doc: Is a directory
qemu_x86_defconfig:CONFIG_USERSPACE=y
qemu_x86_iamcu_defconfig:CONFIG_USERSPACE=y

So, the response to this ticket is that since https://github.com/zephyrproject-rtos/zephyr/pull/9179 was pushed, we built, and ran all socket samples in userspace mode, with API calls accessed using syscalls. (And yep, I don't know how other people, but I run them relatively regularly.)
For the avoidance of doubt, let's also grep the assembly:

        printf("Response:\n\n");
    4563:       68 bf de 01 00          push   $0x1debf
    4568:       e8 43 d9 ff ff          call   1eb0 <iprintf>
    456d:       5f                      pop    %edi
        __asm__ volatile ("mov %%cs, %[cs_val]" : [cs_val] "=r" (cs));
    456e:       8c c8                   mov    %cs,%eax

K_SYSCALL_DECLARE6(K_SYSCALL_ZSOCK_RECVFROM, zsock_recvfrom, ssize_t, int, sock, void *, buf, size_t, max_len, int, flags, struct sock
    4570:       83 f8 2b                cmp    $0x2b,%eax
    4573:       75 26                   jne    459b <main+0x17c>
        __asm__ volatile("push %%ebp\n\t"
    4575:       31 db                   xor    %ebx,%ebx
    4577:       c7 45 f0 00 00 00 00    movl   $0x0,-0x10(%ebp)
    457e:       b9 ff 03 00 00          mov    $0x3ff,%ecx
    4583:       ba 60 03 40 00          mov    $0x400360,%edx
    4588:       be 92 00 00 00          mov    $0x92,%esi
    458d:       8b 45 e4                mov    -0x1c(%ebp),%eax
    4590:       89 df                   mov    %ebx,%edi
    4592:       55                      push   %ebp
    4593:       8b 6d f0                mov    -0x10(%ebp),%ebp
    4596:       cd 80                   int    $0x80
    4598:       5d                      pop    %ebp

So yeah, after that printf() we have call to recv(), which (after heavy macro machinery on the C side) is rendered as int    $0x80.
So, again, to summarize, whether to work in userspace mode or not, is selected by a particular machine, and for at least one, we've been working like that for quite a while. I know that there's work in parallel to enable userspace mode by default for ARM too.
Not closing right away, to avoid any confusion, because as I mentioned, my initial thought regarding this ticket was "that's how it works for quite a while". So, @laperie, please confirm that this addresses the topic.

Hopefully it's answered.


Allow blank issue reports again

I frequently find myself in the situation where the standard bug issue template fields are all irrelevant for what I want to report, and then I have to first shoehorn my info into the template somehow, save the issue, edit it, and remove all the unnecessary parts. This demotivates me from filing casual issues, e.g. when I see a CI test fail. We should have a way of still filing custom issues without all the templatey bits.

For bug reports, only 2 fields are mandatory (I think both of them make sense for a bug, sometimes I just put a dot in one of them), I personally just leave the other fields empty when not relevant or/and fill them later when I have more info.

Yes, but even the two mandatory ones don't always make sense when I know
exactly what a bug is, and then the result is also messy to read unless I
go in and edit things again to remove all the unnecessary boilerplate. We
should really have a way around this.
On Wed, Oct 2, 2024, 09:46 Ayoub Mrini @.***> wrote:

For bug reports, only 2 fields are mandatory (I think both of them make
sense for a bug, sometimes I just put a dot in one of them), I personally
just leave the other fields empty when not relevant or/and fill them later
when I have more info.
—
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/15069#issuecomment-2387831179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEDLGASRHQTAJTE3TJUE43ZZOQEDAVCNFSM6AAAAABPG4NTOGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBXHAZTCMJXHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yep, or as a middle ground, if people abuse it, we could disallow regular free-form posting again but introduce an explicit "I know what I'm doing" issue type for experts that is just one big Markdown field. For now, let's maybe just try allowing blank issue templates again.


Dataset inspect

Initial dataset inspect tool.
Uses dask_cudf to be able to scale. Works for cats, conts, and label. Dealing with an issue related to list dtypes in dask_cudf dataframes.
@jperez999 for feedback.
@rjzamora for feedback about how I am using dask_cudf and for the list dtype problem.

Thanks @albert17 - I'll look at this asap.  Let me know if there is any overlap with #484

Thanks @albert17 - I'll look at this asap. Let me know if there is any overlap with #484

I don't think so.
As far as I can see: Your PR consist in some tools that make sure a dataset is in a correct an efficient way to be processed by NVTabular.
This PR: Given a dataset extract its stats. General Stats like: num_rows,num_cats,num_conts; but also stats per column: max/min/std/%nans..

This PR: Given a dataset extract its stats. General Stats like: num_rows,num_cats,num_conts; but also stats per column: max/min/std/%nans..

Perfect!


Scatter op - Update Golden function

Update the golden function of the scatter op which is implemented specifically for the mistral model (op implemented as composite version)

PR : https://github.com/tenstorrent/tt-metal/pull/14998


User profile levels

Can Redfish users have different profile levels (admin, users)? Check Redfish specification and schema.

I think that a role mechanism could be implemented (mapped from OneView) using Role and ManagerAccount.


Go to Definition of task from launch config

Launch configs can have a preLaunchTask specified. Users should be able to go to the definition of the task by placing the cursor in the string and activating the editor.action.revealDefinition command.
This could also work from within the tasks file itself, for options that take tasks as values.
The current workflow to accomplish this is manually opening the tasks file and searching for the task by name, which can get tedious.

Interesting idea, not sure whether we have info like that for tasks already

We sort of do, but it's not exposed:
https://github.com/microsoft/vscode/blob/eda166b7fa6377d2bd2fd2c33e5a844ea1b7bddd/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L1277-L1278

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Problem: hare mini-provisioner doesn not generate motr confd.xc

Hare mini provisioners config stage generates all the required configuration
files for Motr and Hare but does not convert confd.dhall to confd.xc (Motr
configuration file in Motr xcode grammar.
Solution:

Convert generated confd.dhall in text using dhall text command.
Pass the output of dhall text to m0confgen to generate confd.xc.

Signed-off-by: Mandar Sawant<EMAIL_ADDRESS>

Abandoning this PR as confd.xc is generated by configure script.


Adds accents, fix some typos and wording.

Thank you for the contrib! I'm trying to get aurelia-validation back in top shape in the coming days.
Meanwhile, could you verify you signed the CLA? :)

Alright, CLA signed! :)

CLA confirmed. @janvanderhaegen You can move ahead with this. Apologies for the delay on the confirmation. We are investigating automating this soon.


Function Syntax Suggestions / helpers

Hi there,
I would like to know if there's a way to get some syntax help while writing a MySQL function on dBeaver.
For example, on Navicat when I write a fuction I get a list of the syntax possibilities that Mysql gives me, and after I press Enter I get them auto-written in the code:

Here another example:

I don't see anything similar on dBeaver and I couldn't find any other topic related to this issue.
I'm not sure if this is something (too) specific from Navicat or if it is easily replicable also on dBeaver.
Actually this feature speeds alot the process when you have to write code, since it provides already not only the syntax but also the parenthesis schema without the need of perfectly knowing it and avoiding useless errors.
Thanks

Thank you for the future request


In-memory cache

This PR adds a in-memory cache to the Petastorm loader. This enables very fast reading of data and removing the IO/netowrk bottleneck.

Can you explain what exactly is being cached? From a quick glance I couldn't figure out the difference between the two dataloaders.

@fps7806 Caching works by preserving the values loaded into the ShufflingBuffer into the memory or GPU and not removing them. This way, if in-memory-cache is enabled, the values will only be read once from disk/network into memory. Each worker only caches its own shard of the data and not the entire data set.


Unifi Controller wants MongoDB 4.2 which was dropped.

Describe the bug
Since #236259, MongoDB 4.2 was dropped.
However, the Unifi Controller still depends on it:
https://github.com/NixOS/nixpkgs/blob/af8279f65fe71ce5a448408034a8c06e2b4b2c66/nixos/modules/services/networking/unifi.nix#L45
Steps To Reproduce
Steps to reproduce the behavior:

services.unifi.enable = true;
BANG

Expected behavior
Unifi Controller enables and works as expected.
Additional context
Unifi Controller bundles a stone age MongoDB, 4.2 worked fine however.
I am unsure if later versions work without manually hacking around, but I think more care should've been taken
to ensure it still works.
Notify maintainers
@pennae - Unifi Controller
@wegank - Author of the 4.2 removal PR

Master seems to have 4_4 specfied: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/networking/unifi.nix#L45

Yes master has 4_4 specified and unstable has 4_4 and the update is already live on unstable. And 4_2 is still available on 23.11 So I think we can close this ^^
https://nixpk.gs/pr-tracker.html?pr=236259
When did you encounter that? You might want to update the system state version at some point.

... Huh. Apologies, I was under the impression that it's still the old version. Haven't tried it, but I hope it upgrades on its own.


Feat: add cloud provider & network plugin

The design of cloud provider & network plugin: #15

Codecov Report

Merging #16 (c29ea00) into master (99968ba) will decrease coverage by 3.17%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   29.71%   26.54%   -3.18%
==========================================
  Files           6        6
  Lines         535      599      +64
==========================================
  Hits          159      159
- Misses        368      432      +64
  Partials        8        8

Flag
Coverage Δ

unittests
26.54% <0.00%> (-3.18%)
:arrow_down:

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

Impacted Files
Coverage Δ

...kg/controllers/gameserver/gameserver_controller.go
0.00% <0.00%> (ø)

pkg/controllers/gameserver/gameserver_manager.go
16.83% <0.00%> (-6.15%)
:arrow_down:

pkg/util/gameserver.go
35.83% <0.00%> (-2.22%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

/LGTM


cannot find RaXML results even though is in path

When I try to run FunDI, I hit the following error
perl FunDi.pl -t uniprot-meropsC12.shortnames.nw -a uniprot-meropsC12.shortnames.afasta -s subtree.def -P qmmraxml
Retrieving subtree definitions from subtree.def...
Optimizing branch lengths of input user tree using QmmRAxML
Executing QmmRAxML with following command-line: -s uniprot-meropsC12.shortnames.afasta -t uniprot-meropsC12.shortnames.nw -n uniprot-meropsC12.shortnames.afasta_optimized.out -m PROTGAMMALG -u 9componentsDirichlet_LG.dat -f e
Could not find RAxML_result.uniprot-meropsC12.shortnames.afasta_optimized.out. Exiting...

It appears qmmraxml does not write the results to disk. The program is in the path and I can call it from the command line.
Any clues?

Hi Elipapa,
Yes it does sound like qmmraxml may not be executing correctly. If you run just the qmmraxml command that FunDi outputs what do you get?
	_____________________________

From: elipapa<EMAIL_ADDRESS>Sent: Tuesday, July 12, 2016 7:52 PM
Subject: [dgaston/FunDi] cannot find RaXML results even though is in path (#1)
To: dgaston/FunDi<EMAIL_ADDRESS>When I try to run FunDI, I hit the following errorperl FunDi.pl -t uniprot-meropsC12.shortnames.nw -a uniprot-meropsC12.shortnames.afasta -s subtree.def -P qmmraxmlRetrieving subtree definitions from subtree.def...Optimizing branch lengths of input user tree using QmmRAxMLExecuting QmmRAxML with following command-line: -s uniprot-meropsC12.shortnames.afasta -t uniprot-meropsC12.shortnames.nw -n uniprot-meropsC12.shortnames.afasta_optimized.out -m PROTGAMMALG -u 9componentsDirichlet_LG.dat -f eCould not find RAxML_result.uniprot-meropsC12.shortnames.afasta_optimized.out. Exiting...
It appears qmmraxml does not write the results to disk. The program is in the path and I can call it from the command line.
Any clues?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for the quick reply @dgaston !
Running on its own, it appears it has some trouble in parsing the multifasta?
qmmraxmlHPC -s uniprot-meropsC12.shortnames.afasta -t uniprot-meropsC12.shortnames.nw -n uniprot-meropsC12.shortnames.afasta_optimized.out -m PROTGAMMALG -u 9componentsDirichlet_LG.dat -f e
ERROR: Problem reading number of species and sites

Ah I see the problem now. Qmmraxml only reads phylip format files. FunDi has a parser and will convert standard fasta to phylip on input but it isn't a very sophisticated detection mechanism and just looks for standard fasta file extensions people use like .fasta and .fst so where yours is .afasta it wouldn't try and convert it. So if you change the file name, and assuming it is a standard fasta file it should be able to convert it to phylip for you. Otherwise if you run a conversion program first and give the phylip file to FunDi it should work
On Wed, Jul 13, 2016 at 2:47 AM -0300, "elipapa"<EMAIL_ADDRESS>wrote:
Thanks for the quick reply @dgaston !
Running on its own, it appears it has some trouble in parsing the multifasta?
qmmraxmlHPC -s uniprot-meropsC12.shortnames.afasta -t uniprot-meropsC12.shortnames.nw -n uniprot-meropsC12.shortnames.afasta_optimized.out -m PROTGAMMALG -u 9componentsDirichlet_LG.dat -f e
ERROR: Problem reading number of species and sites
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Closing for now, please re-open if you still have issues with the file conversion.


Update UC-Internal.vcxproj

Removes some unnecessary stuff.

Oops completely forgot, thanks

@Seanghost117 Ever heard of Premake?


Fix : Change master branch to main

This fix should change the edit branch to main in the documentation
I didn't update the CHANGELOG, as it doesn't concern the app itself. Let me know if I should do this

[x] read the CONTRIBUTING guidelines
[ ] raised a GitHub issue or discussed it on the projects chat beforehand
[ ] added unit tests
[ ] added integration tests
[ ] updated documentation if needed
[ ] updated CHANGELOG.md

Fixed !

Thanks!


fix(config): provide typescript backward compatibility

Types of changes

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

Description
#6252
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

👍🏽


Buildstream should identify if an element moves from build-depend to depend, and not recompile.

See original issue on GitLab
In GitLab by [Gitlab user @tcanabrava] on Oct 13, 2020, 15:22
when we have a library that is incorrectly configured, and we need to move an element from build-depend to depend there is no need to recompile.
If this library is down on the stack, it get's worse as everything that depends on it will also be recompiled, but the result library is the same.

In GitLab by [Gitlab user @tristanvb] on Oct 13, 2020, 15:36
Seems like this should produce the same cache key for the element itself, but not for any reverse dependencies which have a build dependency on the element which changed.

When you move a dependency from build-depends to depends, you are making a statement that this dependency is required at runtime.
A reverse dependency which build depends on this element, requires this element to run at the time the reverse dependency builds, thus, the new runtime dependency is necessarily a new build time dependency for any reverse build dependencies.

Is this essentially the behavior you are observing ? Or is the element itself also rebuilt as a result of this, not only its reverse dependencies ?

In GitLab by [Gitlab user @tcanabrava] on Oct 13, 2020, 18:46
you are completely right, I misunderstood the issue.


Clarify current VRF tie breaking implementation

Description
Update/correct potentially misleading names of VRF-related functions (in particular csvLeaderVRF), and provide context for why a particular VRF value is used for tie breaking.
Checklist

Branch

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] If this branch changes Consensus and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[x] Self-reviewed the diff
[x] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[x] Reviewer requested

bors merge

I'm not 100% sure, but I suspect the rename of csvTieBreakVRF might require a (small) integration in eg cardano-node and so should be mentioned in the interface-CHANGELOG.md.
Did you check for that sort of thing already, @amesgen ?

Grepping for csvLeaderVRF does not reveal anything in node (and also not in a few other IOG repos I have checked out). From my intuition, a SelectView should be a purely Consensus-internal thing, so I would actually be surprised/consider it tech debt if it would show up in consumers of the Cardano protocol. Where you thinking about a specific way it might be used in e.g. the node?


feat(webvtt): webvtt colors output

Adds color support for SimpleTextDisplayer and WebVttGenerator (only one place to fix both now thanks to #4941). Although it's limited to the restrictions of WebVTT, so it it only works for colors that can be represented as a class name (green, red, aquamarine etc), not hex colors, rgb, rgba etc.
So if the stream has TTML subtitles with colors in other formats it won't work. The same goes if the WebVTT has custom styling that overrides the colors with a color of another format.
Not sure how to/if to cover those cases. It's definitely possible to convert to non-standard class names like hex_color_77CCAAFF or hex_color_bg_77CCAAFF, so that you can catch and handle that as well with a cuechange listener, but it would increase the complexity a lot for what it does.
I didn't add any tests for the new behavior yet, just fixed the ones I broke because of the changes to defaultTextColor. Would be happy to add tests if you think it looks good.
I'll let you be the judge if it's feat or fix. I'd argue this is the correct behavior and it was effectively working like this before 3.1.0, so it could be a fix.
Fixes #4545

@friday I'll add some minor changes to support more TTML colors (eg), and I'll also try to add some test with your permission.

@friday I'll add some minor changes to support more TTML colors (eg), and I'll also try to add some test with your permission.

Appreciated 👍

@friday I have already added the changes, in case you want to review.

@joeyparrish , I think the web_vtt_layout_integration.js tests should include some of these cases, what do you think?


Image description field

Hello,
I've just installed this plugin, which suits my needs perfectly.
However, would it be possible to add a description for each image? And that this description could be retrieved from the query.
Let me explain, I'm currently displaying images from a camera, and this camera takes pictures every X minutes and stores them in a database with several fields including the base64 encoded image and a "datetime" field.
Currently, using this plugin, I can display all my images, but I can't tell which image was taken at which time.
Thanks

@EvannG1, how and where would you like the time and description to be displayed?
We have had a similar request before and can implement it in the upcoming version.

Here's what our old panel looks like, written using JavaScript.
A slider that allows you to go from one image to another (by default to the right to be on the last image captured) and just below the date.
On your plugin, the date can be displayed below the image (centered?), or below the Previous/Next buttons.

@EvannG1 Ok, I will check what is the best option to display the description.


problem when running verbose

hi everyone,
root@Lab-ELK:/etc/elastalert# python3.8 -m elastalert.elastalert --verbose --rule sshtele.yaml
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/etc/elastalert/elastalert/elastalert.py", line 30, in <module>
    from .alerts import DebugAlerter
  File "/etc/elastalert/elastalert/alerts.py", line 26, in <module>
    from jira.client import JIRA
  File "/usr/local/lib/python3.8/dist-packages/jira/__init__.py", line 10, in <module>
    from jira.client import Comment  # noqa: E402
  File "/usr/local/lib/python3.8/dist-packages/jira/client.py", line 225
    validate=False, get_server_info=True, async=False, logging=True, max_retries=3, proxies=None,
                                          ^
SyntaxError: invalid syntax

what's wrong

pip uninstall jira
pip install 'jira>=2.0.0'

What happened after that? If it is resolved, please close it.

Is this repository a problem for users?

pip uninstall jira
pip install 'jira>=2.0.0'

Hi, I had the same problem as you.
please if you find a solution you can help me
thanks


Docker client requires write permission to $HOME

We run our weird custom docker builder with a non-writable $HOME out of coincidence more than anything, but this came up immediately after upgrading that node to 1.5.0:
Jan 29 18:34:32.286:    + docker inspect --format=. registry.chi.shopify.io:80/base:1.3.5
Jan 29 18:34:32.297:    time="2015-01-29T18:34:32Z" level="fatal" msg="mkdir /u/apps/locutus/.docker: permission denied"

It looks like the client initialization tries to force ${HOME}/.docker into a state it likes, bailing if it can't. This doesn't seem necessary for most operations I'd be requesting.

ping @dmcgowan
On Thu, Jan 29, 2015 at 10:51 AM, Burke Libbey<EMAIL_ADDRESS>wrote:

We run our weird custom docker builder with a non-writable $HOME out of
coincidence more than anything, but this came up immediately after
upgrading that node to 1.5.0:
Jan 29 18:34:32.286:    + docker inspect --format=. registry.chi.shopify.io:80/base:1.3.5
Jan 29 18:34:32.297:    time="2015-01-29T18:34:32Z" level="fatal" msg="mkdir /u/apps/locutus/.docker: permission denied"
It looks like the client initialization tries to force ${HOME}/.docker
into a state it likes, bailing if it can't. This doesn't seem necessary for
most operations I'd be requesting.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/10442.


Large File Save Progress Indicator

When I go to save a large file (wordpress multi-site sqldump) There is no progress indicator. I am sitting here for over an hour before I realized that the save crashed silently.

Can you explain some more about "the save crashed silently", that seems like the real issue here?


Docs: Add an "Overview" router page

I might just copy some styling from Stripe and do this in Plasmic.

Using Plasmic for router UI here
https://github.com/opensource-observer/oso/pull/1012

https://github.com/opensource-observer/oso/pull/1018

Can definitely use improvement, but got a first draft in


Bug during submission of a workflow if go back is clicked

I added a section to a workflow, clicked submit on the section and then on the workflow and saw a summary page as expected.
I clicked the "Go Back" button and did nothing, but clicked submit on the workflow again. The summary page now erroneously showed two entries for the section that I had previously saved.
When I clicked "Go Back" again, two identical sections are now erroneously present in my protocol.
Related, when I started a new workflow to do some testing around this bug, clikc save on a section and the workflow, the summary shows sections from the first workflow, but no sections from the second workflow.
There seem to be some issues with saving a workflow and also accessing multiple workflows.

Awaiting update from EBI containing fix for this issue


babel has error

// Rollup plugins.
import babel from 'rollup-plugin-babel'
import cjs from 'rollup-plugin-commonjs'
import globals from 'rollup-plugin-node-globals'
import replace from 'rollup-plugin-replace'
import resolve from 'rollup-plugin-node-resolve'
import uglify from 'rollup-plugin-uglify'

export default {
  dest: 'build/ReactCdPlayer.js',
  entry: 'src/containers/ReactCdPlayer.js',
  format: 'iife',
  plugins: [
    babel({
      babelrc: false,
      exclude: 'node_modules/**',
      presets: [ [ 'es2015', { modules: false } ], 'stage-0', 'react' ],
      plugins: [ 'external-helpers' ]
    }),
    cjs({
      exclude: 'node_modules/process-es6/**',
      include: [
        'node_modules/fbjs/**',
        'node_modules/object-assign/**',
        'node_modules/react/**',
        'node_modules/react-dom/**',
        'node_modules/prop-types/**'
      ]
    }),
    globals(),
    replace({ 'process.env.NODE_ENV': JSON.stringify('production') }),
    resolve({
      browser: true,
      main: true
    }),
    uglify()
  ],
  sourceMap: true
}

{
  "name": "react-cd-player",
  "homepage": "http://zhangwei900808.github.io/react-cd-player",
  "version": "0.3.0",
  "main": "index.js",
  "dependencies": {
    "classnames": "^2.2.5",
    "material-ui": "next",
    "material-ui-icons": "^1.0.0-alpha.19",
    "node-sass-chokidar": "^0.0.3",
    "npm-run-all": "^4.0.2",
    "react": "^15.6.1",
    "react-dom": "^15.6.1"
  },
  "devDependencies": {
    "babel-preset-es2015": "^6.24.1",
    "babel-preset-es2015-rollup": "^3.0.0",
    "babel-preset-latest": "^6.24.1",
    "babel-preset-react": "^6.24.1",
    "babel-preset-stage-0": "^6.24.1",
    "babel-preset-stage-1": "^6.24.1",
    "gh-pages": "^1.0.0",
    "react-scripts": "1.0.11",
    "rollup": "^0.45.2",
    "rollup-plugin-babel": "^3.0.1",
    "rollup-plugin-commonjs": "^8.1.0",
    "rollup-plugin-node-globals": "^1.1.0",
    "rollup-plugin-node-resolve": "^3.0.0",
    "rollup-plugin-replace": "^1.1.1",
    "rollup-plugin-uglify": "^2.0.1"
  },
  "scripts": {
    "build-css": "node-sass-chokidar src/assets/css/ -o src/assets/css/",
    "watch-css": "npm run build-css && node-sass-chokidar src/assets/css/ -o src/assets/css/ --watch --recursive",
    "start-js": "react-scripts start",
    "start": "npm-run-all -p watch-css start-js",
    "build": "npm run build-css && react-scripts build",
    "test": "react-scripts test --env=jsdom",
    "eject": "react-scripts eject",
    "predeploy": "npm run build",
    "deploy": "gh-pages -d build",
    "rollup-build": "rollup -c",
    "rollup-production-build": "NODE_ENV=production rollup -c"
  }
}

$ rollup -c
🚨   Error: 'default' is not exported by node_modules/material-ui-icons/FastRewind.js
https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module
src/containers/ReactCdPlayer.js (6:7)
4: import PropTypes from 'prop-types';
5: import FastForward from 'material-ui-icons/FastForward';
6: import FastRewind from 'material-ui-icons/FastRewind';
          ^
7: import { LinearProgress } from 'material-ui/Progress';

error Command failed with exit code 1.

Hi @zhangwei900808,
The issue you're currently facing is out of the scope of this small boilerplate project. I invite you to read https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module:

This error frequently occurs with CommonJS modules converted by rollup-plugin-commonjs, which makes a reasonable attempt to generate named exports from the CommonJS code but won't always succeed, because the freewheeling nature of CommonJS is at odds with the rigorous approach we benefit from in JavaScript modules. It can be solved by using the namedExports option, which allows you to manually fill in the information gaps.

Here comes the related fix https://github.com/rollup/rollup-plugin-commonjs#custom-named-exports.

Thank you very match^_^


Add auth feature to connetor and lvm driver

What this PR does / why we need it:
Add auth feature to connetor and lvm driver, and this just for chap incomeuser.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage decreased (-0.05%) to 43.757% when pulling 384f3eb1bd5a638aec5d339162869a167919e7e7 on xxwjj:iscsi_chap into a348294d0af09197e2add522a036a9f9b33b389d on opensds:development.


Correctly display tooltips for buttons in Tree when they overlap cell content

Closes #79629
I copied the 3.x behavior for the SceneTree: once the width becomes too small, the overflowing text of the node's name is hidden.

The way I found to do this is by creating a second column in the TreeItem and adding the buttons to this column to make sure that there is no overlap. I also set the clip property of the first column to false in order to hide the overflow

That doesn't really solve the problem and also introduces an ugly visual bug. If buttons that overlap text don't show their tooltips and instead the text content's tooltip is shown, then we should make sure that the elements which are visually on top contribute to the tooltip ahead of the elements which are below. See Tree::get_tooltip and check if it tests for buttons too late instead.

I found the bug. The get_tooltip function didn't take into account the fact that when we have an h_bar the button is not at the end of the TreeItem. I just had to take the offset between the size of the item and the size of the element being displayed. Now we have the following:

Okay, so after looking through the logic and the suggested change I think that the solution should be to add position to the Button struct and cache it during the drawing step. Then use it here to determine if we are over the button or not. The logic for determining the exact placement of each button is rather complex and it's counter-productive to try and replicate it or approximate its values by some other means.
The change needs to happen in Tree::draw_item around the code that draws the button texture, as that's where we have the exact position and size. We can even cache entire Rect2, as the size can also be limited by other factors and not come from the texture itself. Then the check in the get_tooltip method can just try to see if this rect includes our position.

I modified the implementation to use the cache as explained on the comment above. The result is the same as before (it also works for vertical-scrolling):

Thanks!


Always send first editions to publishing api as major updates

When republishing manuals for #778 after deploying #780 some of the republishes failed with a 500 error from publishing-api (see: https://errbit.integration.publishing.service.gov.uk/apps/552637940da1157bfa000399/problems/585908a76578636a1ecc0000).  Turns out publishing-api doesn't like it if the first edition of a content item has update_type of "minor".  This is on the publishing-api backlog to turn into a proper error (see: https://trello.com/c/3S4m5wAJ/251-throw-an-error-if-the-first-publish-does-not-use-a-major-update-type) but we need to handle it in manuals publisher.  Turns out that's painful because manuals-publisher doesn't really keep things in lock-step with what is in publisher.
There are more notes in the commits if you're interested.

Thanks @cbaines!


ansi_rgb 0.3.2-alpha

Hello! I have released ansi_rgb 0.3.2-alpha. Would you be willing to try it out and let me know what you think?
I think this will have minimal impact on your existing code. I haven't fully tested your code, but as you can see with this PR the changes are very minimal.
0.3.2 makes it much easier to use the .fg and .bg functions with custom color types that you can define privately in your own crate. So it should be easier to extend should the need arise.
0.3.2 also supports 3-, 4-, and 8- bit colors.
If you like the alpha then I can keep you posted as 0.3.2 goes through beta -> release.

@matthew-a-thomas Thanks!


Not able to make Vile Dust

Issue Description:
I can not find any recipe in the 1.12 version to make vile dust.
What happens:
not able to progress in the mod with out the ability to get vile dust.
Because there is no recipes to make vile dust.
What you expected to happen:
pop soul sand into a furnace and get vile dust...
Steps to reproduce:

load game with mod installed
notice there is no recipe to craft vile dust
...

Affected Versions (Do not use "latest"):
forge-1.12-<IP_ADDRESS>7
SoulShards-TOW-1.12-2.7.2-51
Minecraft 1.12

Hm, I guess I never moved that to something that actually got called after the registry rewrite.


Discover a way to deploy DWO on Kubernetes

We're going to release DevWorkspace Operator within RedHat Catalog which is not available on the Kubernetes clusters. So, we need to discover a way to deploy DWO on Kubernetes.
Note that we also have a plan to stop publishing Che operator on operatorhub.io, and Che operator should supply our way.
Possible solutions:

helm chart;
yaml files (what we have now). Then Che Operator on K8s, as it does now - will apply DWO resources;
https://kudo.dev/
...

Since Che is going to deprecate helm charts https://github.com/eclipse/che/issues/20178, it's not an option for us as well.
The plan is to support only yaml files, you can apply directly, or with make install, or Che operator will manage it if you deploy on K8s and enable DevWorkspace support.
Or OLM for OpenShift


Get-WmiObject not available

Can't run this command on PS 7
PowerShell 7.1.3
Copyright (c) Microsoft Corporation.

https://aka.ms/powershell
Type 'help' to get help.

PS C:\> Get-WmiObject
Get-WmiObject: The term 'Get-WmiObject' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

This is expected (and documented). The WMI cmdlets have been replaced with the CIM cmdlets since v3 came out and it was in v6 that the WMI ones were ultimately removed. To be compatible with both you should use the CIM cmdlets. In this case it's Get-CimInstance -Class Name MyClass.

So no bug working as intended. Thanks for the info wasn't aware.


Offset path on shape

I have animated shape with offset path, but after rendering it in .json, it converts into simple line.
Json aep

offset path is not yet supported
I'm working on it but it has a big performance impact

Got it, thanks.


Bug: arr error: not sending info

Describe the Bug
when adding to sonarr, I have an error:
| Sonarr Error: Invalid Series Type: '[722:The Great]' Options: ['standard', 'daily', 'anime'] |
TV_missing.yml is filled correctly
when adding to radarr
no movies are added
I don't know if there is a link but in the meta.log the path seems wrong.
Missing Path: //config/Films_missing.yml
but it is filled correctly
Relevant Collection Config
Plex Meta Manager Info
Version: 1.13.3
Link to logs (required)
https://gist.github.com/Nexius2/0c7e09e3d9f977b57bf4fc164139346e

the title isnt very descriptive but #500 is the same issue and its been fixed in develop


Add search event tracking

Track:

TitleNumber: search for title number
ZeroResults: got to search results page with no results
Page{{number}}: search results page {{number}}

@lgonczar ok I see what you mean now, is this better?

Yes, more or less something like that :+1:
Looks good to me.


Problem with merging AdaLoRA adapters

System Info
peft @ git+https://github.com/huggingface/peft@c8974c5880b28a913e35f050e82402e34d181c63
accelerate==0.29.2
transformers==4.39.3

Ubuntu 22.04.4 LTS
Python 3.11.9
Who can help?
@BenjaminBossan @pacman100
Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder
[X] My own task or dataset (give details below)

Reproduction
I am trying to merge two AdaLoRA checkpoints of the same model and have written this script for it:
from peft import PeftConfig, PeftModel
from transformers import AutoModelForSeq2SeqLM

model_id = "Salesforce/codet5-small"
config = PeftConfig.from_pretrained("checkpoint-6621")
model = AutoModelForSeq2SeqLM.from_pretrained(config.base_model_name_or_path, device_map="auto")

model = PeftModel.from_pretrained(model, "checkpoint-6621", adapter_name="checkpoint-6621")
model.load_adapter("checkpoint-13242", adapter_name="checkpoint-13242")

adapters = ["checkpoint-6621", "checkpoint-13242"]
weights = [1.0, 1.0]
adapter_name = "merged"
density = 0.2
model.add_weighted_adapter(adapters, weights, adapter_name, combination_type="ties", density=density)

model.set_adapter("merged")
print(model.active_adapters)

When I run the script, I get this error:
Traceback (most recent call last):
  File "merge.py", line 15, in <module>
    model.add_weighted_adapter(adapters, weights, adapter_name, combination_type="ties", density=density)
  File ".venv/lib/python3.11/site-packages/peft/tuners/lora/model.py", line 579, in add_weighted_adapter
    combination_type, new_rank, new_target_modules = self._check_add_weighted_adapter(
                                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".venv/lib/python3.11/site-packages/peft/tuners/lora/model.py", line 516, in _check_add_weighted_adapter
    raise TypeError(f"Invalid type {target_module_types[0]} found in target_modules")
TypeError: Invalid type <class 'list'> found in target_modules

The given error is similar to this issue: https://github.com/huggingface/peft/issues/1045
Expected behavior
It is expected that the adapters merge successfully. The same script works for LoRA adapters, resulting in a merged adapter.

The error you encounter is indeed a bug, which can be fixed by adding the lines:
model.peft_config["checkpoint-6621"].target_modules = set(model.peft_config["checkpoint-6621"].target_modules)
model.peft_config["checkpoint-13242"].target_modules = set(model.peft_config["checkpoint-13242"].target_modules)

after model.load_adapter("checkpoint-13242", adapter_name="checkpoint-13242").
However, even with this workaround, this won't work because add_weighted_adapter is only supported for LoRA, not AdaLoRA.


Vogel612: Blacklist /asha24\b

Vogel612 requests the blacklist of the keyword \/asha24\\b. See the Metasmoke search here and the Stack Exchange search here.

\/asha24\\b has been seen in 0 true positives, 0 false positives, and 0 NAAs.

overescaped :/

Already blacklisted. This is a duplicate, close please.

Coverage remained the same at 65.058% when pulling 3a9e3469951128bcea0524e721ef4eb7c8cbb3a8 on auto-blacklist-1524746205.9804928 into b5f11a6f02e6d42e8a03d2bbeff606e06c6f30e8 on master.


fix: change pip2 to pip3

docker-py is deprecated and should not be use. A workaround is to use python 3 to install docker-compose instead of python 2.7. See https://github.com/docker/compose/issues/6339
This fix works for me to install docker-compose 1.24 on a debian stretch. Not sure it's good for everyone...

Hi, thanks for the note. I added some variables which should help you and will be backward compatible:
# dependencies e.g. python3-pip
docker_compose_packages:
  - python-pip3
# pip executable e.g. pip3
docker_compose_pip_executable: pip3


Assets still accessible with http_credentials enabled

The API endpoints are secured when the --http_credentials flag is enabled when starting Marathon, but other endpoints like the one that serves assets is not.
As seen in the screenshot, I have not yet given the name + password but my browser was able to download assets. Only when the JavaScript attempted to request /v2/apps did my browser ask me for credentials:

Old issue. Closing.

@ssorallen has earned the necromancer GitHub achievement

👻👻👻👻👻👻👻👻

@ConnorDoyle Something I appreciated at Facebook: an issue reaper that commented on issues after 6 months of inactivity and closed them automatically after another 6 months of inactivity. An issue that hasn't been touched in 1 year+ probably isn't useful.