Reenable dotty-compiler-plugin test

https://github.com/lampepfl/dotty/issues/5671 is fixed now.

Travis CI says:
[info] [info] Compiling 1 Scala source to /tmp/sbt_e5f965f1/plugin/target/scala-0.14/classes ...
[info] [error] /tmp/sbt_e5f965f1/plugin/DivideZero.scala:2:14: value tools is not a member of dotty
[info] [error] import dotty.tools.dotc._
[info] [error]              ^
[info] [error] /tmp/sbt_e5f965f1/plugin/DivideZero.scala:23:30: <error value tools is not a member of dotty>#dotc.List does not take type parameters
[info] [error]   override def init(options: List[String]): List[PluginPhase] = this :: Nil
[info] [error]                              ^
[info] [error] /tmp/sbt_e5f965f1/plugin/DivideZero.scala:23:45: <error value tools is not a member of dotty>#dotc.List does not take type parameters
[info] [error]   override def init(options: List[String]): List[PluginPhase] = this :: Nil
[info] [error]                                             ^
[info] [error] /tmp/sbt_e5f965f1/plugin/DivideZero.scala:33:63: Stable identifier required, but 31m35m<error value tools is not a member of dotty>31m0m#31m35mdotc31m0m.Nil found
[info] [error]     case tpd.Apply(fun, tpd.Literal(Constants.Constant(v)) :: Nil) if isNumericDivide(fun.symbol) && v == 0 =>
[info] [error]                                                               ^
[info] [error] /tmp/sbt_e5f965f1/plugin/DivideZero.scala:33:91: value symbol is not a member of Any
[info] [error]     case tpd.Apply(fun, tpd.Literal(Constants.Constant(v)) :: Nil) if isNumericDivide(fun.symbol) && v == 0 =>
[info] [error]                                                                                ^
[info] [error] 5 errors found
[info] [error] (plugin / Compile / compileIncremental) Compilation failed

We're now getting ordering conflict for the phase. Looks like a progress.
[info] [info] Compiling 1 Scala source to /tmp/sbt_762bcd99/target/scala-0.14/classes ...
[info] [info] exception occurred while compiling /tmp/sbt_762bcd99/src/main/scala/hello/Hello.scala
[info] java.lang.Exception: Ordering conflict for phase divideZero while compiling /tmp/sbt_762bcd99/src/main/scala/hello/Hello.scala
[info] [error] ## Exception when compiling 1 sources to /tmp/sbt_762bcd99/target/scala-0.14/classes
[info] [error] java.lang.Exception: Ordering conflict for phase divideZero
[info] [error] dotty.tools.dotc.plugins.Plugins$.schedule$$anonfun$11$$anonfun$2(Plugins.scala:267)

[info] java.lang.Exception: Ordering conflict for phase divideZero while compiling /tmp/sbt_762bcd99/src/main/scala/hello/Hello.scala

@liufengyun Do you know why this might be happening ?

@smarter I think the problem is with the phase LinkAll, which has been removed.
Remove the following line will work:
https://github.com/sbt/sbt/blob/develop/sbt/src/sbt-test/compiler-project/dotty-compiler-plugin/plugin/DivideZero.scala#L22
I wondered why the plugin code compiled.

This PR already updates this line: https://github.com/sbt/sbt/pull/4641/files#diff-ddb6ec378e5551c95e253f5a1a088889R23
It compiled previously because the test was using an old version of dotty.


Don't create ON_FALSE control flow edges for infinite DO loops

This PR changes the handling of infinite do-while loops in the control flow analysis to match other infinite loops.
Currently, for FOR and WHILE loops, no ON_FALSE edges are created from the loop condition if it is a true boolean literal (i.e. infinite loops). This is done by first retrieveing the loop's condition and then checking that it is not a true literal.
For WHILE loops, handleWhile does:
Node cond = node.getFirstChild();
if (!cond.isTrue()) {
    // Create ON_FALSE branch
}

handleDo also contains this exact code, but because the first child of a DO loop is its body, not its condition, the value of cond is actually a statement (the body) and so cond.isTrue() is always false and therefore an ON_FALSE branch is always created, even if the condition is a true literal.
This PR corrects handleDo's behaviour, bringing it in line with the other methods.

I've reviewed the change and convinced myself it is correct.
I'll import it and run internal tests to see if I can land it.
Thanks for finding and fixing this!


Actionsheet fix

Does this fix the extra space at the bottom of the Action sheet? - Excess bottom space in the ActionSheet #2624


.git causes crash

Reproducible on
OSX 10.11.4 - Atom 1.7.2
Ubuntu 14.04 - Atom 1.7.2
.git-folder of the Repository https://github.com/ethereum/mist causes a crash - also in safe-mode.

Windows 10 too...

Confirmed using Atom v1.7.3 on Mac OS X 10.11.4.
Crash log: https://gist.github.com/lee-dohm/572d6f40525a0998d2a2b4665e7bc67b

This also happens using the electron based http://gitkraken.com.

This problem seems to be resolved in atom 1.10.2.


Don't expose sensitive data from Faraday::Response#describe (closes #1155)

@k1T4eR Rebase pls

@yivo Done.


NetworkG api is broken

From Anita Graser testing 1.9.1 with QGIS:
Traceback (most recent call last):
File "", line 1, in
File "C:\OSGEO4~1\apps\Python27\lib\site-packages\pysal-1.9.1-py2.7.egg\pysal\network\network.py", line 898, in NetworkG
upperbound=upperbound)
TypeError: init() got an unexpected keyword argument 'distribution'

Pretty sure the error is here. We probably want to pass distributions, not distirbutions :smiley:

If I can get some code to test, that'd be amazing. But, just doing %s/distirbutions/distributions/gc in network/analysis.py returns no errors on specifying/computing a NetworkG while specifying distribution:
In [1]: import pysal; reload(pysal)

In [3]: ntw = pysal.Network(pysal.examples.get_path('geodanet/streets.shp'))

In [4]: ntw.snapobservations(pysal.examples.get_path('geodanet/crimes.shp'), 'crimes', attribute=True)

In [5]: G = pysal.NetworkG(ntw, ntw.pointpatterns['crimes'], distribution='uniform')

In [6]: G.distribution
Out[6]: 'uniform'

This swap is in pull #631

We should add the "nukular" option somewhere too. ;-)
On Thu, Jun 18, 2015 at 1:39 PM Levi John Wolf<EMAIL_ADDRESS>wrote:

If I can get some code to test, that'd be amazing. But, just doing
%s/distirbutions/distributions/gc in network/analysis.py returns no
errors on specifying/computing a NetworkG while specifying distribution:
In [1]: import pysal; reload(pysal)
In [3]: ntw = pysal.Network(pysal.examples.get_path('geodanet/streets.shp'))
In [4]: ntw.snapobservations(pysal.examples.get_path('geodanet/crimes.shp'), 'crimes', attribute=True)
In [5]: G = pysal.NetworkG(ntw, ntw.pointpatterns['crimes'], distribution='uniform')
In [6]: G.distribution
Out[6]: 'uniform'
This swap is in pull #631 https://github.com/pysal/pysal/pull/631
—
Reply to this email directly or view it on GitHub
https://github.com/pysal/pysal/issues/630#issuecomment-113282827.

As an aside, it'd be cool if we could just aspell -r pysal/*...


show blind submissions if exists otherwise shows original notes

@pmandler this PR shows the original notes in the Program Chairs console before the submission deadline.

it is what we usually do to allow them to see the blinded notes.

it is not admin permissions in all openreview.

How is it not admin permissions in all openreview? What am I missing?

basically I'm adding the pc group as a member of the conference group.
for example:
id: 'ICLR.cc/2019/Conference',
members: 'ICLR.cc/2019/Conference/Program_Chairs'

the conference group is like the 'admin' group. We create invitations and groups based on this id.

If it was adding it to get_conference_id() I'd understand, but adding it to ~Super_User1 doesn't make sense to me.

I'm not adding anything to the super user group, where do you see that?

Line 343 of builder.py

I knew I was missing something.  Thanks.


Broken link in getting started page

Steps:

Go to https://docs.litmuschaos.io/docs/getstarted/
From the right navigation bar, go to the example section
Click on the tutorial link in the example section
Ends up in a 404

The link should be https://docs.litmuschaos.io/docs/example/

Fixed in 0.8 release


Create IPP Everywhere printer through IPP

Copied from Apple CUPS #5919
Update cupsd to support a "ppd-name" of "everywhere" to generate the PPD in the background:

[x] Update cupsd to use create_local_bg_thread in add_printer when "ppd-name" is "everywhere"
[x] Update lpadmin to not create PPD locally
[x] Update admin.cgi to not create PPD locally
[ ] Update _ppdCreateFromIPP to copy icon files to /Library/Printers/Icons for macOS
[ ] Update _ppdCreateFromIPP to copy the largest icon file to ServerRoot/icon/PRINTERNAME.png
[ ] Update cupsd to expose printer icons from ServerRoot/icon

P.S. commit 68506ac broke test suite, we will need to investigate further.

This error appeared after the mentioned commit:
E [13/Jul/2021:12:30:21.337402 +0200] [cups-driverd] Unable to open \"/tmp/cups-zdohnal/share/model/everywhere\" - No such    file or directory

Error log from the test run - the error happens during serving CUPS-Get-PPD request which hasn't been migrated to support everywhere model.
Currently I'm not sure whether we want to migrate this operation for everywhere or not, and if we want, how to get the ppd, because cups-driverd works with files, and _ppdCreateFromIPP() returns only char array for now.

@zdohnal Not sure why I didn't catch this when I did testing, but fixed now:
[master 3a37c0de2] Fix regression caused by change for Issue #201.

@michaelrsweet verified, tests are passing now. Thx!


uu

Coverage decreased (-0.1%) to 51.978% when pulling 51a15092a80015caa61a9859e61fc4051d473971 on StefanBranch into 3408139fe6c8e688dd9396f7cce5a00d2ef23a1b on master.


Selecting setting + suggestions can corrupt value

Open settings
Add files.eol
Select the value, with the cursor left to the right of the selection, trigger suggestions, accept the suggestion

The result is missing characters

If the cursor goes to the left side of the selection, it replaces the value as expected.

Also reproes in 1.15.2

actually a duplicate of  #31199


(@prisma/client): Generate mapping of Models to allow mapped types

Problem
Because Prisma does not provide a solution for caching yet, I made use of middlewares like this one. I added the option to provide default values and only cache a specific set of selected properties, however I cannot provide autocompletion for these keys because Prisma.ModelName is a union type of strings
type CacheMiddlewareOptions = {
  model: Prisma.ModelName
  action: Prisma.PrismaAction
  keys?: string[]
  defaultValues?: Record<string, unknown>
  ttlInSeconds: number
}

Suggested solution
It would be nice if each model would be mapped to their select options so we can do:
export type ModelMap = {
  SomeModel: {
    select: // select type here
  }
}

export type ModelName = keyof ModelMap

type CacheMiddlewareOptions<T extends Prisma.ModelName> = {
  keys: Array<keyof Prisma.ModelMap[T]["select"]>
}

Closing as a duplicate of https://github.com/prisma/prisma/issues/5058.
I added link to your comment there.


Help wanted Using react-intl

Hi all,
I'm following this tutorial https://medium.freecodecamp.com/internationalization-in-react-7264738274a0#.pgfd03878 to use internationalization in my app.
Apparently the integreation is done by modifying the .babelrc file but to do so I have to eject, and I prefer not to.
Is there any way to use React-Intl without ejecting create-react-app?

You only need the Babel plugin to generate the translate files which you can do as part of your production build without ejecting although it's a little messy. I'll post how later.

You can just install the required dependencies, add your own .babelrc, and add your "script": "translation" to your package.json.
E.g.:
npm install --save react-intl
npm install --save-dev babel-cli babel-preset-react-app babel-plugin-react-intl
add to the root folder:
{
  "presets": ["react-app"],
  "plugins": [
    [
      "react-intl", {
        "messagesDir": "translations/messages",
        "enforceDescriptions": false
      }
    ]
  ]
}

then add to your package.json's '"scripts"` this line:
"translate": "NODE_ENV=production babel ./src >/dev/null"

You should be able to run both npm start/npm run buildfor running/building the project, andnpm run translate` for generating the translations.

@EnoahNetzach thanks for the answer!
How does : "NODE_ENV=production babel ./src >/dev/null" generate the translation? How does it work exactly?
Thank you 🙏

hi,
Internationlization is a pretty important feature for mobile apps these days. Could we have react-intl inbuilt in the create-react flow?
Regards
Arsh

@firaskrichi it works as intended by react-intl, by using babel's AST to catch translations and replace them (I suggest you to look it up, it's really interesting, if you like to know how exactly it - and in general babel plugins - works!).
The > /dev/null part is there simply to discharge the babel parsed output.
Moreover what I suggested is simply what the official doc tells to do ;)
@arshmakker being that it's easily pluggable, not everybody wants to use i18n, and the specific tool (format.js) is not that reliable if you want to do substantial i18n (@see), I'm not sure at all it would be possible.

This is a really useful discussion and the information provided by @EnoahNetzach has really helped get react-intl into my create-react-app application. I think it would be exceptionally valuable to have i18n working out-of-the-box with create-react-app (or at least some good documentation explaining how to use it).
The main thing that I don't quite understand is how this works with a production build. I've followed the various scripts to combine individual component message files into a single JSON file for the default locale, I've then added a different locale (with translated messages) and updated the IntlProvider locale to be the new locale.
When you use npm run build the new locale is lost (probably because the translate scripts provided by the blogs combine the component messages into locale files in the build directory).
If anyone can shed any light on the correct approach to combining the information provided so far in a production story I'd be very grateful.

@draperd, as a non English native speaker, I find that unfortunately no i18n instrument in js is a "catch all" solution.
I've used a couple and always ended up tweaking them to my needs (e.g. subject/direct object gender and numerosity, probably @gaearon could say something about Russian desinences, I don't know if there is someone from Iceland here, and so on and so forth..).
Whichever translation package we could add, it's going to solve only a limited set of cases and needs, so I'm against adding them in altogether, specially when the integration is not "that" painful.

@EnoahNetzach Thanks for your response... I wasn't suggesting that there is a one-size fits all solution, but it would seem that React-Intl is a pretty good fit. I can understand the rationale of not wanting to tie "create-react-app" to a single solution, however a solution of some kind is almost going to be required. Any serious application is going to want to support i18n so I'm simply suggesting that either something is provided by default, or that documentation is provided to assist people in achieving a solution.
The information you have previously provided has been invaluable but only as far as the development experience goes. My point is that when I'm ready to ship my finished application I need translation to work in production for multiple locales and I've not been able to find that information anywhere so far.
I'll continue to experiment and hopefully find a working solution which I'll then write up, I was just hoping that someone may have already encountered and solved this problem using react-intl.

Just to follow up my previous comment... what I actually need is a per component solution which I've now managed to achieve using the higher order component pattern. I will write this solution up as soon as I can for anyone that comes across this thread as it might be useful.

@draperd good!

As promised... hopefully this might be of use to some people<EMAIL_ADDRESS>

For everyone that the suggestion from @EnoahNetzach  is not working out of the box I had to put following in my package.json, probably as I'm working on windows
"translate": "cross-env NODE_ENV=production ./node_modules/.bin/babel ./src > NUL",

Yep, env var definition and the output indirection to /dev/null work only in Linux/Mac.
cross-env solves the first part, for the second sincerely I have no idea how to accomplish that on Windows..

If you don't want to have to npm install --save-dev babel-cli or create the babel.rc file, you can use the babel API:
// scripts/translate.js

process.env.NODE_ENV = 'development';

const fs = require('fs');
const path = require('path');
const globSync = require('glob').sync;
const mkdirpSync = require('mkdirp').sync;
const transformFileSync = require('babel-core').transformFileSync;

const paths = require('../config/paths');

//////////////////////////////
// Add to `/config/paths.js`:
//  appBuildMessages: .. where to output the messages
//////////////////////////////

globSync(path.join(paths.appSrc, '/**/*.js'))
    .map((filename) => {
        const result = transformFileSync(filename, {
            plugins: ['react-intl']
        });
        const messages = result.metadata["react-intl"]["messages"];
        if (messages.length > 0) {
            const relFn = path.relative(paths.appSrc, filename);
            const outFilename = path.join(paths.appBuildMessages, relFn);
            mkdirpSync(path.dirname(outFilename));
            const outFd = fs.openSync(outFilename, 'w');
            fs.write(outFd, JSON.stringify(messages));
        }
    });

...
  "scripts": {
    ...,
    "translate": "node scripts/translate.js",
   }

I tried the .bablerc solution, but it does not work unless and untill i do an eject, is that just me?

@EnoahNetzach I tried your solution of using a translate script, but I wound up with a "missing script: translate" error. I don't know if this means your solution is no longer valid with the latest version of CRA, or if I have missed a detail.
In my package.json file, I have:
  "scripts": {
    "translate": "NODE_ENV=production babel ./src >/dev/null"
  },

...and am running react 15.5.4, react-scripts 0.9.5, babel-cli 6.24.1, babel-plugin-react-intl 2.3.1, and babel-preset-react-app 2.2.0. My .babelrc sits at the root of my project folder, and is verbatim what you posted above:
{
  "presets": ["react-app"],
  "plugins": [
    [
      "react-intl", {
       "messagesDir": "translations/messages",
       "enforceDescriptions": false
      }
    ]
  ]
}

...and then here's the CLI:

Have I missed something obvious?

@cwalv question about your solution: it appears that you're outputting a separate .json file for each .js file in which you find translatable content (if I'm reading your code right). Is there a reason for doing that as opposed to putting all translatable content into a single .json file? I'm just getting into i18n, so I don't know if you're following an accepted practice, or if that's just a personal preference on your part.
Thanks!

@bmueller-sykes, you're right, I only put in half the script.  I adapted it from https://github.com/yahoo/react-intl/blob/b82a899a59c762fcc590e1fa824b87da6ad409d9/examples/translations/scripts/translate.js, which has a second pass that aggregates separate files into one.  The rest of the script looks like:
// Aggregates the default messages that were extracted from the example app's
// React components via the React Intl Babel plugin. An error will be thrown if
// there are messages in different components that use the same `id`. The result
// is a flat collection of `id: message` pairs for the app's default locale.
const messagesGlob = path.join(paths.appBuildMessages, '**', '*.json');
let defaultMessages = globSync(messagesGlob)
    .map((filename) => fs.readFileSync(filename, 'utf8'))
    .map((file) => JSON.parse(file))
    .reduce((collection, descriptors) => {
        descriptors.forEach(({ id, defaultMessage }) => {
            if (collection.hasOwnProperty(id)) {
                throw new Error(`Duplicate message id: ${id}`);
            }
            collection[id] = defaultMessage;
        });
        return collection;
    }, {});

// For the purpose of this example a fake locale: `en-UPPER` is created and
// the app's default messages are "translated" into this new "locale" by simply
// UPPERCASING all of the message text. For a real translation this would mean some
// offline process to get the app's messages translated by machine or
// professional translators.
let uppercaseTranslator = new Translator((text) => text.toUpperCase());
let uppercaseMessages = Object.keys(defaultMessages)
    .map((id) => [id, defaultMessages[id]])
    .reduce((collection, [id, defaultMessage]) => {
        collection[id] = uppercaseTranslator.translate(defaultMessage);
        return collection;
    }, {});

I released a workaround solution to NPM react-intl-cra, so that you can simply extract messages of Creact React App from the command line.
$ npm i react-intl-cra --save-dev
$ react-intl-cra './src/**/*.js' './messages.json'

DEMO

Thanks @evenchange4 , that npm package really helped my case.
I was struggling with intl after shifting to react-scripts.
REgards
ARsh

This looks really great! Perhaps we should put it into docs?

FWIW, I wound up writing a script for this as well, based on the work from above. It runs at the command line, and trolls through everything in the src folder and outputs all translatable instances into a single JSON file (currently hardwired to en-for-translation.json). I don't know if there's a reason to split up translations across multiple files, and at least for my purposes, I didn't need to. If this is useful to people, please use it!
// This file needs to be at the root of the React project
process.env.NODE_ENV = 'development';

const path = require('path');
const globSync = require('glob').sync;
const fs = require('fs');
const mkdirpSync = require('mkdirp').sync;
const babel = require('babel-core');
const jsonFormat = require('json-format');
const plugin = require('babel-plugin-react-intl');
const _ = require('lodash');
const colors = require('colors');

const paths = {
  root: `${process.env.PWD}/src/`,
  output: `${process.env.PWD}/localization/`
}

const all = []
var totalNodes = 0;

globSync(path.join(paths.root, '**/*.js')).map ( (filename) => {
  const result = babel.transformFileSync(filename, {
    presets: ['react'],
    plugins: ['transform-object-rest-spread','transform-class-properties',plugin.default],
  });
  const meta = result.metadata['react-intl'];

  if (meta.messages.length > 0) {
    meta.messages.map ( (message) => {
      const existing = _.find( all, (m) => m.id === message.id );
      if (existing) {
        console.log(`*** ERROR: Duplicate ID found: ${message.id} ***`.bold.red)
        console.log(`Look in file ${filename} to find duplicate`.red)
      } else {
        totalNodes++
        all.push(message);
      }
      return true;
    })
  }
  return true;
})

const outputFileName = `${paths.output}en-for-translation.json`;
mkdirpSync(path.dirname(outputFileName));
fs.writeFile(outputFileName, jsonFormat(all, {type: 'tab', size: 1}), (error) => {
  if (error) {
    console.log('*** ERROR WRITING FILE'.bold.red.underline);
    console.log('ERROR', error);
  } else {
    console.log(`*** DONE`.bold.green);
    console.log(`${totalNodes} messages ready for translation`.green);
    console.log(`File written ${outputFileName}`.green);
  }
});

@arshmakker, thank you for trying. If there is any problem please feel free to report it to here currently. Maybe I will move it out to a stand-alone repository later.
@gaearon I am happy to send a PR for I18n document improvement. 😀

Thank you @bmueller-sykes. Works like a charm.

To sum up: it would be nice to have standalone CLI (or API to write one) to extract all translation keys to JSON file. react-intl-cra is nice try, except I would prefer standalone repository for this package (it seems to be embedded into mcs-lite).
See no reason why React Intl can provide official solution for this issue. Can we have this as standalone CLI instead of Babel plugin?

@stereobooster Thanks for suggestions! I have received some issues in the monorepo, so I moved it to standalone repository https://github.com/evenchange4/react-intl-cra for better issue tracking (after v0.2.12). 🙌

May I know what does react-intl-cra and other examples here generate in
[
  {
    "id": "...",
    "defaultMessage": "..."
  }
]

format?
If you do import translations from './i18n/messages.json'; and <IntlProvider locale="ka" messages={translatiosn}> it won't work. It accepts key value paris like
{
  "...": "..."
}

Thanks.

Did anybody made a workaround ? I dont' get why react-intl-cra './src/**/*.js' -o messages.json generated me this file :
[
  {
    "id": "test.action.resetViews",
    "defaultMessage": "Reset views",
    "filepath": "./src/messages.js"
  }
]

While I expect
{
  "test.action.resetViews": "Reset views"
}

I want to use react-intl. As @dachinat pointed oput, we need key:value pair.
How can I do without ejecting?

Personnaly, I use the internal script extract-intl from react-boilerplate.
No need of babel-plugin-react-intl or react-intl-cra, just some adaptations according the project, like changing paths and install needed modules (mainly babel-cli) and voila. :D
"extract-intl": "babel-node --presets env,stage-0 -- ./internals/scripts/extract-intl.js"

I have made a tool you can be use with npx that work without any dependency  :
$ npx rollup-umd-scripts install fr en de ch vi --default=en

This will create in your package.json
+  "translation": {
+    "locale": "en",
+    "locales": [
+      "en",
+      "fr",
+      "de",
+      "ch",
+      "vi"
+    ]
  },

You can now extract what's in src directory with :
$ npx rollup-umd-scripts intl extract

You can add the following scripts to your package.json to keep the snippet for any project:
    "extract-intl": "NODE_ENV=production rollup-umd-scripts intl extract --target=src/i18n/translation",

--target is for changing the destination path for the generated json files.
You'r welcome

I'm following up on the questions asked by @dachinat and @kopax
From what I understand, there are two steps:

Extract all the strings that need to be translated
Generate a file that will store the translations for each language

In the gettext universe, the first step is the equivalent of generating the (single) .pot file while the second step is the generation of the (multiple) .po files.
The "messageId": "value" syntax would apply to the result of the second step, while babel-plugin-react-intl only does the first step.
I didn't think the second step would be so complicated to implement.
So far all I've found (outside of this discussion) is this software that provides a GUI to do it. I don't know what it is based on.
I was expecting the guys behind react-intl would provide a command such as translations-update -i build/**/*.json -o src/messages/fr.json.
Did I miss something here or is it really that complicated to have a translations file for each language?

Looking for a way to extract messages from react-intl components and manage locales effortlessly?
The following worked for me: https://github.com/yahoo/babel-plugin-react-intl/issues/108#issuecomment-464368426

Looking for a way to extract messages from components localized using react-intl and manage translations effortlessly?
The following worked for me: https://github.com/yahoo/babel-plugin-react-intl/issues/108#issuecomment-464368426

I was using the same tutorial and couldn't find a tutorial from start => finish without ejecting so wrote one here: https://medium.com/@jamieallen59/using-react-intl-with-create-react-app-6667ee3e19f3. Hope it helps.

@noliiva I have placed this react-boilerplate script in a program that you can run through npx and it work without any dependency :
$ npx rollup-umd-scripts install fr en de ch vi --default=en

This will create in your package.json
+  "translation": {
+    "locale": "en",
+    "locales": [
+      "en",
+      "fr",
+      "de",
+      "ch",
+      "vi"
+    ]
  },

You can now extract what's in src directory with :
$ NODE_ENV=production npx rollup-umd-scripts intl extract

You can add the following scripts to your package.json to keep the snippet for any project:
+  "extract-intl": "NODE_ENV=production npx rollup-umd-scripts intl extract --target=src/i18n/translation",

--target is for changing the destination path for the generated json files.
It's basically the same as evenchange4 except with the standard format for json and without any install.

Hey I get error saying ' You must use a intl declination to use this command!'

@SeunghunSunmoonLee sorry I did a change in that script, you must have in your package.json
+  "declinationId": "intl"

It wont break anymore, otherwise you can fix to an older version.

@jamieallen59 Your tutorial worked great for me. Thanks.

@jamieallen59
You use react-intl v2 version in your guide. Nowadays in v3 a lot of APIs have breaking changes

In my case I'm able to use react-intl with create-react-app by just including 2 simple scripts to extract and compile the translation files:
scripts/i18n-extract.sh:
#!/bin/bash
lang='en'

npm run i18n:formatjs:extract \
	-- 'src/**/*.ts*' --ignore 'src/**/*.d.ts*' \
	--out-file src/src-lang/"$lang".json \
	--id-interpolation-pattern '[sha512:contenthash:base64:6]'

* I define the extension as ts* because I'm using typescript, if you are using javascript just change it to js*
scripts/i18n-compile.sh:
#!/bin/bash
langs=( 'en' 'fr' 'ar' )

if [ "${1:-}" = 'default' ]; then
    langs=( 'en' )
fi

for lang in "${langs[@]}"; do
    src="src/src-lang/$lang.json"
    dest="src/lang/$lang.json"

    if [ ! -f "$src" ]; then
        echo "[warn] src file not found ($src)"
    else
	    npm run i18n:formatjs:compile -- "$src" --ast --out-file "$dest"
    fi
done

* In this case I'm considering the languages en, fr and ar (en being the default), you can change it to your use case.
And in package.json:
"scripts": {
    "start": "HTTPS=true react-scripts start",
    "build": "react-scripts build",
    "test": "react-scripts test",
    "eject": "react-scripts eject",
    "i18n:dev": "echo '{}' > src/lang/en.json",
    "i18n:extract": "bash ./scripts/i18n-extract.sh",
    "i18n:compile": "bash ./scripts/i18n-compile.sh",
    "i18n:formatjs:extract": "formatjs extract",
    "i18n:formatjs:compile": "formatjs compile"
}

I only need bash to be installed and everything works fine so far. I can run npm run i18n:extract in a CI environment to generate the file with the translations in the default language, and in another CI process, when deploying to staging/production I can retrieve the translation files (from wherever they are), put them at src/src-lang/ and call npm run i18n:compile to generate the final (compiled) translation files at src/lang/ (the files in these folders aren't versioned).
In smaller projects you can run npm run i18n:extract locally, put the generated file (default language) in a versioned folder and apply the translation on the other files in the folder based on the changes in the default language. Then when deploying in a CI environment you can just move the files from the versioned folder to src/src-lang/ and run npm run i18n:compile to generate the compiled translation files at src/lang/.
In development I just run npm run i18n:dev to create an empty json file ({}) so as to not receive errors when trying to import the file (the default language messages are already in the code, so the file doesn't need to have translations in the default language).
The only thing I haven't achieved was to be able to generate ids (https://formatjs.io/docs/guides/bundler-plugins), and use the generated ids in development (I don't know if this is possible without ejecting and stop using create-react-app, because it would need to transpile the source files to generate the ids). So, for now, I'm using static ids (if there is the same id for different messages I receive an error, so it's ok).


Merge CP #4 (error codes) into CP #1 (foundational)

Merge CP #4 text into CP #1, updating sections as necessary due to changes in structure and content in current state of foundational CP.

Starting on this now.

Starting on this now.

Excellent! I can make time this afternoon to assist if you need it.

Starting on this now.

Excellent! I can make time this afternoon to assist if you need it.

Thanks. I'll send another message with status when I'm done. This is the only time I can work on this prior to Friday's PCIM meeting (which I may miss, since I'm on vacation this week).

Pull request available for review.


Update upgrade.md

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Relates to #6340

I'm closing this one due to inactivity 😢


Consider flowing IServiceProvider through the ValidationContext created in DataAnnotationsModelValidatorProvider

In the implementation of IValidatableObject.Validate \ custom ValidateAttribute, all you get is an instance of ValidationContext. Short of using CallContextServiceLocator, there is no way to get to services that could be used as part of validation.
ValidationContext does support a way to pass in an IServiceProvider and consume it as part of validation. We should modify DataAnnotationsModelValidatorProvider to pass in HttpContext.RequestServices as the IServiceProvider when setting up the ModelValidationContext instance.

:ping:
This one could be closed

@tuespetre thanks!
This was fixed by https://github.com/aspnet/Mvc/commit/328f4d648b8d921d2e9e9e3849d8afa88d56694b


update changelog for release

copybara seems to be unresponsive to the lgtm internally. Think it's perhaps because there are no affected tests?


app start should also restart sysusers.service

We're not creating users as needed in the sandbox.

Nevermind! debug failure.


Partial completions observed in tokio , async-std crates

This issue is after I observed several instances. RA's provides partial completions . I treid with the following combinations VScode / Emacs with RLS and RA. The issue is , on the below code snippet, the intent is to import TcpStream where as I get only
use tokio::net::ToSocketAddrs

OTTH, the below code I was expecting completions like read,write etc but got nothing.
 let mut stream = TcpStream::connect("<IP_ADDRESS>:6142").unwrap();
stream.<no_completions>

Am I missing something here

Importing TcpStream works for me as soon as I enable the tcp feature in tokio, which doesn't seem to be enabled by default.
TcpStream::connect("<IP_ADDRESS>:6142").unwrap() seems wrong; connect is an async function, so you'd have to await it. If you await it, we still don't infer the type though; I think there we run into #2603 again.

Importing TcpStream works:

And so does the type inference for TcpStream::connect:

Importing TcpStream works:

And so does the type inference for TcpStream::connect:


[BUG] VMs removed a few days ago will still be pushed in

Describe the bug
Open the VM list page, delete some VMs, don't close them, don't do anything, look at the list page later and sometimes you will see the deleted VMs

May be to re-subscribe to the cause of problems

Can't reproduce now, will follow up if there is any update

Can't reproduce now, will follow up if there is any update

May be an issue on k3s version v.1.20+
link to #446

Validation Version

master-85a12ff7-head

Steps
https://github.com/rancher/harvester/issues/413#issuecomment-794735413
Result

Can't reproduce


Add a "pause on error" command to the VM service

When debugging, it's very useful for a user to be able to pause the program when an error occurs. Of course there's already top-level error handling that can be done, but specifically being able to pause at the site where the error is thrown is most useful for inspecting local state and setting relevant breakpoints for a future run.
This is supported via Observatory, so I assume it's available privately in the VM service. Making it public would be great, and would help improve the debugging UI for the test runner.

This would be great for Sky development, too.

We have implemented the ability to break when any exception is thrown.  This is currently implemented in the hidden method _setExceptionPauseInfo.
I agree that this service method should be made public.


Unnecessary memory usage

Hi @dbashford,
First, congrats and thanks for this module.
I'll try to not extend myself too much on this case. Extracting a text from a PDF, i noticed that the memory usage of the process was higher than i expected, and probably higher than it should be. So i wrote another script to extract the text and make the cleaning following the same modules ('pdf-text-extract', 'html-entities') and functions ('cleanseText') that you use, but only with the necessary parts for a PDF.
The result was:
PDF Size: 13mb
mem usage with the "lean script": 30mb
mem usage with textract: 96mb
Textract is using 3 times more memory, to do the same processing, with the same modules and functionalities.
So i digged a bit on your code and i noticed that you require all of the extractors ('lib/extractors/*') without knowing yet which one will be used. I didn't look all your code, but i believe that this process of require everything instead of require just only the extractors needed for the current case can be one of the main causes of this unnecessary memory usage.
Maybe this is not a big deal for extracting text from one file, in just a moment in time.
But thinking on a scenario where you have multiple files to be extracted for second, with multiple process doing the work in parallel, this can be bad for scaling.

Lazy load node modules has, in the past, been considered an anti-pattern, or at least been discouraged. I'll need to dig into what the current opinion of it is.
However, as textract involves extracting plenty of things, and sometimes you may only want to extract thousands of PDFs, I can see how waiting to load would be beneficial enough to outweigh any platform concerns.
I'm currently working on getting 2.0 out the door and I rather not increase the scope of that.  I can take a long look at this once I've pushed that out.

Will look at this as part of 2.1 after the 2.0 release.

Ask in case you need to exchange some ideas.
Probably you already know where to look, but i will give you a simple suggestion.
At lib/extract.js you have this:
// discover available extractors
  extractors = fs.readdirSync( extractorPath ).map( function( item ) {
    var fullExtractorPath = path.join( extractorPath, item );
    // get the extractor
    // eslint-disable-next-line global-require
    return require( fullExtractorPath );
  });

Maybe you can have an index of your current extractors to check for its availability, and then require the one you need.


1.21 crashing

since 1.21 changes so much it would seem that one of the mixins no longer routes correctly and the game crashes on start. would love to see sfc updated to 1.21

It may take a long time to port to 1.21.
I tried to implement #2 on 1.20.1 but something was wrong and stuck me for several month now. Even worse my life is going busy recently. So...😩

unfinish port in a new repo for 1.21 and neoforge:
https://github.com/Rivmun/SuperFancyCloudsRearchitected
Mojang changed a lot of code in renderClouds(). Forgive my poor code skill, I don't know how to render SFC on correctly position yet.


Fix: Convert properties to string before regex

It is not uncommon for developers to create schemas in YAML, for example
when developing an Open API Specification. When keys are numeric and
parsed with the yaml parser they are typed and throw an error when
attempting to validate patternProperties or similar pattern matching
validation. This changeset converts the property to a string before
attempting to match the regex.

Hey, thanks.
This is a dupe of #286 / #285.
Have a look and let me know what you think.

(for which by the way my opinion at the minute is still that this should be up to the deserializer totake care of immediately after deserializing)

Sounds fair. I'll move down stream to open API validator that lead me here in the first place and probably round trip yaml to JSON.
The Type Error was cryptic and annoying, but again more of a down stream issue.

Great!
On Wed, Apr 25, 2018, 20:29 Grant Callaghan<EMAIL_ADDRESS>wrote:

Closed #401 https://github.com/Julian/jsonschema/pull/401.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Julian/jsonschema/pull/401#event-1595373195, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAUIXjmw045Yk_YrbEDOaJ3Lh2FJgJ_Oks5tsRT_gaJpZM4TkIXK
.


show error message when service account is missing (hence revision will not be created)

knctl got stuck waiting for new revision because service account did not exist, hence Configuration wasnt updated, hence Revision will never be created.
$ knctl service show -s simple-app2
Service 'simple-app2'

Name             simple-app2
Domain           simple-app2.default.example.com
Internal Domain  simple-app2.default.svc.cluster.local
Annotations      -
Age              2m

Service 'simple-app2' conditions

Type                 Status  Age  Reason           Message
ConfigurationsReady  False   2m   RevisionFailed   Revision creation failed with message: "Internal error occurred: admission
                                                   webhook \"webhook.build.knative.dev\" denied the request: mutation failed:
                                                   serviceaccounts \"serv-acct1\" not found".
Ready                False   1s   RevisionMissing  Configuration "simple-app2" does not have any ready Revision.
RoutesReady          False   2m   RevisionMissing  Configuration "simple-app2" does not have any ready Revision.

Pods conditions

Pod  Type  Status  Age  Reason  Message

Succeeded

The same category of issue occurs if --template buildpack references a BuildTemplate that doesn't exist yet. It only appears in build-controller logs:
knative-build/build-controller-79d6cc9d57-7svtq[build-controller]: {"level":"error","logger":"controller.build-controller","caller":"build/build.go:141","msg":"Failed to validate build: buildtemplates.build.knative.dev \"buildpack\" not found","knative.dev/controller":"controller","knative.dev/controller"


chore: fix

Security Test: https://clerk.com/docs/security/vulnerability-disclosure-policy

!snapshot


Discussion: Macro and new features

Jörg-Michael, hi! I'd like to discuss three issues with you. I am grateful for your assistance.

Macro: This macro code is not working with the clean up rules that I want to apply. Despite this, there are no issues with the code. I'm curious if it's something else or just my poor code writing. I am grateful for your support.

Apply rules to project: Is it possible to apply clean-up rules to the entire project? At the moment, I can only apply clean-up rules to a single file, and I hope we can change that.
The old system: At the moment, there are some projects created on the previous system, thus there are some circumstances where we can not apply the clean up rules. How should I handle these situations?

Hi Tài,
ad 1: your code is fine and you're not doing anything wrong here – this is just a limitation of ABAP cleaner which would be very difficult to overcome. Therefore the message says "not supported", and I hope this case doesn't come up too often.
More precisely, a macro definition can contain incomplete code blocks such as an opening IF without an ENDIF…
  DEFINE check_initial.
    IF &1 IS INITIAL.
  END-OF-DEFINITION.

and using that macro would also lead to incomplete code blocks (3x closing ENDIF with no opening IF visible):
  check_initial lv_any.
  check_initial lv_other.
  check_initial lv_third.
      ENDIF.
    ENDIF.
  ENDIF.

This can be handled by a compiler by inserting the macro definition 3 times, but ABAP cleaner has no mechanism for this (and in some cases can't have it, because the macro definition might be "out of sight" for ABAP cleaner).
In the parser implemented in ABAP cleaner, WHEN is a bit of a special case, because it opens a block (next line is indented further) and at the same time closes the previous WHEN (a bit like ELSEIF), until the last WHEN is closed with ENDCASE. So, things would work if you extracted the "WHEN &1" line from the macro:
  DEFINE call_again.
    IF &1 IS NOT INITIAL.
      ...
    ENDIF.
  END-OF-DEFINITION.

and used it like this:
  CASE ....
    WHEN 1.
      call_again lv_any.
    WHEN 2.
      call_again lv_other.
  ENDCASE.

But that would be a workaround for the sake of ABAP cleaner, which may not be a good idea.
ad 2: No, there is currently no mechanism for that, either. ABAP cleaner runs completely locally on your machine, getting the code from ADT, and returning the changed code to it, so the entire handling of changes (on your transport) is left to ADT. You may, however, want to check whether the section Calling ABAP cleaner from the Command Line could help in your setup and workflow.
ad 3: Not sure what you mean here, what circumstances do you have in mind? If this is about older ABAP releases which don't yet support newer ABAP syntax, that could be covered by this setting on the ABAP cleaner UI:

Kind regards,
Jörg-Michael

Hi Tài,
sorry I had no better news that "out of scope" here, but these are indeed technical limitations of how ABAP cleaner is implemented, so closing this for now.
Kind regards,
Jörg-Michael


purge pending requests at the end of each phase

Instead of adding the same code at the end of each phase, we leveraged
the existing Phase trait. The Phase::next method is now split in
two parts:
trait Phase<R> {
    async fn run(&mut self) -> Result<(), StateError>;
    fn next(self) -> Option<StateMachine<R>>;
}

The run method runs the phase, and the next method just consumes
the phase to produce the next one. By separating these two bits of
logic, we can inject code right after the phase ran to completion
and before the transtion is happens:
impl<R, S> PhaseState<R, S>
    where Self: Phase<R>
{
    pub async fn run_phase(mut self) -> Option<StateMachine<R>> {
           self.run().await.unwrap();
           do_custom_stuff_common_to_all_the_phases();
           self.next()
    }
}

In our case, do_custom_stuff_common_to_all_the_phases() consists in
purging the request channel. Again, this is implemented for all the
PhaseState via the new Purge trait, which is very similar to the
Handler trait:
pub trait Purge<R> {
    fn handle_outdated_request(&mut self, req: R);
}

Finally, because we needed to be able to know the current phase, we
added a bunch of is_xxx() methods to the Phase trait, that return
true if the state machine is in the specific phase.

We are at the end of the Sum phase. The sum dict was published (to the fetcher service) and is now available to the update clients. Now, the state machine tries to clear the channel, which takes lets say about 30 seconds. At the same time, an update participant sends an update message, which however, is rejected by the PetMessages service because the state machine hasn't published the PhaseEvent::Update yet.

True in theory, but I don't think handling the pending requests at the end of the phase takes long in practice (a few ms per request maybe?), so I'm not too worried about it for the time being.


Protobuf serialization for schema

Codecov Report

Merging #200 into master will increase coverage by 0.06%.
The diff coverage is 82.97%.

@@            Coverage Diff             @@
##           master     #200      +/-   ##
==========================================
+ Coverage   66.29%   66.36%   +0.06%
==========================================
  Files          84       84
  Lines       15706    15691      -15
==========================================
+ Hits        10412    10413       +1
+ Misses       4466     4450      -16
  Partials      828      828

Impacted Files
Coverage Δ

rel/parse_sql.go
60.43% <100%> (+0.23%)
:arrow_up:

datasource/schemadb.go
68.11% <100%> (ø)
:arrow_up:

plan/projection.go
47.93% <33.33%> (ø)
:arrow_up:

schema/schema.go
92.72% <89.28%> (+2.16%)
:arrow_up:

lex/dialect_sql.go
85.77% <90.9%> (+1.44%)
:arrow_up:

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 20c2209...a9269c7. Read the comment docs.


Project4-ShipTracking-SAR

Proposed changes
Ship detection notebook from the ODC sprint.
Checklist (replace [ ] with [x] to check off)

[x] Remove any unused Python packages from Load packages
[x] Remove any unused/empty code cells
[x] Remove any guidance cells (e.g. General advice)
[x] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[x] Include relevant tags in the final notebook cell and re-use tags if possible
[x] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated

Starting a clean PR.


Initial task failed for action RecaptchaAction(action=getOobCode)with exception - An internal error has occurred. [ unexpected end of stream on com.android.okhttp.Address@80d85b16 ]

Is there an existing issue for this?

[x] I have searched the existing issues.

Which plugins are affected?
Auth
Which platforms are affected?
Android
Description
I use flutter + firebase to develop,
I had some errors using firebase's email login
I have been in the AndroidManifest. XML configuration for android: usesCleartextTraffic = "true" still rerun this mistake, my firebase console Also enable email login
Errors are as follows
I/FirebaseAuth(21869): Email link sign in for<EMAIL_ADDRESS>with empty reCAPTCHA token
W/System  (21869): Ignoring header X-Firebase-Locale because its value was null.
W/LocalRequestInterceptor(21869): Error getting App Check token;  using placeholder token instead. Error: com.google.firebase.FirebaseException: Too many attempts.
Initial task failed for action RecaptchaAction(action=getOobCode)with exception - An internal error has occurred. [  unexpected end of stream on com.android.okhttp.Address@80d85b16 ]
Here are some of my configurations
AndroidManifest.xml
<manifest xmlns:android="http://schemas.android.com/apk/res/android">
    <application
        android:label="wallpaper_book"
        android:name="${applicationName}"
        android:usesCleartextTraffic="true"
        android:icon="@mipmap/ic_launcher">
        <activity ......

firebase console
enter image description here
Reproducing the issue
FirebaseAuth.instance
    .sendSignInLinkToEmail(
        email<EMAIL_ADDRESS>        actionCodeSettings: ActionCodeSettings(
            url: 'https://wallpaper-book.firebaseapp.com/__/auth/',
            // This must be true
            handleCodeInApp: true,
            // iOSBundleId: 'com.example.ios',
            androidPackageName: 'com.example.wallpaper_book',
            // installIfNotAvailable
            androidInstallApp: true,
            // minimumVersion
            androidMinimumVersion: '12'))
    .catchError((onError) {
  OtherUtils.showToastInfo('Error sending email verification $onError',
      backgroundColor: const Color(0xFF7c2eed), textColor: Colors.white);
}).then((value) {
  Loading.dismiss();
});

}
Firebase Core version
2.31.1
Flutter Version
3.19.6
Relevant Log Output
I/FirebaseAuth(21869): Email link sign in for<EMAIL_ADDRESS>with empty reCAPTCHA token
W/System  (21869): Ignoring header X-Firebase-Locale because its value was null.
D/TrafficStats(21869): tagSocket(133) with statsTag=0xffffffff, statsUid=-1
D/Linux   (21869): [Posix_connect Debug]Process com.example.wallpaper_book :9099
E/RecaptchaCallWrapper(21869): Initial task failed for action RecaptchaAction(action=getOobCode)with exception - An internal error has occurred. [ unexpected end of stream on com.android.okhttp.Address@80d85b16 ]

Flutter dependencies
dependencies {
// Import the Firebase BoM
implementation platform('com.google.firebase:firebase-bom:33.0.0')
// TODO: Add the dependencies for Firebase products you want to use
// When using the BoM, don't specify versions in Firebase dependencies
implementation 'com.google.firebase:firebase-analytics'
// Add the dependency for the Firebase Authentication library
// When using the BoM, you don't specify versions in Firebase library dependencies
implementation("com.google.firebase:firebase-auth-ktx")

// Also add the dependency for the Google Play services library and specify its version
implementation("com.google.android.gms:play-services-auth:20.7.0")

// Add the dependencies for any other desired Firebase products
// https://firebase.google.com/docs/android/setup#available-libraries
}
Additional context and comments
No response

Can you try to reproduce this issue again with the latest flutterfire packages?

您能尝试使用最新的 flutterfire 包再次重现此问题吗？

Yes, I tried it with the latest version. Same result
firebase_auth: ^5.0.0
firebase_core: ^3.0.0

Does the same also happen to the flutter auth example app?
https://github.com/firebase/flutterfire/tree/master/packages/firebase_auth/firebase_auth/example

Flutter auth 示例应用程序是否也发生同样的事情？
https://github.com/firebase/flutterfire/tree/master/packages/firebase_auth/firebase_auth/example

Yes the same, is not my computer environment problem

Flutter auth 示例应用程序是否也发现了一些有趣的事情？
https://github.com/firebase/flutterfire/tree/master/packages/firebase_auth/firebase_auth/example

My own projects as well as flutter_auth_example have no problem logging in with google

So is the issue still happening or it has been solved? if so please indicate what was the solution as other users might find it useful.

So is the issue still happening or it has been solved? if so please indicate what was the solution as other users might find it useful.

It did not solve, I just explained my current operation situation, running on my machine, using google login is OK, using email login will report an  error

Hello @JuYiYang, Can you please send a complete minimal running reproduction of this issue?

Facing the same issue as well.

@TarekkMA https://github.com/firebase/flutterfire/tree/master/packages/firebase_auth/firebase_auth/example the OP mentioned that the example for email link sign-in is facing the same issue, is that enough for a complete minimal running reproduction of the issue?

@cheejiayuan512 @JuYiYang I wasn't able to replicate this issue with the example app. Maybe check if you are using the AppCheck plugin to set the token correctly?

Hi Tarekk, turns out it was user error. The email was sent to my spam folder which I did not check, and I needed to click on the link within the emulator itself instead of copypasting it from desktop to emulator. Not sure about OP, thank you!

@cheejiayuan512 Glad that the issue is solved for you. We'll keep this issue open until the OP provides more insight on whether this solution also helped them or not.

Hello @JuYiYang, Can you please send a complete minimal running reproduction of this issue?
Of course
test.zip

Of coursehttps://drive.google.com/file/d/13C9L8HrL_BcoytNg89rO6Jn7AmW1XbO-/view?usp=drive_link

Of course this file is only 80mb but I wonder why the hard disk is so big[https://drive.google.com/file/d/13C9L8HrL_BcoytNg89rO6Jn7AmW1XbO-/view?usp=drive_link](url)

Hello @JuYiYang, Can you please send a complete minimal running reproduction of this issue?

Of course this file is only 80mb but I wonder why the hard disk is so big[https://drive.google.com/file/d/13C9L8HrL_BcoytNg89rO6Jn7AmW1XbO-/view?usp=drive_link](url)

嘿@JuYiYang。我们需要更多信息来解决此问题，但 7 个工作日内没有更新。我将问题标记为过时，如果接下来 7 天内没有新的更新，我将自动关闭它。
如果您有更多信息可以帮助我们彻底解决此问题，请添加评论！

I send the code link
https://drive.google.com/file/d/13C9L8HrL_BcoytNg89rO6Jn7AmW1XbO-/view?usp=sharing

I'm getting this error when trying to run your project, please try to fix this issue and upload again (or use GitHub if possible)
[test] flutter pub get --no-example
Resolving dependencies...
Note: intl is pinned to version 0.19.0 by flutter_localizations from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.

Because wallpaper_book depends on flutter_localizations from sdk which depends on intl 0.19.0, intl 0.19.0 is required.
So, because wallpaper_book depends on intl ^0.18.1, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Try upgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

--

[test] flutter pub get --no-example
Resolving dependencies...
Note: intl is pinned to version 0.19.0 by flutter_localizations from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.

Because wallpaper_book depends on flutter_localizations from sdk which depends on intl 0.19.0, intl 0.19.0 is required.
So, because wallpaper_book depends on intl ^0.18.1, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Try upgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

--

[test] flutter pub get --no-example
Resolving dependencies...
Because wallpaper_book depends on intl ^0.19.1 which doesn't match any versions, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

--

[test] flutter pub get --no-example
Resolving dependencies...
Because wallpaper_book depends on intl ^0.19.1 which doesn't match any versions, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

--

[test] flutter pub get --no-example
Resolving dependencies...
Note: intl is pinned to version 0.19.0 by flutter_localizations from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.

Because wallpaper_book depends on flutter_localizations from sdk which depends on intl 0.19.0, intl 0.19.0 is required.
So, because wallpaper_book depends on intl ^0.18.1, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Try upgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

我在尝试运行您的项目时收到此错误，请尝试修复此问题并再次上传（或尽可能使用 GitHub）
[test] flutter pub get --no-example
Resolving dependencies...
Note: intl is pinned to version 0.19.0 by flutter_localizations from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.

Because wallpaper_book depends on flutter_localizations from sdk which depends on intl 0.19.0, intl 0.19.0 is required.
So, because wallpaper_book depends on intl ^0.18.1, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Try upgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

--

[test] flutter pub get --no-example
Resolving dependencies...
Note: intl is pinned to version 0.19.0 by flutter_localizations from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.

Because wallpaper_book depends on flutter_localizations from sdk which depends on intl 0.19.0, intl 0.19.0 is required.
So, because wallpaper_book depends on intl ^0.18.1, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Try upgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

--

[test] flutter pub get --no-example
Resolving dependencies...
Because wallpaper_book depends on intl ^0.19.1 which doesn't match any versions, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

--

[test] flutter pub get --no-example
Resolving dependencies...
Because wallpaper_book depends on intl ^0.19.1 which doesn't match any versions, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

--

[test] flutter pub get --no-example
Resolving dependencies...
Note: intl is pinned to version 0.19.0 by flutter_localizations from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.

Because wallpaper_book depends on flutter_localizations from sdk which depends on intl 0.19.0, intl 0.19.0 is required.
So, because wallpaper_book depends on intl ^0.18.1, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Try upgrading your constraint on intl: flutter pub add intl:^0.19.0
exit code 1

This may be the intl version of the problem, my native can work properly, can you try to upgrade to 0.19.0 try


Closes Issue # 1049 --> Twitter Logo Was Not Visible In Both Mode

Fix #1049
Issue
Twitter Logo Was Not Visible In Both Modes
Bug

After Fix
Image 1

Image 2

@sanjay-kv  Please Check The Above Pull Request


Object.keys is expensive when called on thousands of keys. Gotta go fast

We use this on a client where our ring contains ~ 200 nodes. With 160 replicas, that is 32,000 items in the ring. CPU profiling has revealed that calling Object.keys(this.ring) turned out to be expensive when called frequently (called twice in getNode())

Benchmark results
ORIGINAL should get correct ring length (27082 ms)
FAST should get correct ring length (167 ms)

Benchmark code:
describe('Faster Consistent Hashing class', () => {
  const nodes = Array(200)
    .fill(1)
    .map((_, idx) => idx);
  const chash = new ConsistentHashing(nodes);
  const chashFast = new FastConsistentHashing(nodes);
  it('ORIGINAL should get correct ring length', async () => {
    await benchmark.record(
      () => {
        chash.getRingLength();
      },
      { iterations: 10000 },
    );
  });
  it('FAST should get correct ring length', async () => {
    await benchmark.record(
      () => {
        chashFast.getRingLength();
      },
      { iterations: 10000 },
    );
  });
});


[Beta 14] Casting Gauntlet Render Bug

I'm using beta 14 because of bug #858
When I hold the casting gauntlet in the my survival world the skybox seems to glitch out, making a black square around the sun, sometimes some scattered black dots in the sky and the horizon line becomes orange, it doesn't make the game umplayable, just makes it a bit offseting and weird.

Are you using Optifine or shaders?

disable AA from optifine

AA is disabled, still getting this bug


Add api key, basic auth and Authorization header change support for Websocket APIs

@Arshardh  are we going to ship this in 3.0.0 release or after 3.0.0 , could you please remove the 3.0.0 label  if not


apend builder with requestment.txt supported

/kind feature
Describe the solution you'd like
[A clear and concise description of what you want to happen.]
requestment.txt support when using append builder, just like docker builder did; since i mostly use fairing sdk inside container whitout docker，we can't use docker builder, but the suppot of requestment.txt  is convenient.
or should i install docker inside the container? that's a little complicated as i'm known
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

You can try the cluster build, thanks.
https://github.com/kubeflow/fairing/blob/950d8054ac4f4761179b02c0af0c49288ace4e3a/kubeflow/fairing/builders/cluster/cluster.py#L15


Add normalization

In this project me and my coding partner @wacoo  working on normalization and have following features: :100:

Insert enough data to check execution time. :+1:
For optimization apply the index property on each column that is use for filtration. :+1:
Below are the Before and After Queries execution time.   :+1:

Before:

0a-117ee37dbc24)
After Applying Index:

@faisalakb, please kindly get your partner to make some commits else I'll have to invalidate 🙏


stdenv: make isELF more precise

Motivation for this change
Things done

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

[ ] 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 nox --run "nox-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)
[ ] Fits CONTRIBUTING.md.

see also #47244

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: stdenv
Partial log (click to expand)
cannot build derivation '/nix/store/nf04aykq06dnd6crhsccrpj8vagaznh7-flex-2.6.4.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/dzp61f7ds3mpngnix717h6mlg5n8bga3-gnutar-1.30.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/sq6gc110gy3a76p7yxmwzdz4zhd49jk7-bootstrap_cmds-dev-tools-7.0.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/srkrvawcx6q7r154zk8kqfyaf0pfjijz-cctools-binutils-darwin-wrapper.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/vyh5gpsdgbq5g44pf8qzm15iz1gz28rd-clang-wrapper-5.0.2.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/s18v0ik66f42dbkcs253z11bn4hdyzh5-libkrb5-1.15.2.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/ai18hw31vj7mida7wv4bzb95h7irarcd-curl-7.61.1.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/a92ymmiksabzkay25mj4llzr8ykcfljy-swift-corefoundation.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/1k7a5li0wi7pvif0dibrsfcksvm9kryy-stdenv-darwin.drv': 42 dependencies couldn't be built
error: build of '/nix/store/1k7a5li0wi7pvif0dibrsfcksvm9kryy-stdenv-darwin.drv' failed

Timed out, unknown build status on aarch64-linux (full log)
Attempted: stdenv
Partial log (click to expand)
cannot build derivation '/nix/store/k1qg9fjxj6w9xkkjndmbj5n297995ard-help2man-1.47.6.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/jip8r8fpbhcwbr6bzq50dmh10pg1k88r-binutils-wrapper-2.30.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/y163ixn03ivk39g5fdfpsl64ynmc3vwm-diffutils-3.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/mizr0k2y0z3djn9bxij1cmjba6qkz7j6-findutils-4.6.0.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/vfh9vzpz9kllskzb553iplinmgzd5gmv-libtool-2.4.6.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/ni7xdz4j0k9aml0rxchzyacby0rsblad-gcc-wrapper-7.3.0.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/i6xpqpix8vxqkqwmq8v2rg0ivrpc0slf-hook.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/vqcyrixjs4pl7965sarsmrsr9pg19aiy-patch-2.7.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/r6in5v8xkzywr4pd542fmvvvsph52n8n-stdenv-linux.drv': 25 dependencies couldn't be built
error: build of '/nix/store/r6in5v8xkzywr4pd542fmvvvsph52n8n-stdenv-linux.drv' failed

Timed out, unknown build status on x86_64-linux (full log)
Attempted: stdenv
Partial log (click to expand)
cannot build derivation '/nix/store/ddp9l8k09qafcn48f49phzj7lidkk4jv-help2man-1.47.6.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/7imsadizzan9lxx1yqkdf9pab5r02mcj-binutils-wrapper-2.30.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/pvpax20wyhni0f7rdzilfc1f90hi8b4w-diffutils-3.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/ar8qacllap14xyf0j3n9y2c3r31i0i6j-findutils-4.6.0.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/793aqkdqvylf32fan4dgksmcyc3i1fdf-libtool-2.4.6.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/34j45a5pais1bjd33j93swj6hqbllvyx-gcc-wrapper-7.3.0.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/z48kvng71xhvk6bwm5s739k3rnij17l8-hook.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/i3k1a4sccmzbfns5wlgharpbhpkdbsh5-patch-2.7.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/wycidl4ijw6bzmh9xpdhb3a5m971p8g2-stdenv-linux.drv': 23 dependencies couldn't be built
error: build of '/nix/store/wycidl4ijw6bzmh9xpdhb3a5m971p8g2-stdenv-linux.drv' failed


Document the CLI environment variables used for testing

What this PR does / why we need it
Document the CLI environment variables used for testing
Environment variables that are user-facing are documented in #714
You can read the new doc in markdown at https://github.com/marckhouzam/tanzu-cli/blob/feat/docTestEnvVars/docs/dev/README.md#environment-variables-for-testing-the-cli
Which issue(s) this PR fixes

Fixes # N/A
Describe testing done for PR
Visual check at https://github.com/marckhouzam/tanzu-cli/blob/feat/docTestEnvVars/docs/dev/README.md#environment-variables-for-testing-the-cli
Let CI run
Release note

Document the CLI environment variables used for testing

Additional information
Special notes for your reviewer

lgtm, this section (as well as the one updated in #714) seems an appropriate place to reference the doc about using tanzu config set env as a means to persist the env vars.
I think that detail was explained in a row in https://github.com/marckhouzam/tanzu-cli/blob/feat/docTestEnvVars/docs/full/README.md#cli-configuration.

lgtm, this section (as well as the one updated in #714) seems an appropriate place to reference the doc about using tanzu config set env as a means to persist the env vars. I think that detail was explained in a row in https://github.com/marckhouzam/tanzu-cli/blob/feat/docTestEnvVars/docs/full/README.md#cli-configuration.

Good idea.
How about:
"These variables can be set in the shell or using tanzu config set env.<VAR> <VALUE>
(see the documentation about this command)."

I've updated the PR with both @anujc25's and @vuil's suggestions


Fill json field using column->key syntax fails when column contains non-object

Laravel Version: 7.28.4
PHP Version: 7.4.14
Database Driver & Version: Mysql 5.7.26

Description:
fill of json field results in exception if the json field is not an object
Steps To Reproduce:
Create a json column in a table. For example, I have a user_info column in my users table.
If I set the json column to null before I start, it works as expected:
>>> User::find(3)->forceFill(['user_info' => null])->save()
=> true
>>> User::find(3)->forceFill(['user_info->name' => 'scott'])->save()
=> true
>>> User::find(3)->user_info
=> "{"name": "scott"}"

However, if I set the column to another valid json string which doesn't parse as an object, it throws an exception
>>> User::find(3)->forceFill(['user_info' => '"json string which is not an object"'])->save()
=> true
>>> User::find(3)->forceFill(['user_info->name' => 'scott'])->save()
<warning>PHP Warning:  Illegal string offset 'name' in /var/www/html/station/vendor/laravel/framework/src/Illuminate/Support/Arr.php on line 567</warning>
=> true

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? We'll help you out and re-open this issue if so.
Thanks!

Sure. I upgraded to
Laravel Version 8.27.0
PHP Version: 7.4.15
Database Driver & Version: Mysql 8.0.21
Same problem.

Please try a support channel:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks.


Alignment of buttons in the gallery item is off

          i guess this is for another issue?

Originally posted by @exezbcz in https://github.com/kodadot/nft-gallery/issues/4766#issuecomment-1398418160

@exezbcz can say more
please close if duplicate

https://github.com/kodadot/nft-gallery/issues/4785

Thanks @petersopko


Traffic Shaper documentation seems to be wrong

There seems to be an issue in yours traffic shaper documentation.
Mapping all >destinations-masks< to >share bandwidth<
and all >source-masks< to >limit all connection by whole pipe bandwidt< sounds wrong.
Detailed explained here:
https://forum.opnsense.org/index.php?topic=4949.0

This is fixed now, can also be closed @fichtner ...


💭 Capturing photo of Drawable Frames of SkiaFrameProcessor

Question
I have gone through the documentation that states
Skia Frame Processors are currently preview-only. Any content drawn to the Frame will not be visible in captured photos, snapshots or videos.
I don't want drawn content to be captured in my photos, I just want to capture photos without the drawn components.
However, when I try to create a Camera component that can take photos on drawable frames an error appears as follows.
Camera.onError(session/invalid-output-configuration): Failed to configure the Camera Session because the output/stream configurations are invalid! [session/invalid-output-configuration: Failed to configure the Camera Session because the output/stream configurations are invalid!]
Am I missing something? Is it not possible at all to capture photos when Drawable frames are being rendered?
What I tried
No response
VisionCamera Version
"react-native-vision-camera": "^4.0.3"
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

This is possible, but it seems like something went wrong when trying to create the session. Please create a bug report with native logs.


Problem in OSX Bash

Once I hit the button on the screen all key input moves the cursor back to the top. I can easily build a tree this way but I guess this is an unwanted behaviour.

Steps to reproduce:
Simple write at the end of the terminal window a really long command like I did here.

node version: v8.9.4
Office 365 Cli version<EMAIL_ADDRESS>

I had this same issue yesterday with the current beta version

This is a known issue in vorpal that we use as a base to build the CLI on. It appears only in the immersive mode. So far, there is no fix for it. I'll leave it up to you guys if you want to keep this issue open until there is a fix or not. Thank for reaching out 👍

It's taken a while but I've found and fixed the issue in vorpal, the underlying engine behind the CLI. The issue has been fixed in our fork of vorpal and the fix should be available immediately in all versions of the CLI after reinstalling the CLI. For more information see https://github.com/pnp/vorpal/commit/f90d9e1924cfbcf21e4b5abae333f06b79b26561


Minor typo

YOU vs. You.

@mikewaretena : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mikewaretena Thanks!


Add rpush command

Added support for rpush command
created lread and lwrite method in storage trait to read and write list in memory
added test for rpush command

Thank you for your contrib @Murugesh-R :)

There is an error - can you take a look?


mittus documentation

Does Mitsu have a documentation ???

The best I can offer is the auto-generated ones on rubydoc: https://www.rubydoc.info/gems/mittsu/0.3.2

solved

solved


Dark line between modeline and echo area

Wondering if there's a way to get rid of this. Here's a question I opened on StackExchange.

If you're using my emacs.d, changes are you're also using window-divider-mode, which adds this border. You can either turn off the bottom border with (setq window-divider-default-bottom-width 0) or turn off window-divider-mode.

;; The native border "consumes" a pixel of the fringe on righter-most splits (in
;; Yamamoto's emacs-mac at least), window-divider does not.
;; NOTE Only available on Emacs 25.1+
(when (boundp 'window-divider-mode)
  (setq window-divider-default-places t
        window-divider-default-bottom-width 1
        window-divider-default-right-width 0)
  (window-divider-mode +1))

Brilliant, thanks. I've pulled a lot of stuff from your config, and that was one of the pieces.


fix: retrieving meshes for a region present in the onthology but absent in the annotation atlas

Description
What is this PR
Bug fix
Why is this PR needed?
Sometimes you want to display some regions by selecting them in the onthology. However you don't always know if they have a corresponding mesh. An example is RSPd4 in Allen's CCFv3
What does this PR do?
Skip those problematic regions
How has this PR been tested?
import brainrender
atlas = brainrender.Atlas("allen_mouse_25um")
print(atlas.get_region("RSPd4", "RSPd5"))

the above code crashes without this PR. With it, it just adds region RSPd4
Is this a breaking change?
No
Does this PR require an update to the documentation?
No
Checklist:

[x] The code has been tested locally
[ ] Tests have been added to cover all new functionality (unit & integration)
[ ] The documentation has been updated to reflect any changes
[x] The code has been formatted with pre-commit

Thanks @carlocastoldi!

sorry for the dumb question, but what's the release cycle like for brainrender? When can I approximately expect to update it in my project and have the fix in?
Thank you in advance!

 However please see #247. brainrender requires a considerable amount of work before the next release on PyPI.
We hope to update the project and move to a more regular release cycle soon, but there is no ETA as yet.

That's awesome. Thank you a lot for the clarification!
Hope I can help in future, perhaps ^^

Related to #247 I think I remember that some of the problems originated from some breaking changes in vedo.
This is again a rather critical moment because since about a month or so I'm planning to restructure some class inheritances which would almost certainly affect brainrender.
At this very moment I would suggest to keep it on hold and not trying to adapt it to the vedo 2023.4.6, until I make up my mind about the API changes.
I will let you know "soonish" how things will evolve!
Thanks for your patience.

Thanks @marcomusy, much appreciated. Please do let us know.

So the vedo restructuring is basically completed!
I'm quite happy with the result because now it's much better organised, ..and most of the examples are unchanged!
This means the API is pretty much the same.
You can already start playing with it if you wish in restruct_mem branch with:
git clone https://github.com/marcomusy/vedo.git
cd vedo
git checkout restruct_mem
pip install -e .
vedo --info

and you should now have vedo 2023.5.0+dev17a.
The main difference is that now the Mesh (or Volume) object is no more a subclass of vtkActor but it's an independent object containing a vtkActor which can be still accessed by class member .actor.
So plotter.actors becomes plotter.objects.
Analogously the underlying vtkPolydata (or vtkImagedata) are accessible by class member .dataset. (in principle you don't need to access neither of the 2).
Check out vedo/docs/changes.md for more details.


temp_id on update fix

Part of RHMAP-2528
This fix addresses the create -> update issue row 4 in spreadsheet
@wei-lee used js as a reference, but there is still one error the UidMapping will grow and because it's saved on the device it will crash the app eventually. Don't know when we can clear this list but we need to build this in as well.

looks good. Have you seen the app crashes because of the the UidMapping? What error did you get? Yeah, I am not sure about the best way to clear that map too.

@wei-lee haven't seen the app crash, but when it will continue to grow it will eventually take up either all the memory or all the disk space, right?

yeah. But in that case, the crash won't be caused by the UidMap, the whole dataset size will probably grow over the maximum memory that is allowed for an app on the device. It's one of the known limitations of the framework.
We can definitely look into improvements in the future.

@edewit @wei-lee same story with the historyChange, like the uidmapping, this is temporary infomation that could be cleaned to minimize the size of the storage. let add a jira to track it.

history change should be removed. It's not required anymore?
We the others including meta data which will be clean update automatically after each sync loop. So the only data that will keep growing are the dataset and the uid map.


Build native library just once and various updates to reduce compile times

Builds most of the C++ runtime library into a libweldrt instead of rebuilding it each time you compile a Weld module (which took about 500 ms with O2 optimizations). A few key functions are still saved as a .bc file to be inlined.
Makes the LLVM optimization level configurable.
Changes the way we generate code for structs to reduce the number of temporary structs stored in registers, which can slow down / confuse LLVM codegen. We use getelementptr on the struct to load the required field directly instead.

Looks good, you didn't run into any of the previous issues we had when linking weldrt into the main libweld (unused symbols getting removed, etc.)?

No, it stays linked dynamically, and the C++ compiler doesn't drop any of those symbols.

Just checked the assembly/optimized LLVM code and it looks good - you can merge it!


Spyder won't start

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Spyder won't start from shortcut or from the anaconda navigator, it might have to do with installing a package with pip but after reseting spyder and uninstalling and installing everything it still gives the same error
What steps reproduce the problem?

starting spyder from anaconda

What is the expected output? What do you see instead?
I expect spyder to start
Paste Traceback/Error Below (if applicable)

i dont have a traceback but here is the error code
Exit code:<PHONE_NUMBER>

Versions

Spyder version: 5.4.3
Python version:
Qt version:
PyQt version:
Operating System name/version: windows 11

Dependencies

i cant start spyder

Hi @tijmen03 thank you for the feedback! At first glance seems like something broke in your anaconda installation (maybe due to the package installed via pip). Checking the issue tracker seems like this could be related with https://github.com/spyder-ide/spyder/issues/21124 or https://github.com/spyder-ide/spyder/issues/21217. Could you check if following some of the suggestions on those issues helps? Let us know!

I think i have the same issue as #21217 with that the docking station we have is causing the problem

Thank you for letting us know! I will close this then as a duplicate of #21217 and potencially https://github.com/spyder-ide/spyder/issues/20344


T-SNE报错

Branch
1.x branch (1.x version, such as v1.0.0rc2, or dev-1.x branch)
Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version.

Environment
1.0.0rc6
Describe the bug

TypeError: 'int' object is not iterable
Reproduces the problem - code sample
No response
Reproduces the problem - command or script
No response
Reproduces the problem - error message
No response
Additional information
No response

please check the input of out_indices


Off-hand functionality

Further testing showed me the Umbrellas, or at the very least the Gliding enchantment doesn't work while being held on the Offhand, which was quite the surprise, well, I mention it since it seemed like the natural behavior.
Hopefully this is added or fixed on a future update.

This is intended behavior, so that you can 'toggle' the gliding effect of the umbrella, and to balance the slow falling a bit by making you unable to use other items. However, it's very unintuitive and other people have assumed it was a bug in the past, so I do plan on changing it eventually.


[BackPort] Conservative updates

This is a back port of https://github.com/rust-lang/cargo/pull/6611 as discussed in https://github.com/rust-lang/cargo/issues/6609

r? @ehuss
(rust_highfive has picked a reviewer for you, use r? to override)

:warning: Warning :warning:

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

Looks like #6649 will be needed to get this to pass.  I'll post it in a few minutes.

Or you can push to this branch whatever is easier.

@bors r+
I'm planning to update beta (on rust-lang/rust) towards the end of the week.

:pushpin: Commit 63412be12bddef17e88a003f18e9747a8fa5978f has been approved by ehuss

:hourglass: Testing commit 63412be12bddef17e88a003f18e9747a8fa5978f with merge f099fe94b66f0a2f80370be8f2d3db2a55b97050...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: ehuss
Pushing f099fe94b66f0a2f80370be8f2d3db2a55b97050 to rust-1.33.0...


TypeScript: References include definition, which breaks "editor.gotoLocation.multipleReferences": "goto" usage

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: insider
OS Version: Windows

Steps to Reproduce:

Set "editor.gotoLocation.multipleReferences": "goto" setting
Create untitled TypeScript file:

const test = () => {}
test()
test()

Ctrl+clicking test on first line or execute Go to References  command

Expected: focus on line 2
Actual: stays on the same position
References provide isn't being called on Go to References command, but specifically TypeScript extension still includes definition into references. I hope you take this issue as it is really annoying, there is absolutely no need to do this + I can call definitions manually when I want.

I was just curios why its happening only with TypeScript extension and it seems that:
Maybe don't need to use this?  Why is only TypeScript ext using it?https://github.com/microsoft/vscode/blob/5128d2c32c217dcc33da2527a6936acd083f0603/extensions/typescript-language-features/src/languageFeatures/references.ts#L35
https://github.com/microsoft/vscode/blob/f3f9f7a10d877fd2177daec3e1f6054706c7e5bc/src/vs/editor/contrib/gotoSymbol/browser/goToSymbol.ts#L63
Its hardcoded here, there is no way to disable it even not via API,
And finally this change was also not clear for me
https://github.com/microsoft/vscode/commit/c4d648124751ada64d0d138a1fef4f1d59196181#diff-d897e7805588a8bb44eb7cb12ae2d505658e8c29b8c4bd7625233b6dedbb90bb

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

I'm sorry to mention you here @robeady but it seems to be closed in favor of your https://github.com/microsoft/vscode/issues/114673 did you a find a solution since then? We're personally using extension to workaround that. 😞

no, I didn't, but I don't know if the issue I reported has been fixed since I last checked in on it


improve printing of byte sizes

Note that I am using Binary Prefix (Base 1024), not Decimal Prefix (Base 1000). Both are commonly used, though I believe Binary Prefix is more common when talking about RAM usage.

Both are commonly used, though I believe Binary Prefix is more common when talking about RAM usage.

Hardware often deals with binary because that's literally how much space can exist in RAM or on disk as well - software, including what is displayed to users, is often the more intuitive decimal usage since it is less confusing to an average user to specify something like 123456 bytes and get 117 MiB out of it.
Also a tad more intuitive to easily to calculate decimal version from things like shape of array and itemsize of dtype since no correction factor is needed
For that reason I'd be inclined to use binary=False in backend configurations, but binary=True in memory errors
But this is a thing that is usually a personal preference

Yeah, I was reading up a bit on conventions for this. Sometimes MB means binary and sometimes it means decimal. There isn't an easy way to know. I chose to go with binary base with the "i" in the middle since that is explicit, but I agree it's less intuitive. I think you're right that it may be better to have memory config in decimal

Looks good to me

Note that I am using Binary Prefix (Base 1024), not Decimal Prefix (Base 1000). Both are commonly used, though I believe Binary Prefix is more common when talking about RAM usage.

I also prefer base 1024 in general, and I think that the 'i' in between makes it clear.
But I remember that @CodyCBakerPhD told me something along those lines:

Hardware often deals with binary because that's literally how much space can exist in RAM or on disk as well - software, including what is displayed to users, is often the more intuitive decimal usage since it is less confusing to an average user to specify something like 123456 bytes and get 117 MiB out of it.

Just to be concrete, this problems happens because the GenericDataChunkIterator takes units on base 10, right? That's the part of the ecosystem where the user is expected to input units in base 10?


alsaLib: 1.2.3 -> <IP_ADDRESS>

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/alsa-lib/versions.
meta.description for alsaLib is: "ALSA, the Advanced Linux Sound Architecture libraries"
meta.homepage for alsaLib is: "http://www.alsa-project.org/"
meta.changelog for alsaLib is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found <IP_ADDRESS> with grep in /nix/store/w8k92774hrc93pgxr5gsm1riarh4vbgh-alsa-lib-<IP_ADDRESS>

directory tree listing: https://gist.github.com/19482ca33b01cf0b10b5e17b7e8e8051

du listing: https://gist.github.com/455de7954c785115be28c97568f91707

Rebuild report (if merged into master) (click to expand)

8989 total rebuild path(s)

3338 package rebuild(s)

3335 x86_64-linux rebuild(s)
2825 i686-linux rebuild(s)
112 x86_64-darwin rebuild(s)
2717 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
DisnixWebService
EmptyEpsilon
MMA
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer
SDL2_net
SDL2_ttf
SDL_Pango
SDL_gfx
SDL_gpu
SDL_image
SDL_mixer
SDL_net
SDL_sound
SDL_stretch
SDL_ttf
Sylk
_20kly
_90secondportraits
a2jmidid
abbaye-des-morts
abcde
abcl
acoustidFingerprinter
adapta-gtk-theme
adlplug
adom
adoptopenjdk-bin
adoptopenjdk-hotspot-bin-11
adoptopenjdk-hotspot-bin-13
adoptopenjdk-hotspot-bin-8
adoptopenjdk-icedtea-web
adoptopenjdk-jre-bin
adoptopenjdk-jre-hotspot-bin-11
adoptopenjdk-jre-hotspot-bin-13
adoptopenjdk-jre-hotspot-bin-8
adoptopenjdk-jre-openj9-bin-11
adoptopenjdk-jre-openj9-bin-13
adoptopenjdk-jre-openj9-bin-8
adoptopenjdk-openj9-bin-11
adoptopenjdk-openj9-bin-13
adoptopenjdk-openj9-bin-8
aegisub
aeolus
agg
ahoviewer

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w8k92774hrc93pgxr5gsm1riarh4vbgh-alsa-lib-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A alsaLib https://github.com/r-ryantm/nixpkgs/archive/a27c35a2c1784fcbd34ac597373fdb2efff414c0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w8k92774hrc93pgxr5gsm1riarh4vbgh-alsa-lib-<IP_ADDRESS>
ls -la /nix/store/w8k92774hrc93pgxr5gsm1riarh4vbgh-alsa-lib-<IP_ADDRESS>/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

Changes https://github.com/alsa-project/alsa-lib/compare/v1.2.3...v<IP_ADDRESS>

newer on master

newer on master


Constant(static final) doesn't work on annotation

Literally, as like the title, it seems only String literal can make lombok works, but constant variable can't.
For example,
@EqualsAndHashCode(of = "id")

the above works fine, But
@EqualsAndHashCode(of = Foo.ID)

this fails, where Foo is a external class that has public static final String ID = "id";. Of course, the source can be compiled, but when I just decompile and check the generated code, then I can see there is no comparison between values of id, like below, where Category is the class lombok annotations are attached on.
    public boolean equals(Object o) {
        if (o == this) {
            return true;
        } else if (!(o instanceof Category)) {
            return false;
        } else {
            Category other = (Category)o;
            return other.canEqual(this);
        }
    }

    protected boolean canEqual(Object other) {
        return other instanceof Category;
    }

    public int hashCode() {
        int result = 1;
        return result;
    }

I can't build the recent version with ant.
It think some classes in utils/lombok dir (Eclipse.java etc) now depends on AnnotationValues class, and the AnnotationValues is not be compiled before src/utils dir.


Vanishing brine in evaporation towers

This is for Mekanism <IP_ADDRESS>7 / MC 1.7.10.
I have two max height evaporation towers for my reactor setup that I'm trying to scale up. The brine one is producing 300mb/tick and the lithium one is producing 30mb/tick as expected feeding into a typical 1x decondenstrator / 7x SNA (un-upgraded) / Reactor setup.
By my understanding I should be dramatically overproducing both brine and lithium for my current rate 2 reactor, but I'm not building up a backlog of brine at all. I'm making 300mb/tick, 300mb/tick is going into the lithium producing tower, and 30 mb/tick of lithium is flowing out consistently.
The SNAs, decondenstrator, and all pipes "downstream" of the lithium tower are 100% full. However the brine tower is perpetually empty (as is the pipe connecting the two towers). I'm not aware of any other possible destinations for the lithium/brine on this network. I have eliminated the use of all other mods in this setup trying to troubleshoot, only Mekanism pipes are being used, etc.

I'm not sure I entirely understand your question...
The Lithium tower converts Brine into Lithium at a rate of 10:1, so it takes in 300mb of Brine and converts it into 30mb of Lithium.
Knowing this, you will require 10 full-size Brine towers (3,000mb) to match the Lithium tower, in order to produce 300mb of Lithium.
The other thing could be the Lithium might be stored in the pipes themselves, as they have their own storage capacity.

Not an issue. There's a 1/10 ratio from brine to lithium.

Hello Aidan,
I am sorry if I explained it poorly, but I think you misunderstood the situation I am experiencing.
The lithium tower and everything "past" it is completely full. The buffers on the pipes are full, the machines are full, and the tower is full. There is nowhere on my network for 30 mb/tick to go (I was trying to ramp up).
The issue is that the full 300 mb/tick of brine keeps getting consumed. I suspect what is happening is that the lithium tower is converting all of the bring to lithium per tick, and the excess is being lost/discarded somewhere. Is that intentional? My expectation is that both towers would have built up a backlog of brine.

I've looked into the issue again and it turns out I had forgotten to scale the production to the amount needed. Should be fixed now :)


Add maxFramePayloadLength and maxMessagePayloadLength parameters to MessageBuffer

This PR adds optional parameters to the MessageBuffer to allow for size limits on Frames and Messages.
This also sets a default limit for frame and message lengths to 1/4 of configured memory limits for PHP.
Frames and messages are detected almost as soon as the library can possibly detect them, which will allow for library users to mitigate some DoS issues.
Fixes #35.

Awesome work! 👍


[Feature] Thermostat cluster is missing a lot of events

Feature description
The thermostat cluster is missing many essential events.
Events for things like
a) call for cool, but not cooling
b) call for heat, but not heating
c) heat exceed limit (heat did not shut off)
d) cool exceed limit (cool did not shut off)
e) run time exceed (heat or cool running for hours with no effect)
f) maximum temp exceed (area got too hot)
g) minimum temp exceed (area got too cold, freeze)
h) filter lifetime exceeded
Platform
all
Platform Version(s)
No response
Anything else?
No response

More missing events:
Remote sensor X battery needs replacement
Communications lost to remote sensor X


mappo and mappo_ns dont run

Hey y'all,
I'm trying to replicate your results from you paper but am unfamiliar with torch. I encountered an error when trying to run mappo and mappo_ns:
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cpu and cuda:0! (when checking arugment for argument mat1 in method wrapper_addmm)
I have tested all the other algorithms and none of them have this error. I was wondering if you have encountered this error before and know of a way to force all the tensors to either go to the CPU or the GPU (preferably).
The system that I am running has a GPU with CUDA 11.4 and CuDNN 8.2 and is running Windows 10.
thanks for your attention,
Peter

Hi @at-peter
Thank you for spotting this. We believe this has been resolved with commit a16fd23b2fc9c1b76953375a36108149377d5468. Let us know if the issue persists for you.