Module parse failed: Unexpected character '�' (1:0)

Hi,
I am having the following issue while trying to make sharp work on my local dev environment:
[Vue warn]: Error in v-on handler: "Error:
Something went wrong installing the "sharp" module

Module parse failed: Unexpected character '�' (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
(Source code omitted for this binary file)

- Remove the "node_modules/sharp" directory then run
  "npm install --ignore-scripts=false --verbose" and look for errors
- Consult the installation documentation at https://sharp.pixelplumbing.com/install
- Search for this error at https://github.com/lovell/sharp/issues
"

found in

---> <BFormFile>
       <AccountSummaryView> at src/components/Account/AccountSummaryView.vue
         <MyAccount> at src/views/MyAccount.vue
           <App> at src/App.vue
             <Root>

My code looks like this (using vue, btw, but i dont't think it can affect the output)  :
// previous code ...

methods: {
    uploadimage() {
      console.log("uploadimage method called")
      const sharp = require("sharp");
      sharp(this.profileimgfile)
        .resize({
          width: 300,
          height: 300,
        })
        .webp()
        .toBuffer()
        .then((img) => {
          this.$store.dispatch("modifyuserprofilepic", img);
        })
        .catch((err) => {
          console.log(err);
        });
    },
  },

// more code ...

Did you see the documentation relating to installation?

Yes, but no information there about this issue.

Have you ensured the architecture and platform of Node.js used for npm install is the same as the architecture and platform of Node.js used at runtime?

Local dev machine so I guess it's ok

Are you using the latest version? Is the version currently in use as reported by npm ls sharp the same as the latest version as reported by npm view sharp dist-tags.latest?

mario@mgmac boxy-pe % npm ls sharp
<EMAIL_ADDRESS>/Users/mario/Documents/CodeProjects/boxy-pe
└──<EMAIL_ADDRESS>

If you are installing as a root or sudo user, have you tried with the npm install --unsafe-perm flag?

Not using root

If you are using the ignore-scripts feature of npm, have you tried with the npm install --ignore-scripts=false flag?

Here's the output of that command:
mario@mgmac boxy-pe % rm -fr node_modules/sharp
mario@mgmac boxy-pe % npm install --ignore-scripts=false --verbose
npm info it worked if it ends with ok
npm verb cli [
npm verb cli   '/usr/local/bin/node',
npm verb cli   '/usr/local/bin/npm',
npm verb cli   'install',
npm verb cli   '--ignore-scripts=false',
npm verb cli   '--verbose'
npm verb cli ]
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm verb npm-session c0fc00b5699356fd
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm timing stage:loadCurrentTree Completed in 2276ms
npm timing stage:loadIdealTree:cloneCurrentTree Completed in 26ms
npm timing stage:loadIdealTree:loadShrinkwrap Completed in 1181ms
npm timing stage:loadIdealTree:loadAllDepsIntoIdealTree Completed in 1984ms
npm timing stage:loadIdealTree Completed in 3628ms
npm timing stage:generateActionsToTake Completed in 200ms
npm verb correctMkdir /Users/mario/.npm/_locks correctMkdir not in flight; initializing
npm verb lock using /Users/mario/.npm/_locks/staging-7dfe15593b271ae3.lock for /Users/mario/Documents/CodeProjects/boxy-pe/node_modules/.staging
npm timing action:extract Completed in 222ms
npm timing action:finalize Completed in 15ms
npm timing action:refresh-package-json Completed in 59ms
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm timing action:preinstall Completed in 117ms
npm info linkStuff<EMAIL_ADDRESS>npm verb linkBins [
npm verb linkBins   { semver: 'bin/semver.js' },
npm verb linkBins   '/Users/mario/Documents/CodeProjects/boxy-pe/node_modules/sharp/node_modules/.bin',
npm verb linkBins   false
npm verb linkBins ]
npm info linkStuff<EMAIL_ADDRESS>npm timing action:build Completed in 27ms
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
><EMAIL_ADDRESS>install /Users/mario/Documents/CodeProjects/boxy-pe/node_modules/sharp
> (node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)

info sharp Using cached /Users/mario/.npm/_libvips/libvips-8.10.0-darwin-x64.tar.br
prebuild-install info begin Prebuild-install version 5.3.5
prebuild-install info looking for cached prebuild @ /Users/mario/.npm/_prebuilds/7f763b-sharp-v0.26.0-napi-v3-darwin-x64.tar.gz
prebuild-install info found cached prebuild
prebuild-install info unpacking @ /Users/mario/.npm/_prebuilds/7f763b-sharp-v0.26.0-napi-v3-darwin-x64.tar.gz
prebuild-install info unpack resolved to /Users/mario/Documents/CodeProjects/boxy-pe/node_modules/sharp/build/Release/sharp.node
prebuild-install info install Successfully installed prebuilt binary!
npm verb lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
npm verb lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/Users/mario/Documents/CodeProjects/boxy-pe/node_modules/sharp/node_modules/.bin:/Users/mario/Documents/CodeProjects/boxy-pe/node_modules/.bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/mysql/bin
npm verb lifecycle<EMAIL_ADDRESS>CWD: /Users/mario/Documents/CodeProjects/boxy-pe/node_modules/sharp
npm timing audit submit Completed in 2119ms
npm http fetch POST 200 https://registry.npmjs.org/-/npm/v1/security/audits/quick 2118ms
npm timing audit body Completed in 1ms
npm timing action:install Completed in 6552ms
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm timing action:postinstall Completed in 8ms
npm verb unlock done using /Users/mario/.npm/_locks/staging-7dfe15593b271ae3.lock for /Users/mario/Documents/CodeProjects/boxy-pe/node_modules/.staging
npm timing stage:executeActions Completed in 7095ms
npm timing stage:rollbackFailedOptional Completed in 0ms
npm info linkStuff<EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm timing stage:runTopLevelLifecycles Completed in 13678ms
npm verb saving []
npm verb shrinkwrap skipping write for package.json because there were no changes.
npm info lifecycle undefined~preshrinkwrap: undefined
npm info lifecycle undefined~shrinkwrap: undefined
npm verb shrinkwrap skipping write for package-lock.json because there were no changes.
npm info lifecycle undefined~postshrinkwrap: undefined
added 2 packages from 65 contributors and audited 1378 packages in 14.813s

53 packages are looking for funding
  run `npm fund` for details

found 1 high severity vulnerability
  run `npm audit fix` to fix them, or `npm audit` for details
npm verb exit [ 0, true ]
npm timing npm Completed in 16677ms
npm info ok

What is the complete output of running npm install --verbose sharp? Have you checked this output for useful error messages?

Maybe the same as above

What is the output of running npx envinfo --binaries --system?

mario@mgmac boxy-pe % npx envinfo --binaries --system
npx: instaló 1 en 3.631s

  System:
    OS: macOS 10.15.6
    CPU: (4) x64 Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz
    Memory: 295.73 MB / 8.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 12.14.0 - /usr/local/bin/node
    npm: 6.13.4 - /usr/local/bin/npm

Just to add more information, this is the warning I receive on compile:
WARNING  Compiled with 2 warnings                                                                                15:46:23

warning  in ./node_modules/sharp/lib/libvips.js

Critical dependency: the request of a dependency is an expression

warning  in ./node_modules/sharp/build/Release/sharp.node

Module parse failed: Unexpected character '�' (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
(Source code omitted for this binary file)

@ ./node_modules/sharp/lib/constructor.js 11:2-40
@ ./node_modules/sharp/lib/index.js
@ ./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader/lib??vue-loader-options!./src/components/Account/AccountSummaryView.vue?vue&type=script&lang=js&
@ ./src/components/Account/AccountSummaryView.vue?vue&type=script&lang=js&
@ ./src/components/Account/AccountSummaryView.vue
@ ./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader/lib??vue-loader-options!./src/views/MyAccount.vue?vue&type=script&lang=js&
@ ./src/views/MyAccount.vue?vue&type=script&lang=js&
@ ./src/views/MyAccount.vue
@ ./src/router/index.js
@ ./src/main.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080&sockPath=/sockjs-node (webpack)/hot/dev-server.js ./src/main.js

Hi, sharp requires the Node.js runtime and will not work in a browser.

Oh, ok, it would've been great to use sharp client-side (because I'm using firebase/aws s3 for storage and I don't really have the need to do some server-side processing) and the other libraries for doing this in the browser are not as good as this one or as well-documented as this is... thanks anyway for your response.
If there is a way to make this work in a browser (a workaround or something like that) please let me know.
Thanks anyway, please close this issue if you consider it appropriate.

Add to  webpack config
{
  externals: {
    sharp: 'commonjs sharp'
  }
}

Add to  webpack config
{
  externals: {
    sharp: 'commonjs sharp'
  }
}

To fix this problem when using Vue, add this to your vue.config.js:
configureWebpack: {     externals: {       sharp: "commonjs sharp",     },   },

Note adding it to webpack externals means that it won't be included in the bundle, which may not be what you want. If using node, you can include it with node-loader.
Install it: npm i -D node-loader
In webpack.config.js:
module: {
  rules: [
    { test: /\.ts$/, loader: "ts-loader" },
    { test: /\.node$/, use: "node-loader"}
  ]
}

(Source)

In my case I just had to add:
{ test: /\.node$/, use: "node-loader"}

https://sharp.pixelplumbing.com/install#bundlers


Separate const of ndarray from const of its data.

My understanding is that the following should succeed because a references a writable array, and it does not matter that a itself is const qualified.   I have immutable metadata, but the data of the actual array is mutable.
    m.def("write_should_succeed",
          [](const nb::ndarray<>& a) {
              if (a.dtype() == nb::dtype<double>()) {
                  auto ptr = static_cast<double*>(a.data());
                  *ptr = 7.0;
              }
          });

However, this does not compile because a.data() returns a pointer-to-const:
error: static_cast from 'const Scalar *' (aka 'const void *') to 'double *' casts away qualifiers
                  auto ptr = static_cast<double*>(a.data());
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Also, the following compiles, but it should not because the nb::ro should imply that a.data() returns a pointer-to-const:
    m.def("should_not_compile",
          [](nb::ndarray<nb::ro>&& a) {
              if (a.dtype() == nb::dtype<double>()) {
                  auto ptr = static_cast<double*>(a.data());
                  *ptr = 7.0;
              }
          });

If you agree, please consider this PR.
Also, this adds the constexpr static bool is_ro to ndarray.
I guess I just thought that would be nice....
If you prefer another name, or don't like it at all, that's fine too.

Hmmm, macos C++ compiler doesn't like static_assert(a.is_ro); in the tests.
Is there a mac-specific idiom that should be used?
If not, I can simply && this into the return value and let the python assert check it.

Hmmm, macos C++ compiler doesn't like static_assert(a.is_ro); in the tests. Is there a mac-specific idiom that should be used? If not, I can simply && this into the return value and let the python assert check it.

The problem is that you are accessing a constexpr static member through an instance that is not constexpr. See this discussion. As I understand, this is only valid in C++23 due to P2280R4

Thanks, Roman.   I'm now accessing the class static data member is_ro through the type of a
          [](nb::ndarray<nb::ro> a) {
              static_assert(decltype(a)::is_ro);

and, if a is a reference
          [](const nb::ndarray<nb::ro>& a) {
              static_assert(std::remove_reference_t<decltype(a)>::is_ro);

In general, I am open to this change (modulo naming tweaks). However, I am concerned a bit by the magnitude and complexity of the added tests. Is it really necessary to add that much replicated code to test this change? Could you create a single test function that is simply instantiated multiple times?

Certainly.   (I've had project managers that insisted on simple, repetitive, fall-through code in tests.)
Please advise (at your convenience) on naming preference, or if the boolean should be removed.
I was thinking it's useful for code like this:
template<typename... Ts>
std::ostream& operator<<(std::ostream& os, const nb::ndarray<Ts...>& a) {
    return os << "Array (" << ((a.is_ro) ? "read-only" : "writable")
              << ") at " << a.data();
}

Also, I was not sure what to do about auto v = a.view<double, nb::ndim<1>>(); if a is nb::ndarray<nb::ro>
Should the non-constness of the double override the read-only of a, or should the constness be "sticky"?
I probably won't be using views, so I don't have strong opinion.
I guess I'd probably go with sticky as more in line with the light-weight philosophy of views.   In other words, the user probably intended const double.   It's still possible to change a read-only ndarray to writable using
nb::ndarray<double> wa{a};  // wa is writable

I'll be away for a few days, so no hurry responding.

Since template parameter packs are recursively examined right to left, given nb::ndarray<float, nb::shape<2, 2>> a, the view a.view<nb::shape<4>>() is still 2x2.   This patch changes this so that the shape for the view becomes 4.  The implementation is that if the shape has already been set, ignore any more shapes that are encountered to its left.
I made it an assertion error to have two conflicting order or framework.   Note that changing from 'C' to 'F' doesn't transpose the 2D array because it doesn't change the strides.   So I thought it should be an error.
I made is_ro sticky.   So ndarray<const float> and ndarray<nb::ro, float> and ndarray<float, nb::ro> are all the same.
In particular, a read-only view of writable data can be obtained but not the reverse.
It would be easy to change the code if another choice is preferred.   The hard part is deciding on the best API from a user's perspective.

As a brainstorming exercise, I experimented some in PR #498

I re-based this upon the current HEAD (so as to manually resolve merge conflicts with recent commits) and force-pushed.

Sorry for leaving this dormant for a long time. I mainly just have two pieces of feedback.

Thank you!


Removed all usages of django.conf.urls.patterns

This is a clean PR which can replace #721.

@jsma Thanks for your contribution. As https://github.com/divio/django-filer/pull/721 is now completed, I'll close this as duplicate given that the former was first.


Feat.module controller v2

Summary by CodeRabbit

New Features

Introduced comprehensive documentation for ModuleController V2, including architecture, operational procedures, deployment methods, and scaling strategies.
Added tutorials on module management, such as information inquiry, online/offline procedures, and incompatible upgrades.
Provided detailed deployment and rollback guidelines for modules within Kubernetes environments.

Documentation

Enhanced technical documentation accessibility and organization for users and developers.
Added multilingual support with Chinese documentation for ModuleController V2.

等待代码和版本发布后合并


Obtaining Transform position using getLookAt

Hello,
Is there a way to obtain the transform position of the point being looked at via the head tracker?
I was looking into getLookAt but it only gives the vector.

Hi @sam16222 I think you are asking about getting the geometry/object at the center of the screen?
If so that's a ray casting problem.

You can call GVRScene.getMainCameraRig().getHeadTransform() to get the camera view matrix. It is updated every frame when your head moves.

@sam16222 Do you need more help?

Hello. We had a holiday here so I wasn't able to try it out. I'll get back by EOD.

@liaxim I tried using what you suggested, but couldn't quite achieve what I wanted to. Here's what I want to do.
Take a spherical mesh. It is blank all around. I want to point at a position and tap. At that location I should be able to spawn an image or a text box. Can you think of a way to do this? I tried using getHitLocation() but it didn't quite work.

@sam16222 It didn't quite work in what way? Thanks.

It's just not giving me the exact position. As explained in the situation above, can you tell me a way to do it? I just want to look at a location, tap and then spawn a text box at the location, with the appropriate rotation values.

@sam16222
Let me know if the following is of use:
If you follow the example
https://github.com/gearvrf/GearVRf-Demos/blob/master/gvr-eyepicking/
You can modify the PickHandler's onInside() function and query the collision position:
public class PickHandler implements IPickEvents
{
public void onEnter(GVRSceneObject sceneObj, GVRPicker.GVRPickedObject pickInfo)
{
}
public void onExit(GVRSceneObject sceneObj)
{
}
public void onNoPick(GVRPicker picker) { }
public void onPick(GVRPicker picker) { }
public void onInside(GVRSceneObject sceneObj, GVRPicker.GVRPickedObject pickInfo) {
//This function is called whenever your looking direction intersects with an object.
//This is supposed to be the exact position you look for
float[] hitLocation = pickInfo.hitLocation;
//Do something about this position, such as spawning an image or text box
...
}
}

@sam16222


thesillyhome addon cloud - Model generation failed

thesillyhome addon cloud - v 0.0.1
Starting to parse DB data
INFO:root:<---Starting the parsing sequence--->
2023-02-04 16:32:35,320 - root - INFO - <---Starting the parsing sequence--->
INFO:root:Querying data from internal homeassistant db.
2023-02-04 16:32:35,321 - root - INFO - Querying data from internal homeassistant db.
ERROR:root:Uncaught exception
Traceback (most recent call last):
File "/usr/local/lib/python3.9/runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/local/lib/python3.9/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/thesillyhome_src/thesillyhome/src/thesillyhome/model_creator/main.py", line 22, in
homedb().send_data()
File "/thesillyhome_src/thesillyhome/src/thesillyhome/model_creator/parse_data.py", line 68, in send_data
new_last_updated_date = res.scalar_one().replace(tzinfo=timezone.utc)
AttributeError: 'NoneType' object has no attribute 'replace'
2023-02-04 16:32:36,667 - root - ERROR - Uncaught exception
Traceback (most recent call last):
File "/usr/local/lib/python3.9/runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/local/lib/python3.9/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/thesillyhome_src/thesillyhome/src/thesillyhome/model_creator/main.py", line 22, in
homedb().send_data()
File "/thesillyhome_src/thesillyhome/src/thesillyhome/model_creator/parse_data.py", line 68, in send_data
new_last_updated_date = res.scalar_one().replace(tzinfo=timezone.utc)
AttributeError: 'NoneType' object has no attribute 'replace'
Model generation failed.

Hi @simonepittis,
Thanks for raising this. The same issue was raised on discord. This is due to a new update on the HA schema.
Working on a fix.

This should be fixed. Feel free to retest.


Update to the latest anycloud deploy module, which un-DRYs some code but since it's only two places, it's okay

@dfellis looks good, but seems to be complaining about cargo:rerun-if-changed=alan/anycloud.ln https://github.com/alantech/alan/pull/420/checks?check_run_id=1944448313#step:3:435

@dfellis looks good, but seems to be complaining about cargo:rerun-if-changed=alan/anycloud.ln https://github.com/alantech/alan/pull/420/checks?check_run_id=1944448313#step:3:435

It worked locally because I have an alan application available for that build to work. :/ We'll have to do something bonkers and circular to get the build process to pass. :(


GPU / TPU memory requirements for training

In the paper it states that you're using the TPU v4 chips, which have 32 GB memory accessible per TPU core I believe. When trying to train the high res version (on nvidia GPU currently) I seem to use > 48 GB of VRAM for a batch size of 1 per GPU.
I'm using code in the example notebook with minimal modification.
Are there any other things that need to be done in order to get the memory useage down below 32GB for the high res model? I note that gradient checkpointing and bfloat16 are already setup in the example notebook in this function if I'm understanding it correctly?
def construct_wrapped_graphcast(model_config: graphcast.ModelConfig, task_config: graphcast.TaskConfig):
        # Deeper one-step predictor.
        predictor = graphcast.GraphCast(model_config, task_config)

        # Modify inputs/outputs to `graphcast.GraphCast` to handle conversion to
        # from/to float32 to/from BFloat16.
        predictor = casting.Bfloat16Cast(predictor)

        # Modify inputs/outputs to `casting.Bfloat16Cast` so the casting to/from
        # BFloat16 happens after applying normalization to the inputs/targets.
        predictor = normalization.InputsAndResiduals(
            predictor,
            diffs_stddev_by_level=diffs_stddev_by_level,
            mean_by_level=mean_by_level,
            stddev_by_level=stddev_by_level)

        # Wraps everything so the one-step model can produce trajectories.
        predictor = autoregressive.Predictor(predictor, gradient_checkpointing=True)
        return predictor

Do the model inputs (xarray datasets) need to also be cast to a different precision instead of float32?
I'm unsure how to get the memory usage down further so that it could be run on the TPU v4 or something like an 40 GB A100.

Hi Chris, I'm afraid it requires some further tricks not included in this codebase to train a 0.25deg model unrolled to 3 days within TPU RAM limits on TPUv4. The main ones being some additional gradient checkpointing, and offloading some gradient checkpoints from device to host RAM. Our code for this is more tied to internal infrastructure and so the scope for this initial open sourcing was mainly focused on supporting inference, and basic gradient computation for versions of the model that fit more easily into RAM.

Closing as no work imminently planned in this direction in the open source codebase, although the above might give you some hints if you wanted to roll your own.

Closing


add color

I don't know if this a planned feature or on your mind, but it sure would be great :D

Yep definitely want this.


Exoplayer issues

Hi,
i do have some issues which are probably not directly caused by the implementation, but maybe you have an idea? I placed the surface with align contents on the screen and play a video. The video is for a while always just on 1/4 of the screen before it updates to the display.
Some android devices are also unable to play mp4 (h264, aac) files which work fine on others devices. Same with audio streams like: http://c11.prod.playlists.ihrhls.com/1477/playlist.m3u8.
Is there a chance to figure out why this happens?
Christian

I should probably be more specific. I'm unable to play UDP protocols like udp://@<IP_ADDRESS>:5000. According to the log exoplayer complains that the protocol is unknown.

Hello, everything has been fixed in the latest committed version.


Restore last vmd session's window state on startup

Saves the state of the vmd window on save and then restores it when vmd is opened again.
Is a feature I wanted as the first thing I always do when calling vmd is move the window.

Hey George! That's an awesome feature :100: :bangbang:
I tried it out and I noticed that the electron-window-state module stores the window state in a JSON file in path.join(app.getPath('userData'), 'window-state.json') (see electron-window-state:index.js#L23) which turns out to be /home/max/.config/Electron/window-state.json. On Linux at least, probably similar on OS X. This means that it will clash with other Electron apps that use the same module and don't use a custom path. And I don't think Electron's config directory is the correct place to store this information.
vmd reads its own configs from the following locations: $HOME/.vmdrc, $HOME/.vmd/config, $HOME/.config/vmd, $HOME/.config/vmd/config, /etc/vmdrc, and a number of other places. So these can't be used either even if the window state was merged with existing settings as the format does not necessarily have to be JSON (vmd supports JSON, INI and YAML).
The easiest solution would probably be to set the file option in windowStateKeeper() to something like "vmd-window-state.json" but then the file would still be in Electron's config directory which is the wrong place IMO. Or maybe Electron can be configured so that app.getPath('userData') returns something app-specific.
Another option would be to store the file alongside vmd's config files but vmd may not always have it's own directory (e.g. ~/.vmdrc is not a directory). But It's not ideal either since this type of state is more cache rather than configuration.
The third option would be to handle window state preservation in the renderer process and store the state in localStorage. Then the information would be somewhere in ~/.cache (or similar) which is where one would expect persisted application state.
A killer bonus feature would be if window state preservation could be turned off using a configuration option. For example vmd --window-preservestate=false and window.preserverstate = false in the config file. Some people have external programs that manage the positioning of windows and this could interfere with them. It could be turned on by default.

Hi, I had a look into the localStorage option, but it seemed pretty involved and I don't really have heaps of time at the moment 😔
I changed the file option in windowStateKeeper() to "vmd-window-state.json" and added the window.preservestate = true default.
Hope this is all good, let me know if you want me to squash.

Yeah I guess that's best for now to get the feature in. Maybe it can be refactored later. I'll publish a new release in couple secs.
Thanks!

<EMAIL_ADDRESS>is on npm :tada:


Can't read transactionbuilderconfigbuilder_new in React Vite app

As I was instantiating Lucid.new(), I got an error.
Error log:

TypeError: Cannot read properties of undefined (reading 'transactionbuilderconfigbuilder_new')
at TransactionBuilderConfigBuilder.new (webpack-internal:///./node_modules/lucid-cardano/esm/src/core/libs/cardano_multiplatform_lib/cardano_multiplatform_lib.generated.js:18331:26)
at Lucid.new (webpack-internal:///./node_modules/lucid-cardano/esm/src/lucid/lucid.js:78:119)

tsconfig.json
  "compilerOptions": {
    "target": "ES2020",
    "useDefineForClassFields": true,
    "lib": ["ES2020", "DOM", "DOM.Iterable"],
    "module": "ESNext",
    "skipLibCheck": true,

    /* Bundler mode */
    "moduleResolution": "bundler",
    "allowImportingTsExtensions": true,
    "resolveJsonModule": true,
    "isolatedModules": true,
    "noEmit": true,
    "jsx": "react-jsx",

    /* Linting */
    "strict": true,
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "noFallthroughCasesInSwitch": true
  },
  "include": ["src"],
  "references": [{ "path": "./tsconfig.node.json" }]
}

vite.config.ts
import { defineConfig } from "vite";
import react from "@vitejs/plugin-react";

export default defineConfig({
  optimizeDeps: {
    include: ["pdfjs-dist"], // optionally specify dependency name
    esbuildOptions: {
      supported: {
        "top-level-await": true,
      },
    },
  },
  plugins: [react()],
  build: {
    target: "esnext",
  },
  server: {
    host: "<IP_ADDRESS>",
  },
});

package.json
 "@types/node": "^16.18.37"
 "lucid-cardano": "^0.10.6",
 "react": "^18.2.0",
 "vite": "^5.2.0"

Appreciate any help.

You seem to run the Lucid.new() method using lucid-cardano. I started my Cardano blockchain dev career 1 year ago. We need to enable some properties to fix the issue.
I did that with craco.config.js by adding the following properties.
asyncWebAssembly: true,
lazyCompilation: false,
syncWebAssembly: true,
topLevelAwait: true

You have to do the same thing for lucid to work in vite app.

Thank you for your reply. I tried to use webpack loaders in vite, but it's failed.

You can't use webpack loaders in vite app mostly, you need to use vite plugins instead. You can also think about local dev server migration if you have to use lucid-cardano.

I found some links and tried them. Vite app still gives me an error, but it's another problem. This issue is solved.


Fix "Fork, Conditional Jump, Encodings" test and fix the "Return" operation.

The return operation was setting state.progress to the length of the recipe.
Under async this is incorrect behaviour because after any flow control operation the Recipe executes state.progress + 1, and then at the beginning of the recursed Recipe.execute checks the length and returns if currentStep === recipe.opList.length
Return now sets state.progress equal to state.opList.length - 1.
This bug could instead have been fixed by changing finish condition from currentStep === recipe.opList.length to currentStep >= recipe.opList.length to prevent this class of bugs in the future. I'll leave that decision to @n1474335.
Additionally there was a type in the test "Fork, Conditional Jump, Encodings" which was just a missing newline in the input which was required to match the output.
Screenshot of tests passing

Good spot. I changed currentStep === recipe.opList.length to currentStep >= recipe.opList.length as it seems like a good defensive measure and I can't think of any scenario where it wouldn't be wanted. If such a scenario presents itself in future we can always change it again.


issue #773 reopen

Is there a solution for #773 somewhere ?
Is that been released?

This is kind of duplicated with #773. I will close this. Can we track issue with #773?


Use frame buffer list instead of the fundamental one

Using the fundamental buffer list conflates buffer-visiting activity
across all frames, and makes things more confusing when using multiple
frames to help with context switching.
By using frame buffer lists, visiting buffers in some other frame does
not affect the order in which they are proposed in this frame.

Applied, thanks.


Add plan/diff command

A plan or diff-current-to-next command would be useful to have.
We can get something pretty close already by combining colmena and nix-diff:
$ colmena build --on laptop
[INFO ] Using configuration: ./deployment.nix
[INFO ] Enumerating nodes...
[INFO ] Selected 1 out of 5 hosts.
       ✅ 11s All done!
laptop ✅ 10s Evaluated laptop
laptop ✅ 1s Built "/nix/store/r829pyjngqxmn7k1j9j39w3rcaarg2d1-nixos-system-laptop-21.11pre-git"

$ nix-diff --color always --character-oriented /var/run/current-system /nix/store/r829pyjngqxmn7k1j9j39w3rcaarg2d1-nixos-system-laptop-21.11pre-git | less -R

There are a few problems with the above:

nix-diff is a separate package to install,
the diff isn't great (the full diff contains the entirety of options.xml and is 26MB in size); that said, i think that if we diffed just a few specific things like system-path, that would be useful,
the diff requires the previous system derivation to be available locally.

If you're interested in a feature like this, I'd be up for rewriting nix-diff as a rust library and trying to integrate it into colmena.

Nix itself has a related command: https://nixos.org/manual/nix/stable/command-ref/new-cli/nix3-profile-diff-closures.html

That is a very neat command.
On my system, I get something like this:
$ nix profile diff-closures --profile /nix/var/nix/profiles/system
...
Version 123 -> 124:

Version 124 -> 125:

Version 125 -> 126:
  chromium: 98.0.4758.102 → 99.0.4844.51
  chromium-unwrapped: 98.0.4758.102 → 99.0.4844.51, -2113.2 KiB
  cpupower: 5.10.102 → 5.10.103
  firefox: 97.0.1 → 97.0.2
  firefox-unwrapped: 97.0.1 → 97.0.2, -1489.6 KiB
  initrd-linux: 5.10.102 → 5.10.103
  linux: 5.10.102, 5.10.102-modules → 5.10.103, 5.10.103-modules, +18.4 KiB
  nix-diff: ∅ → 1.0.16, +2257.7 KiB
  signal-desktop: 5.31.1 → 5.33.0, +12308.2 KiB
  thunderbird: 91.6.1 → 91.6.2
  thunderbird-unwrapped: 91.6.1 → 91.6.2, +8.5 KiB
  zfs-kernel: 2.1.2-5.10.102 → 2.1.2-5.10.103

That's very user-friendly, but it doesn't show some changes.  The 124 -> 125 update was a change to networking.hosts, but it shows up as empty in the diff-closures output.
For my tastes, I think nix-diff shows too much output, but nix profile dif-closures shows too little output.

nix-diff would benefit greatly from a semantic differ, not only on the nix level, but also on the major config language levels, with json & xml as a minimum.
When using nix-diff that was the biggest pain point.
In terms of evaluation, maybe https://github.com//kamadorueda/nixel is useful.
In terms of semantic differ, for json I only know nixpkgs#jd-diff, which iirc is written on Go. On the xml side maybe there are a couple of libraries laying around.

With NixOps, I was using:

With NixOps, I was using:
However, the activation output is hidden by Colmena.

I am using an equivalent snippet. You must run Colmena with the -v flag to get activation output, e.g. colmena apply --on foobar -v. Although a flag to make it verbose during activation but not build would be cool I guess!


Userspace read blocks timer interrupts

While doing a serial read from a user program, the timer interrupt seems to block.

The default options I used for interrupt handlers disabled interrupts while the handler is running. This should have been changed for the syscall interrupt.
Fixed by commit: 3864781


Flaky Relay test

https://github.com/firezone/firezone/actions/runs/8152130288/job/22281106356?pr=3956#step:5:1289

Can't reproduce without a backtrace unfortunately. I opened https://github.com/firezone/firezone/pull/4137 so in case this ever happens again, we can look into it.

And we have one!
https://github.com/firezone/firezone/actions/runs/8289707456/job/22686594463?pr=4156


template-ocean-eez hard-coded to use fsm eez

SizeCard.tsx is hardcoded to use fsm's eez area as denominator value.

Resolved via #208


nlohmann/json_fwd.hpp: No such file or directory

I cloned the repository with its dependencies, and built the lib:
git clone --recursive git@github.com:zeroxs/aegis.cpp
cd aegis.cpp
mkdir build
cd build
cmake ..
make
sudo make install

No error until here.
Then I created a little project, with a single main.cpp file, containing the template code given in the Readme - in which I took care to replace "TOKEN" with my bot's token, obviously.
When compiling it with
g++ -o bot src/main.cpp -Wall -Wextra -Werror

I get an error:
/usr/local/include/aegis/fwd.hpp:12:10: fatal error: nlohmann/json_fwd.hpp: No such file or directory

I'm investigating on this, looking for some lib I wouldn't have installed on my PC.

You're missing all the libs being linked, all the include directories, everything.

You're missing all the libs being linked, all the include directories, everything.

You mean I have to install all libs ? I didn't understand it that way, as they are downloaded with the repository.

No. Right now, you are calling g++ and telling it to only compile the main.cpp. Nothing more. What about the json lib? g++ doesn't know where it is. zlib, openssl, asio, etc. You're missing all the parameters for g++. I would highly recommend using cmake as it does it all for you.

Gosh I didn't realise how much I could forget in a few months of not coding C++...


Update GH action for pushing the Docker image

This action is using the old name of Redocly CLI and is deprecated itself. We should replace with an official Docker action:
https://github.com/crazy-max/ghaction-docker-buildx#moved-to-docker-organization
Additional context
Openapi CLI had been renamed into Redocly CLI in scope of this ticket: https://github.com/Redocly/redocly-cli/pull/672

We also need to update our docs.

@tatomyr Do you think we covered everything here in the recent docker work?


calling dismissAllModals shows animation if there are no modals

small bug
if we don't have any modals shown, we shouldn't show the dismiss animation

done. available in version 2.1.1.


Improve fetch-build-scan-data-cmdline-tool startup perf by setting Picocli sys prop

Closure support is only useful when using Picocli with Groovy. Disabling
support for closures reportedly improves startup performance. See
the Picocli 4.7.0 release notes.

I tried to test this by running 100 invocations of --help and comparing the before and after results:

Each row is the total time in seconds it took to run 100 invocations of --help.
There seems to be no noticeable difference, but I guess it doesn't hurt to merge this anyway.

We have decided to close this PR as there seems to be no noticeable performance gains by setting the picocli.disable.closures system property to true.


[Minor] Advanced tokens in footer component

New advanced tokens in footer component.

Changes
I would do the following changes:

socialIconSeparation for socialIconsGutter

Regarding fontSizeBase
This token shouldn't be available, any font-size property should use the root font-size the browser gives. Then relative units can be changed on-demand element by element (e.g. bottomLinksFontSize, copyrightFontSize...) or even use fixed units if wanted.
Suggestions
Regarding the divider of the bottom links
In order to keep consistency between all the divider tokens and with the bottomLink.... ones:

lineThickness to bottomLinksDividerThickness
lineStyle to bottomLinksDividerStyle
lineLinksSeparation to bottomLinksDividerSpacing
lineColor to bottomLinksDividerColor

Regarding: fontColor
Could be interesting to have separate color properties for custom content, links, and copyright as we have for other font properties?
Regarding customContentMinHeight
It's possible to play with the padding prop to increase the breathing space the custom content has. And to reach 15px a font size of 10px needs to be used. I'm not really confident leaving open space to edit min/max-widths/heights, but I understand it can be used to create more breathing space when there's not custom content.


Fix authentication issue on ci image job

@spikymonkey mind looking at this when you have a minute? It's not high priority. See the pipeline for more details.


Make changes in the admin tab immediately

Adding or removing members in the admin tab should be performed immediately. Currently you need to:

Add or remove a user
Press the update button

Most other solutions directly act on e.g the "add" button.

fixed in #154


ledger: 3.2.1 -> 3.3.0

Description of changes

https://github.com/ledger/ledger/releases/tag/v3.3.0
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thanks for merging, @marsam 🙏 Very much looking forward to a current version of ledger installed via nixpkgs on my workstation 😄


Window scroll fails with MDL as well as some other cases where window.pageYOffset is always 0

Ability to take a DOM element (or selector or something) that can be used for calculating scroll.
With MDL, document.getElementsByTagName('main')[0].scrollTop worked.

Not familiar with MDL, but I see your point.  Would you mind giving a more detailed example of your use case?  A plnkr or fiddle would be great.
Thanks!

Go to page https://getmdl.io/ and open developer console. Run window.pageYOffset. It will always return 0. This will work just fine on yahoo.com giving correct values. This 0 behavior is because of the following css in material.css
.mdl-layout {
display: flex;
flex-direction: column;
height: 100%;
overflow-x: hidden;
overflow-y: auto;
position: relative;
width: 100%;
}

Gotcha.  Thanks for the example.  There they don't scroll the window, the scroll an element, which you can also do with redux-infinite-scroll.
From our Readme:

Important If you are using an element as the scrollable component, rather than the window, you MUST pass a fixed height to the containerHeight prop.

By default, elementIsScrollable is set to true, so it's assumed you aren't listening to the window.
I'm closing this, but feel free to ask more questions if you need to.

Wait, I thought scrolling element only works with the element that infinite scroll makes i.e. "holder" element or the div above it. Not a top level div. How do u point the redux infinite scroll to listen to the scroll event of the top scrolling element.


Add dependency of dovecot.

Signed-off-by: Petr "Stone" Hracek<EMAIL_ADDRESS>

rebased and merged via CLI


fix: bug with AIM server availability check

Description of the change
Fixes the bug in https://github.com/foundation-model-stack/fms-hf-tuning/issues/131
Related issue number
https://github.com/foundation-model-stack/fms-hf-tuning/issues/131
How to verify the PR
Run an end-to-end test using https://github.com/foundation-model-stack/fms-hf-tuning/blob/main/tuning/sft_trainer.py in a venv or conda environment which has the AIM package installed. Even without an AIM server running the training should succeed.
Was the PR tested

[x] I have added >=1 unit test(s) for every new method I have added.
[x] I have ensured all unit tests pass

@HarikrishnanBalagopal I believe we do not need so much changes to the code. The idea is if Aim is installed but AIMSTACK_DB and AIMSTACK_SERVER are not set then we should not use Aim.
Just change this one line
  if aim_server:
        aim_callback = AimCallback(
            repo="aim://" + aim_server + "/", experiment=aim_experiment
        )
    if aim_db:
        aim_callback = AimCallback(repo=aim_db, experiment=aim_experiment)
   else:
-       aim_callback = AimCallback(experiment=aim_experiment)
+       aim_callback = None

@HarikrishnanBalagopal I believe we do not need so much changes to the code. The idea is if Aim is installed but AIMSTACK_DB and AIMSTACK_SERVER are not set then we should not use Aim.
Just change this one line
  if aim_server:
        aim_callback = AimCallback(
            repo="aim://" + aim_server + "/", experiment=aim_experiment
        )
    if aim_db:
        aim_callback = AimCallback(repo=aim_db, experiment=aim_experiment)
   else:
-       aim_callback = AimCallback(experiment=aim_experiment)
+       aim_callback = None

I think that will also require a change here to avoid adding Nones into the list of callbacks https://github.com/foundation-model-stack/fms-hf-tuning/blob/c2f2f8c2179f38f1754d621d4bb13cd4c1eaca52/tuning/sft_trainer.py#L247

Wait for #89 which fixes this.

Wait for #89 which fixes this.

Sure, I have made the change in this PR as well.

if this is in #89 perhaps we can close this PR

@HarikrishnanBalagopal If #89 fixes this issue, lets close this PR. Thanks @anhuong !


Request for the addition of Santali language (sat) plural form with addition of new language lem

Hi, i was translating document from English to Santali language and found some missing things, instead of making changes in code and creating pull req i thought to discuss, what can be done for this issue.
There is missing plural form for Santali language where, It's plural form should be:

Plural-Forms: nplurals=3; plural=n==1 ? 0 : n==2 ? 1 : 2

This plural form is also used in Launchpad (https://translations.launchpad.net/+languages/sat) and Ponoon (Mozilla - https://pontoon.mozilla.org/sat/)
I have been translating Santali language using Ol Chiki script (Olck), but always having problem creating a new translated file from pot files as always i have to edit po file using Notepad and change langauge tag to sat-Olck-IN (where sat is the code of langauge, Olck is the script, and IN is for India).
Is it possible to give an option for these language form below for Santali language while creating a translated po file from pot, it will save mine and others time for just doing same repetative task in Notepad by changing name tags. These are mostly used language tags while translating in Santali language.
sat for Santali language.
sat-Olck for Santali language with Ol Chiki script.
sat-Olck-IN for Santali language with Ol Chiki script in India.
sat-Deva for Santali language with Devanagari script.
sat-Deva-IN for Santali language with Devanagari script in India.
Ref of Santali language written in 5 script but with primary as Ol Chiki (Olck) - https://unicode-org.github.io/cldr-staging/charts/37/supplemental/languages_and_scripts.html

"sat-Olck-IN" is not a valid locale identifier.
I don't know why you say you have to edit the file manually, you can enter any language code as well as any plural forms expression.
Plural forms expressions are regularly updated from CLDR data. I'm not going to maintain a separate database of plural form expressions - been there, done that, didn't work well.
Finally, this issue seems to be mixing at least 3 unrelated things (one of which is in subject only, with no discussion), so I'm not ever sure what action it asks for.

sorry for replying late, actually I was a new to Poedit and was doing Voodoo programming when I first opened the issue. Now I have read the documentation and clear that how to enter any language code as well as any plural forms expression and how to deal with it. I think we should close this issue for without any reason and unclear.


pb_msg

Is it possible to release pb_msg as a standalone package for ROS?

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


[INVESTIGATE] Find Potential Alternatives to Jackson

Is your feature request related to a problem?
Currently we use Jackson to map the configuration map into a string for the ingest pipeline step. The configuration map can be completely arbitrary and we just want to pass this along as a string to the create ingest or search pipeline actions. Jackson has been a root cause for a good amount of CVEs, opening this a place to comment on future options to use instead of Jackson.

Consider JsonB.  I think that's what the Opensearch Java client uses.
https://javaee.github.io/javaee-spec/javadocs/javax/json/bind/Jsonb.html

implementation 'jakarta.json.bind:jakarta.json.bind-api:2.0.1'

import jakarta.json.bind.Jsonb;
import jakarta.json.bind.JsonbBuilder;
import java.util.Map;

public class ParseUtils {
    public static String mapToJson(Map<String, Object> map) {
        try (Jsonb jsonb = JsonbBuilder.create()) {
          return jsonb.toJson(map);
        }
    }
}

In the interest of completeness, I also looked at:

org.json:  Recent CVEs. https://ossindex.sonatype.org/component/pkg:maven/org.json/json
JSON.simple: Last release 2012, no CVEs<EMAIL_ADDRESS>Moshi: Active development. No CVEs. https://ossindex.sonatype.org/component/pkg:maven/com.squareup.moshi/moshi
GSON: Last CVE 2021 https://security.snyk.io/package/maven/com.google.code.gson:gson
Jackson: Recent CVE's: https://security.snyk.io/package/maven/com.fasterxml.jackson.core:jackson-databind

Among all the packages, JsonB (formerly part of Java EE), JSON.simple (old), and Moshi are the only ones without CVEs in the last 3 years.  I'd reject JSON.simple as abandonware.  Given Java client uses JsonB it's my strong preference.

In the interest of completeness, I also looked at:

org.json:  Recent CVEs. https://ossindex.sonatype.org/component/pkg:maven/org.json/json
JSON.simple: Last release 2012, no CVEs<EMAIL_ADDRESS>Moshi: Active development. No CVEs. https://ossindex.sonatype.org/component/pkg:maven/com.squareup.moshi/moshi
GSON: Last CVE 2021 https://security.snyk.io/package/maven/com.google.code.gson:gson
Jackson: Recent CVE's: https://security.snyk.io/package/maven/com.fasterxml.jackson.core:jackson-databind

Among all the packages, JsonB (formerly part of Java EE), JSON.simple (old), and Moshi are the only ones without CVEs in the last 3 years. I'd reject JSON.simple as abandonware. Given Java client uses JsonB it's my strong preference.

Thank you for the investigation here! I'll take a look at these, I also think using a package that is used in core/clients/other plugins could help since more people will be working on addressing any problems, especially if core is just in charge of upgrading version of the package and we take it downstream

Completed in #586


Please update npm version with the latest commit

My apologies for missing this. The driver has been updated to v1.0.7 and the code here and at npm has been updated.


Source Code Understand

It's a good project,I'm reading the code now.
The f1.ts,f2.ts,until f8.ts is the spritesheet of character.
But What's does the meaning of p1.ts,p2.ts,p3.ts,player.ts?

This is mostly historical. p* used a different png file file. We've since
moved over just to 32x32folk.png which is what f* uses.
On Wed, Oct 18, 2023 at 1:48 AM For @.***> wrote:

[image: image]
https://user-images.githubusercontent.com/56914169/276179041-a5aea068-8078-481d-add7-5d91102764ba.png
It's a good project,I'm reading the code now.
The f1.ts,f2.ts,until f8.ts is the spritesheet of character.
But What's does the meaning of p1.ts,p2.ts,p3.ts,player.ts?
—
Reply to this email directly, view it on GitHub
https://github.com/a16z-infra/ai-town/issues/160, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWEFGBRHYCQK2UMRCYXGIDX76JVRAVCNFSM6AAAAAA6FEFZOKVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2DSMRQG44DONI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This is mostly historical. p* used a different png file file. We've since moved over just to 32x32folk.png which is what f* uses.
…
On Wed, Oct 18, 2023 at 1:48 AM For @.> wrote: [image: image] https://user-images.githubusercontent.com/56914169/276179041-a5aea068-8078-481d-add7-5d91102764ba.png It's a good project,I'm reading the code now. The f1.ts,f2.ts,until f8.ts is the spritesheet of character. But What's does the meaning of p1.ts,p2.ts,p3.ts,player.ts? — Reply to this email directly, view it on GitHub <#160>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACWEFGBRHYCQK2UMRCYXGIDX76JVRAVCNFSM6AAAAAA6FEFZOKVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2DSMRQG44DONI . You are receiving this because you are subscribed to this thread.Message ID: @.>

Thank you, bro!
I also have a question.
How to change the background of this game?
I have found tilemap.json,firstmap.ts,rpg-tileset.png
So which is the map?

On Wed, Oct 18, 2023 at 11:21 PM For @.***> wrote:

This is mostly historical. p* used a different png file file. We've since
moved over just to 32x32folk.png which is what f* uses.
… <#m_-8122984999735306677_>
On Wed, Oct 18, 2023 at 1:48 AM For @.> wrote: [image: image]
https://user-images.githubusercontent.com/56914169/276179041-a5aea068-8078-481d-add7-5d91102764ba.png
https://user-images.githubusercontent.com/56914169/276179041-a5aea068-8078-481d-add7-5d91102764ba.png
It's a good project,I'm reading the code now. The f1.ts,f2.ts,until f8.ts
is the spritesheet of character. But What's does the meaning of
p1.ts,p2.ts,p3.ts,player.ts? — Reply to this email directly, view it on
GitHub <#160 https://github.com/a16z-infra/ai-town/issues/160>, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWEFGBRHYCQK2UMRCYXGIDX76JVRAVCNFSM6AAAAAA6FEFZOKVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2DSMRQG44DONI
https://github.com/notifications/unsubscribe-auth/ACWEFGBRHYCQK2UMRCYXGIDX76JVRAVCNFSM6AAAAAA6FEFZOKVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2DSMRQG44DONI
. You are receiving this because you are subscribed to this thread.Message
ID: @.>
Thank you, bro!
I also have a question.
How to change the background of this game?
I have found tilemap.json,firstmap.ts,rpg-tileset.png
So which is the map?

firstmap.ts is the map. The tileset is in public/assets/.
If you check out the branch martin-editor, there is a much more
sophisticated default map and a few others to play with. Also a level
editor.
.m

—
Reply to this email directly, view it on GitHub
https://github.com/a16z-infra/ai-town/issues/160#issuecomment-1770137201,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWEFGBOBXIEBVKQIIK4DC3YADBHJAVCNFSM6AAAAAA6FEFZOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZQGEZTOMRQGE
.
You are receiving this because you commented.Message ID:
@.***>

thanks for your answer!

Bro, how to control the talk frequence by code? I have seen the converse.js,maybe use this?


Combine Cases Macro doesn't check simulation timesteps

Describe the bug
The combine cases macro will gleefully combine cases with different weather file timesteps, leading to odd phenomena like PV generating at night.
To Reproduce
Steps to reproduce the behavior:

Set up a SAM case with two PV cases and a generic system
Give one PV case an hourly weather file, and another case a subhourly file
Set up a generic system to combine the cases, and run
See PV generating at night!

Expected behavior
The combine cases macro should at least throw an error if a user tries to do this, or maybe if we have lots of time interpolate to a consistent timestep.

Official vote to describe more of our bugs as "gleeful"! 😝


cleanup event handlers and expose props more consistently

I've done some generic cleanup.  I found that using MainButton standalone is a pretty smooth experience, so I've rename onToggleMenu to just onClick.  I also noticed some warnings when using with React 0.13

Thanks, published as 0.2.0


can i add click event on line

hellow.
i use leaderline.js in my project ,now i need to add a click event on the line to know which line i click.
but in the document i dont see any relative info.

Hi @WRZack, thank you for the comment.
This may help you: https://github.com/anseki/leader-line/issues/21
Or
https://github.com/anseki/leader-line/issues?q=is%3Aissue+event+listener
Note that it is not recommended, and it is not supported.

hi @anseki thank you for your reply.
i see #70 and i try to use code in my project like https://jsfiddle.net/g4ksLh9v/
but it dont work ,the following is part of my code
const line = new LeaderLine(
document.getElementById(startBoxId),
document.getElementById(endBoxId),
{
color: '#8F97A0',
size: 4,
path: 'grid',
startSocket: 'right',
endSocket: 'left',
startSocketGravity: 20,
endSocketGravity: 20
}
)
const l = document.querySelector('.leader-line:last-of-type')
l.addEventListener('dblclick', () => {
line.remove();
}, false);

I copied your code to JSFiddle instead of you.
https://jsfiddle.net/yh1afr08/
It seems that it works fine.

@anseki thank you for your help.
but i doesnt work in my project ,my project is a vue project and i try to add zindex on the svg,but it also doesnt work.

Sorry, my English is poor.
What does the "corrent" mean?
Also, could you show me an example via JSFiddle?

@anseki sorry for my bad spelling，i want say 'correct '.i set zindex on the svg and the svg get style zindex what i set .and it work in correctly in JSFiddle https://jsfiddle.net/e47uoyqm/4/ ,i think maybe it's about my framework.
Now I've changed my way to achieve my function instead of click event.
But I still want to thank you for your help

Yes, as I said repeatedly, the UI with clicking line is not good idea.
I think that your changing the way is good. :smile:
So, could you close this issue if it was solved?

No reply came, then this abandoned issue is closed.


svc.yaml: set type:ClusterIP to avoid repeated updates

We compare the type on apply. It is defaulted to ClusterIP on the server, but is empty in the operator manifest.

/lgtm


Add new languages: sw, wo, ff, ar. Bug 1151944

Marked as not ready, waiting on L10n to catch up

Do other repos like webpay, marketplace-stats, fireplace etc need these?

fireplace reads it from zamboni (as a data attribute).  I kind of think (hope?) the others do too :)

we're doing these one at a time, so closing this


TypeError: init() got an unexpected keyword argument 'server_hostname'

I'm creating this as an issue because I figured it's worth us documenting this.
I'm currently using:

Python 2.7.11 (python --version)
Locust 0.7.4 (locust --version)
Gevent 1.1.1 (pip show gevent)

All guides I read pointed to an incompatibility with Python 2.7.8+ and Gevent, so the recommended course of action was to downgrade Python to a time prior to this change. We spent around 4 hours trying to get a lower version of Python to work (with pyenv and virtualenv), until we realised that with a lower version, pip would be broken and not allow us to install packages.
I then investigated further and realised that gevent has since been repaired, but we need the latest version.
The ticket for me was:

Remove all trace of pyenv and virtualenv.
Upgrade Python to the latest - by running brew install python
Install Locust sudo CFLAGS='-std=c99' pip install locustio
Upgrade Gevent to the latest version pip install gevent==1.1.1

I bumped the gevent version to 1.1.1, and released a new minor version of Locust today (0.7.5). Would that fix the issue, or is there still a compatibility issue between python 2.7.8 and the gevent 1.1.1?

Upgrading gevent should be good enough; ultimately, that's what fixed the issue for us.
Thanks!

Great, thanks!


fix lint 🙈

When I rebased and merged https://github.com/pymc-labs/pymc-marketing/pull/138 the lint problem persists. I hope this fix it :)

Is your pylint more strict than ours/CI? Mine is not deleting single empty lines. Maybe it's a new version?

We are not using pylint right? In the make file:
check_lint:
	pip install -r lint-requirements.txt
	flake8 .
	isort --check-only .
	black --diff --check --fast .

😄

I meant whatever lint was in pre-commit, just defaulted to thinking it was pylint

I created a conda env from scratch and it seems to be running in the ci/cd as well. One thing I have noticed is that we are not specifying the revisions (versions) of the lint-packages.

Yeah I suspect it's a new behavior

We should treat the CI as the ground truth though, otherwise there will be inconsistencies soon enough again

I will create an issue


Clarify that duplicate only copies exported members and fails with a constructor

These are some quirks about Resource.duplicate I wish I knew before I used it. If one or both of these behaviors are a bug, I'll modify the PR and open an issue.
I can't decide what the most helpful and informative sentence is, so here are some I came up with:
More concise, but also vague about if it works with default values:

Fails if the constructor has parameters without default values.

Probably more helpful, but long (this is what the PR uses):

Copies correctly if there is no constructor or the constructor has default values, otherwise it fails.

What do you mean exactly by constructor in this context? Do you have examples of situations where duplicate doesn't work properly?

What do you mean exactly by constructor in this context? Do you have examples of situations where duplicate doesn't work properly?

Sorry, I should've included my tests:
extends Node2D

class Player extends Resource:
	export var health = 10
	var state = "running"
	func _init():
		pass

class PlayerWithConstructor extends Resource:
	export var health = 10
	var state = "running"
	func _init(_health):
		health = _health

class PlayerWithDefault extends Resource:
	export var health = 10
	var state = "running"
	func _init(_health = 10):
		health = _health

func _ready():
	var player = Player.new()
	player.state = "idle"
	print(player.duplicate().state) # prints "running", non-exported variables aren't duplicated
	PlayerWithDefault.new(3).duplicate() # works
	PlayerWithConstructor.new(3).duplicate()
	# fails with this error:
	# _create_instance: Condition "r_error.error != Variant::CallError::CALL_OK" is true. Returned: __null
	# <C++ Source>  modules/gdscript/gdscript.cpp:127 @ _create_instance()
	# <Stack Trace> Node2D.gd:19 @ _ready()

Sorry, I should've included my tests:
extends Node2D

class Player extends Resource:
	export var health = 10
	var state = "running"
	func _init():
		pass

class PlayerWithConstructor extends Resource:
	export var health = 10
	var state = "running"
	func _init(_health):
		health = _health

class PlayerWithDefault extends Resource:
	export var health = 10
	var state = "running"
	func _init(_health = 10):
		health = _health

func _ready():
	var player = Player.new()
	player.state = "idle"
	print(player.duplicate().state) # prints "running", non-exported variables aren't duplicated
	PlayerWithDefault.new(3).duplicate() # works
	PlayerWithConstructor.new(3).duplicate()
	# fails with this error:
	# _create_instance: Condition "r_error.error != Variant::CallError::CALL_OK" is true. Returned: __null
	# <C++ Source>  modules/gdscript/gdscript.cpp:127 @ _create_instance()
	# <Stack Trace> Node2D.gd:19 @ _ready()

I updated the description, ready to merge.

Thanks!


[Firebase Messaging, Firebase Analytics]: Firebase service issues with SPM.

Is there an existing issue for this?

[x] I have searched the existing issues.

Which plugins are affected?
Analytics, Messaging
Which platforms are affected?
iOS
Description
Please, fix the SPM Integration like what have done for the "firebase_core" package in the latest version.
https://github.com/firebase/flutterfire/pull/16834
Reproducing the issue
[✓] Flutter (Channel stable, 3.27.1, on macOS 15.2 24C101 darwin-arm64, locale
en-EG)
• Flutter version 3.27.1 on channel stable at
/Users/muhammad/.fvm/versions/3.27.1
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 17025dd882 (3 days ago), 2024-12-17 03:23:09 +0900
• Engine revision cb4b5fff73
• Dart version 3.6.0
• DevTools version 2.40.2
[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
• Android SDK at /Users/muhammad/Library/Android/sdk
• Platform android-35, build-tools 35.0.0
• Java binary at:
/Library/Java/JavaVirtualMachines/jdk-19.jdk/Contents/Home/bin/java
• Java version Java(TM) SE Runtime Environment (build 19.0.2+7-44)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 16.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 16C5032a
• CocoaPods version 1.16.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2024.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 21.0.3+-79915917-b509.11)
[✓] VS Code (version 1.96.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.102.0
[✓] VS Code (version 1.94.2)
• VS Code at /Volumes/Data/apppps/Visual Studio Code.app/Contents
• Flutter extension version 3.102.0
[✓] Connected device (4 available)
• iPhone 16 (mobile)              • ECD81F69-447D-41FD-98D3-486A7D9722F3 • ios
• com.apple.CoreSimulator.SimRuntime.iOS-18-1 (simulator)
• macOS (desktop)                 • macos                                •
darwin-arm64   • macOS 15.2 24C101 darwin-arm64
• Mac Designed for iPad (desktop) • mac-designed-for-ipad                •
darwin         • macOS 15.2 24C101 darwin-arm64
• Chrome (web)                    • chrome                               •
web-javascript • Google Chrome 131.0.6778.205
[✓] Network resources
• All expected network resources are available.
Firebase Core version
3.9.0
Flutter Version
3.27.1
Relevant Log Output

Flutter dependencies
dependencies:
flutter:
sdk: flutter
firebase
firebase_core: ^3.9.0
state management
flutter_bloc: ^8.1.6
bloc: ^8.1.4
local storage
hive: ^2.2.3
hive_flutter: ^1.1.0
hive_generator: ^2.0.1
path_provider: ^2.1.5
shared_preferences: ^2.3.4
#localization
intl: ^0.19.0
flutter_localizations:
sdk: flutter
apis packages
http: ^1.2.2
mime: ^2.0.0
http_parser: ^4.1.0
collection: ^1.19.0
dio: ^5.7.0
other packages
cupertino_icons: ^1.0.8
shimmer: ^3.0.0
lottie: ^3.3.0
flutter_easyloading: ^3.0.5
loading_animation_widget: ^1.3.0
url_launcher: ^6.3.1
font_awesome_flutter: ^10.8.0
carousel_slider: ^5.0.0
email_validator: ^3.0.0
page_view_indicators: ^2.0.0
pinput: ^5.0.0
flutter_svg: ^2.0.16
cached_network_image: ^3.4.1
eva_icons_flutter: ^3.1.0
flutter_typeahead: ^5.2.0
equatable: ^2.0.7
getwidget: ^5.0.0
page_transition: ^2.1.0
file_picker: ^8.1.2
photo_view: ^0.15.0
package_info_plus: ^8.1.2
pretty_dio_logger: ^1.4.0
connectivity_plus: ^6.1.1
fluttertoast: ^8.2.10
flutter_local_notifications: ^18.0.1
device_preview: ^1.2.0
flutter_multi_formatter: ^2.13.0
country_code_picker: ^3.1.0
syncfusion_flutter_sliders: ^28.1.35
syncfusion_flutter_core: ^28.1.35
country_picker: ^2.0.27
image_picker: ^1.1.2
gap: ^3.0.1
hugeicons: ^0.0.7
line_icons: ^2.0.3
fluentui_system_icons: ^1.1.270
flutter_staggered_grid_view: ^0.7.0
go_router: ^14.6.2
internet_connection_checker: ^3.0.1
persistent_bottom_nav_bar_v2: ^5.3.1
collection: ^1.19.0
get_it: ^8.0.3
flutter_hooks: ^0.20.5
Additional context and comments
No response

firebase_analytics: ^11.3.6
firebase_core: ^3.9.0
firebase_messaging: ^15.1.6
[!] Unable to find a specification for firebase_core depended upon by firebase_analytics
You have either:

mistyped the name or version.
not added the source repo that hosts the Podspec to your Podfile.

When I am using Firebase core package only the app works well.

Hi @muhammadkamel, thank you for the report. A similar issue already exists, which you can track for updates here.


I can make payment form for buy Telegram Premium?

I can not find the necessary methods in the documentation, so I decided to turn here. Is it possible to pay for Telegram Premium via TDLib?

You can only pay via a credit card to Apple, Google, or a payment provider at https://t.me/PremiumBot.

Hi Guy's any one know how can I get visa card for charge telegram premium free


optional flag setting

Overview
As an interface library, all the compile options propagate and this includes warning flags. This PR sets warning flags only for debug mode. However, while I am writing this, it occurs to me that we should probably set warnings for release mode or actually keep it for both. Instead add an option for a fast-build.
Plus, typo fix and changed GOOGLETEST option to BEZMAN_GOOGLETEST.
EDIT
As a header only library, it really shouldn't populate any compile flags unless it is critical. It should be 100% user's choice. Just to not disturb previous setup, there's an option to populate compile flags (BEZMAN_USE_COMPILE_FLAGS). An alternative would be using similar approach as logger and define a dummy target that holds all the flags.

keeps failing because "-O3>"  instead of -O3. On that note, I think we should get rid of any flags beside warnings at library level. As header-only / interface library, setting optimization flags is quite intrusive. Instead, that should be a choice of front-end targets.

sure thing!

ufff, that was a long trial and error run to satisfy CI.
So warning message will be printed in 2 conditions:

If other project adds bezman with add_subdirectory() and BEZMAN_USE_COMPILE_OPTIONS=ON
Whenever front-end tries to link bezman target that's built with the option ON.

This is a lot of work to keep the previous workflow. The other option, which includes another dummy target might be a better and user friendly solution. If you'd prefer having that, call me. maybe


set SourceText internal slots

DO NOT MERGE until https://github.com/tc39/Function-prototype-toString-revision reaches stage 3.

By the way, let me know what you want to do about the numbering.

The Function#toString proposal is now stage 3! Can this be merged?

Other than the list numbering inconsistency thing (which should be a separate issue), yes.

I'll fix the numbering when I do the PR.


[bug] Python - Cuda error (without using Cuda)

I've recently upgraded to xgboost version 2.0.3 (Python), and since then I cannot use it anymore as keeps crashing. The following simple code fails to run:
import xgboost as xgb
import numpy as np

train = xgb.DMatrix(np.array([1,2,3]).reshape((-1, 1)), label=np.array([2,3,4]))  # Weird enough, if I don't specify the label the error does not show up

And the error message shows the following traceback:
Traceback (most recent call last):
  File "/home/nicola/mega_workspace/trader/debug_5.py", line 4, in <module>
    train = xgb.DMatrix(np.array([1,2,3]).reshape((-1, 1)), label=np.array([2,3,4]))
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/site-packages/xgboost/core.py", line 730, in inner_f
    return func(**kwargs)
           ^^^^^^^^^^^^^^
  File "/home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/site-packages/xgboost/core.py", line 869, in __init__
    self.set_info(
  File "/home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/site-packages/xgboost/core.py", line 730, in inner_f
    return func(**kwargs)
           ^^^^^^^^^^^^^^
  File "/home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/site-packages/xgboost/core.py", line 932, in set_info
    self.set_label(label)
  File "/home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/site-packages/xgboost/core.py", line 1070, in set_label
    dispatch_meta_backend(self, label, "label", "float")
  File "/home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/site-packages/xgboost/data.py", line 1218, in dispatch_meta_backend
    _meta_from_numpy(data, name, dtype, handle)
  File "/home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/site-packages/xgboost/data.py", line 1159, in _meta_from_numpy
    _check_call(_LIB.XGDMatrixSetInfoFromInterface(handle, c_str(field), interface_str))
  File "/home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/site-packages/xgboost/core.py", line 282, in _check_call
    raise XGBoostError(py_str(_LIB.XGBGetLastError()))
xgboost.core.XGBoostError: [20:49:47] /home/conda/feedstock_root/build_artifacts/xgboost-split_1712072663242/work/src/data/array_interface.cu:44: Check failed: err == cudaGetLastError() (0 vs. 2) :
Stack trace:
  [bt] (0) /home/nicola/Software/miniconda3/envs/text_xgb/lib/libxgboost.so(dmlc::LogMessageFatal::~LogMessageFatal()+0x6e) [0x7f12a9d2164e]
  [bt] (1) /home/nicola/Software/miniconda3/envs/text_xgb/lib/libxgboost.so(xgboost::ArrayInterfaceHandler::IsCudaPtr(void const*)+0xdb) [0x7f12aa3801fb]
  [bt] (2) /home/nicola/Software/miniconda3/envs/text_xgb/lib/libxgboost.so(xgboost::MetaInfo::SetInfo(xgboost::Context const&, xgboost::StringView, xgboost::StringView)+0x126) [0x7f12a9f0f426]
  [bt] (3) /home/nicola/Software/miniconda3/envs/text_xgb/lib/libxgboost.so(XGDMatrixSetInfoFromInterface+0xf7) [0x7f12a9d02927]
  [bt] (4) /home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/lib-dynload/../../libffi.so.8(+0xa052) [0x7f12c86a5052]
  [bt] (5) /home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/lib-dynload/../../libffi.so.8(+0x8925) [0x7f12c86a3925]
  [bt] (6) /home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/lib-dynload/../../libffi.so.8(ffi_call+0xde) [0x7f12c86a406e]
  [bt] (7) /home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/lib-dynload/_ctypes.cpython-311-x86_64-linux-gnu.so(+0x92e5) [0x7f12c87bc2e5]
  [bt] (8) /home/nicola/Software/miniconda3/envs/text_xgb/lib/python3.11/lib-dynload/_ctypes.cpython-311-x86_64-linux-gnu.so(+0x8837) [0x7f12c87bb837]

It surprises me that it throws an error related to Cuda, even though I'm trying to use only classic CPU xgboost.
My configuration is as follows:
xgboost version: 2.0.3 (Python 3.11, clean anaconda environment with only xgboost installed)
OS: Debian 12, with Nvidia drivers 550.54.15  and Cuda 12.4
Hardware: RTX 4000 series card present

The code above used to run flawlessly in Python xgboost 1.7.x.

Haven't been able to reproduce with CUDA 12.3, trying 12.4 now.

Still haven't reproduced it.

That's odd, how come that getting the last error is cudaErrorMemoryAllocation.

I boiled it down to my cuda installation being apparently broken (was not expecting since I installed through Debian packages provided by Nvidia).
Nevertheless, I don't know if it should be considered a bug that any xgboost (even CPU-based) application crashes due to issues on the Cuda layer.

That makes sense, I will open a PR to workaround that. It's just XGBoost needs to know whether the data is from GPU or CPU, and we use CUDA runtime to obtain this information. As a result, there's a CUDA error when checking the input data.


Default gray is too dark

I can't see now's ids :D

Do you have another color on mind ? Maybe something like #FAFAFA, but I don't know if that is going to fix now's ids. I use that color on my default theme because is more readable.

@rauchg could you post the hex value of the hidden text? Will make it a little easier to track down. It's entirely possible it's using a value that's not one of the 16 we've got defined in the ~/.hyperterm.js file

This should be closeable after merge of #272

Now that #272 is merged, the colors look like this: (default config)


[gui][fix] Set cmpData to null when no newcheck filter is set

Set cmpData API parameters to null when only the diff type is set and no
newcheck filter (run ids, tags) is set.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- web/server/vue-cli/src/store/modules/base-filter.js  1

See the complete overview on Codacy


CLI format should be compatible the GNU Coding Standards

TypeScript error line format doesn't appear to be compatible with the GNU Coding Standards.
Actual behavior:
utils/user/info.ts(91,9):

Expected behavior:
utils/user/info.ts:91:9

The compiler error messages are compatible with MSBuild error message format.
For other tools, the Compiler API provides a raw data and it should be straight forward to extract the needed information.

MSBuild error message format is a platform specific format for Visual Studio. The current issue is just a proposal to follow GNU Coding Standards on other platforms where the tsc is available.

We can not break existing users who have been getting this behavior for so long.
so the only option is to add another output format. not sure if adding multiple formats would be worth the hassle.
I would recommend using the API to integrate with the build tool similar to what gulp-typescript, broccoli-typescript, grunt-typescript, etc..

We can not break existing users who have been getting this behavior for so long.

How is it possible if the current format is used only by VS? ))

I would recommend using the API to integrate with the build tool similar to what gulp-typescript, broccoli-typescript, grunt-typescript, etc..

I like the tsc command and I don't see any sense to use Grunt or something else as an alternative...

That's an assumption - we've had people relying on the fact that our error messages forwarded to stderr in the past, and when we switched to stdout they were very unhappy. People will come to rely on these patterns.
Why not just wrap the output with sed or something similar to get what you want?

Why not just wrap the output with sed or something similar to get what you want?

I haven't seen this format before. And I haven't seen any editor (probably only VS) that can work with this format as well.
:91:9 — is a standard  format for systems are based on POSIX standard
(91,9): — is a standard format for Windows system only
That's why I don't see any reason why this format should be on my *nix system.

Why not just wrap the output with sed or something similar to get what you want?

Why do you not want to do exactly the same thing? 😄

Why is the conversation on this repo in English when so many more people speak Chinese? Reasons, man 😃


[Feature] Support for Analyzing Multiple Inputs Applications Together

Kantra currently can't analyze input multiple applications at once. Adding this feature would let users analyze several projects together, using the same settings and producing combined reports similar to how Windup supports analyzing multiple applications simultaneously.  This change would save time, ensure consistency, and give better insights.

Let's clarify the CLI arguments for multiple applications analysis, assuming this is not Hub-related, but pure kantra CLI solution. Windup looks to use --input option [1] that could be file or a directory, is it the right way to allow use multiple --input as an array or applications?
[1] https://access.redhat.com/documentation/en-us/red_hat_jboss_migration_toolkit/2.4/html-single/windup_user_guide/index#command-line-arguments

@aufi I think that the main benefit here for having multiple apps, is so that we have a static report generated with multiple apps as part of the assessment.
This is particularly useful for microservices, or even larger systems deployed as a handful of deployable units, all part of the same "business application". This will give the end user a single report with all the analysis in a single report.
Now, whether --input can receive a list of inputs (e.g., --input my app.jar,myotherappsource/,someotherapp.jar) or requiring one --input for each input (e.g. --input myapp.jar --input otherapp.jar --input sourcefolder/), I think that's up to the engineering team to decide. Both options are scriptable.

PR was merged, docs/usage: https://github.com/konveyor/kantra?tab=readme-ov-file#analyze-multiple-applications

Related paragraph from kantra readme:
Analyze multiple applications
By design, kantra supports single application analysis per kantra command execution. However, it is possible use --bulk option for executing multiple kantra analyze commands with different applications to get an output directory and static-report populated with all applications analysis reports.
Example:
kantra analyze --bulk --input=<path/to/source/A> --output=<path/to/output/ABC>
kantra analyze --bulk --input=<path/to/source/B> --output=<path/to/output/ABC>
kantra analyze --bulk --input=<path/to/source/C> --output=<path/to/output/ABC>

@aufi I don't understand how that example relates to what @brunoborges was asking for. Could you elaborate further how would that work if I want to analyze applications A, B and C and get the reports for the three of them merged together?

I think the implementation is almost there.
But instead of calling kantra three times with the --bulk parameter, the --input parameter should allow multiple inputs, either by comma separated list, or by adding multiple --input

Thank you for comment Ramon and Bruno, adding some description what lead us to focus on little different way to getting static report including multiple applications analyses. If the proposed --bulk option would not satisfy you use-case, let's discuss it.
Initialy requested command like below would introduce issues with different input applications technologies, output directory structure incompatible with single application analysis, unexpectable running time before returning the report and our aim move such use-cases primary to use Hub-based Konveyor installation.
kantra analyze --input A --input B --input C --output=<path/to/output/ABC>
# or
kantra analyze --input A,B,C --output=<path/to/output/ABC>

The --bulk option allows to run applications analysis (single application per kantra call), but put results of multiple application analyses into a single static report (when pointing to the same output directory and it doesn't make assumptions about applications technologies/languages, sources, targets, etc.
It is true that there was also nearly working PR with initialy requested functionality (currently broken by multiprovider changes and replaced by the multiple inputs with --bulk option), but it could be refreshed and completed.
This discussion probably should involve also @pranavgaikwad and @shawn-hurley (when they're available).

Example of currently proposed solution:
A user have apps A, B and C in a directory path/to/source and wants analyze them with kantra.
In order to run analysis and get single static report covering all 3 applications A, B, C, commands below should be executed (kantra analyze with --bulk option to the same output directory)
kantra analyze --bulk --input=<path/to/source/A> --output=<path/to/output/ABC>
kantra analyze --bulk --input=<path/to/source/B> --output=<path/to/output/ABC>
kantra analyze --bulk --input=<path/to/source/C> --output=<path/to/output/ABC>

I should have mentioned one more option we agreed at some point with dev-colleagues and that was not use multiple input fields (or list of applications), but add --input-bin-dir option to analyze all applications within the directory.
Example command:
# have application A.war, B.war, C.ear in directory ./apps
kantra analyze --input-bin-dir=./apps --output=<path/to/output>

Let me summarize current state. There is merged&built upstream solution/workaround combining multiple application analyses into single static report with --bulk option.
To fulfil request from this RFE, we might discuss if real multiple --input options and/or --input-bin-dir options support will be implemented too. Let's talk about it on regular bi-weekly call.


Sending 155k messages to client causes client communication to fail.

I have a custom log module at https://github.com/GRMrGecko/LogSQL which I have not played back in 2 days, so 2 days worth of messages is needing to be received by my client and that adds up to 155,160 messages. With ZNC 1.2, this was handled just fine. But when I decided to try upgrading, first to 1.6 then to 1.7.x, ZNC could not handle this.
On the versions I tested, ZNC will only send 6,993 messages back to the client and after that, ZNC will no longer be able to send messages to the client however the client is able to send messages to ZNC.
The number is very close to 7k, which makes me think there is some kind of limit in the newer versions of ZNC that caps out at 7K. If that is the case, how do I disable this so called limit, or increase it to 999,999?
Thanks for any help on this. I am surviving by downgrading back to ZNC 1.2.

I don't have any idea on this issue, but

I am surviving by downgrading back to ZNC 1.2.

1.4 was security fix release and you probably want to use at least it.

http://wiki.znc.in/ChangeLog/1.4
http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2014-9403

Well, debug your module. znc -D, gdb, strace, etc. can help you with this.
You use PutUser(), right? There's no any special limit for it.

If you're sure 1.2 handled this, you can try also git bisect

I debugged with -D already and saw that it says that all messages were sent. One thing I didn't think of doing is searching for the last UNIX time that was sent and looking for any messages. I don't really have time for that now as homework is available and work starts tomorrow.


Question on Segmentation task?

Hey @kaylode, does the Theseus only support Binary Class Segmentation for now right? I try to use it but it returned an runtime error at dice loss, like this:
File "/content/main/theseus/segmentation/losses/dice_loss.py", line 37, in forward
    num = torch.sum(torch.mul(predict, target), dim=1) + self.smooth

RuntimeError: The size of tensor a (6750208) must match the size of tensor b (65536) at non-singleton dimension 1

Thanks!

Ah yes, i saw you're working on it 💯

Ah yes. I'm working on multiclass segmentation right now. You can co-test  with me on the segmentation branch, it works fine for me on several datasets. But please note that you have to customize your own dataset and dataloader to read/ preprocess . The targets must be one-hot  encoded (B, num_classes, H, W) to be able to work with all the loss functions and metrics.

Okay, I'm testing it right now. Will you include coco dataset reader in the future. In coco format there's also a 'segmentation' part in 'annotation'. I think you may consider to support coco, as I saw there're still many datasets use it.

Thanks for your opinion.  Most datasets use coco annotations for instance segmentation tasks. The current segmentation branch in this project supports only semantic tasks and semantic ground truth requires less information than instance task (only need a grayscale mask). But I will give it a thought later.


Added inline keyword to functions in header

It was uncompilable with VC++ if it was included in multiple files.

Merged to develop


Allow 2D cubic interpolation

Description
Allow cubic interpolation of 2D data
Fixes # 2017
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ flake8
[ ] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Now in #2258


Check if Python 3.12 works

I don't expect it to work, but let's see.

It's not working due to Numba. Maybe we should not test against our optional dependencies? Numba and Tensorflow are usually slow to support the latest Python versions (for understandable reasons).

@hofaflo what do you prefer?

Don't test on 3.12, but include Numba in our tests.
Test on 3.12, and remove Numba from our tests.

If we don't change anything, we're in option 1. I tend to prefer option 2 though, because Numba is only an optional dependency, and I'd like to support Python 3.12 (and any future releases) ASAP. In addition, we are also not testing Tensorflow functionality (also an optional dependency), which would also delay our support for latest Python releases, so we're already inconsistent in testing/not testing optional deps.

I would like to keep the Numba test to make sure nothing breaks there, but skip it for now, like we did last year (see #124)

It looks like we will have to wait until Python 3.12 is released, unless we want to enable pre-release dependencies (e.g. cffi). I say we just wait.

Seems like we need to wait for https://github.com/dateutil/dateutil/issues/1284 (they have already fixed it, but have not cut a new release yet). Or we just silence this warning for now and hopefully not forget about it.


Performance regression after incremental compiler change

Context
https://github.com/gradle/gradle/pull/16536 seems to introduce some performance regression, which can be ~6% in some scenario:
https://builds.gradle.org/repository/download/Gradle_Master_Check_PerformanceTestSlowLinuxbucket11/42534158:id/results/performance/build/test-results-largeMonolithicJavaProjectPerformanceTest.zip!/performance-tests/report/tests/clean-assemble-with-empty-local-cache---largeMonolithicJavaProject---TaskOutputCachingJavaPerformanceTest.html#result1990747628
Bisect to https://github.com/gradle/gradle/commit/a59133ca10bd660cf47802adc634b05b344d6419
Or 1%~2% in other scenarios:
https://builds.gradle.org/repository/download/Gradle_Master_Check_PerformanceTestTestLinuxbucket33/42539557:id/results/performance/build/test-results-largeMonolithicGroovyProjectPerformanceTest.zip!/performance-tests/report/tests/assemble-for-non-abi-change---largeMonolithicGroovyProject---JavaIncrementalExecutionPerformanceTest.html#result1990747628
https://builds.gradle.org/repository/download/Gradle_Master_Check_PerformanceTestSlowLinuxbucket5/42534152:id/results/performance/build/test-results-largeJavaMultiProjectPerformanceTest.zip!/performance-tests/report/tests/clean-checkout---largeJavaMultiProject---JavaFirstUsePerformanceTest.html
I'll have to rebaseline all to unblock master.

CC @oehme @melix @gradle/execution

This is expected, as stated in the PR. We are doing a bit more work at the end of the task now, to get cacheability.

As @oehme said, this is expected. Let's close this issue.


Connatix docs update

🏷 Type of documentation

[ ] new bid adapter
[X] update bid adapter
[ ] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

📋 Checklist

[X] Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself
[ ] For new adapters check submitting your adapter docs

Sorry, but the gpp_supported metadata field has been replaced with gpp_sids as described in https://docs.prebid.org/dev-docs/bidder-adaptor.html#submitting-your-adapter
If you support the IAB’s GPP consent string, add gpp_sids with a comma separated list of section names, e.g. gpp_sids: tcfeu, tcfca, usnat, usstate_all, usp. Default is None.

I have made the necessary changes. Thanks!


NER Demo

Is your feature request related to a problem? Please describe.
Implement a demo for NER with Streamlit.
Reference

An example
A tutorial
spacy-streamlit

This should be the MIMIC-III demo (not a new one), right?

The NER model is not specific to the MIMIC-III dataset, so I feel it should be able to accept arbitrary medical text as input once implemented.


How Run Telegram Bot Api Local With Proxy?

I am using wifi network with manual proxy, how to run telegram-bot-api with proxy?
I've added these parameters
--proxy=http://<IP_ADDRESS>:8080

in the terminal but the local api bot can't be accessed / doesn't work

sorry, I set the wrong network connection

The proxy option changes HTTP proxy server for outgoing webhook requests. It doesn't affect connections to Telegram servers.

The proxy option changes HTTP proxy server for outgoing webhook requests. It doesn't affect connections to Telegram servers.

Can telegram api local bots use proxies?
i tried but it shows this error in logs

i tried curl http://localhost:9000/bot<token_bot>/getMe but it freezes, but i can run bot/telegram using my proxy

@azkadev See #399.


Fix: When a user is mentioned, a sound is triggered. However, the current implementation plays the sound regardless of who is mentioned.

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/36542
PROPOSAL: https://github.com/Expensify/App/issues/36542#issuecomment-1944928931
Tests

Open the app in two separate sessions, one for User A and one for User B.
Log in to User A's account and navigate to any report that both user A and user B have access to.
Mention a user who is different than User B in the report.
Verify that User B does not receive an "Attention" notification sound for the mention. They should receive an ordinary "Receive" sound.
Send a message mentioning User B in the report.
Verify that User B receives an "Attention" notification sound for the mention.

[x] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

Same as tests.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos
https://github.com/Expensify/App/assets/77965000/3c73a537-4387-4b1d-8c0a-a6ea07fc45c1

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Actually, I think we'll need to handle mobile seperately, ignore me!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.4.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.4.44-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅