No mecab-ipadic-utf8 on centos 7, how can I use wordfreq on Japanese in this case?

Hi, I try to use wordfreq on Japanese on Centos 7. I keep getting an error of Couldn't find the MeCab dictionary named 'mecab-ipadic-utf8', however, there's no such package on Centos 7. It's called mecab-ipadic, how can I run wordfreq on Centos 7 in this case? Thank you so much.

Hi, I try to use wordfreq on Japanese on Centos 7. I keep getting an error of Couldn't find the MeCab dictionary named 'mecab-ipadic-utf8', however, there's no such package on Centos 7. It's called mecab-ipadic, how can I run wordfreq on Centos 7 in this case? Thank you so much.

To be able to use wordfreq in Japanese, you need to have a UTF-8 compatible version of MeCab installed. If your package manager doesn't provide one (I checked and confirmed that the CentOS version is the incompatible EUC-JP version), you need to install it from source:
git clone https://github.com/taku910/mecab
cd mecab/mecab
./configure --enable-utf8-only
make
sudo make install

That should install MeCab with a UTF-8 Japanese dictionary in /usr/local/lib/mecab/dic, where wordfreq will find it.

Oh, there's more that you need to actually get the dictionary:
cd ../mecab-ipadic
./configure --enable-utf8-only
make
make install

I haven't confirmed that this part works, unfortunately, and I can't read Japanese well enough to follow the documentation.

Thanks for your comment. Referring to https://centos.pkgs.org/7/nux-dextop-x86_64/mecab-ipadic-<IP_ADDRESS>70801-10.el7.nux.1.x86_64.rpm.html, I thought mecab-ipadic on centos 7 is for UTF-8 use ony. So I wonder for wordfreq/mecab.py, should mecab-ipadic added to the list of line 58 'ja': ['mecab-ipadic-utf8', 'ipadic-utf8']. Because now I have mecab-python3 work with current
configuration.

Oh, I see! On CentOS, unlike on Ubuntu, the unmarked version is the UTF-8 one. I saw the reference to "EUCJP", but that's a separate version of the package, marked because it's the EUC-JP version. I do appreciate it defaulting to UTF-8.
So it sounds like that fix would work for CentOS. The problem, then, is that the dictionary of the same name on Ubuntu would be the wrong one. When you run MeCab with the wrong encoding of dictionary, it runs, but outputs complete nonsense results. Can you think of any way to select the correct dictionary for the operating system?

Thank you for confirming my fix for CentOS. To be honest, I don't know. I think it's better to add OS info to  MECAB_DICTIONARY_NAMES for dictionary selection if possible. Also, I'm curious about how mecab-python3 works as it also requires mecab-ipadic-utf8.

Hi, adding 'mecab-ipadic' doesn't work but changing line 53 to return MeCab.Tagger("-Ochasen") works.

I'm sorry, I don't understand what that's doing.

Oh I get it: if we leave out the dictionary, MeCab will use whatever Japanese dictionary it prefers.
Unfortunately, on Ubuntu, the default Japanese dictionary is mecab-jumandic-utf8, not mecab-ipadic-utf8. So if jumandic is installed, this would appear to work but give results that aren't compatible with wordfreq's vocabulary.
This is hard. I would like to make this work on multiple flavors of Linux and make sure that it never silently gives wrong results, and I don't know how to test that part. Part of the problem is that I don't have a way to test what happens on CentOS.

I really appreciate your contribution to wordfreq, to test on CentOS, probably you can use a docker image of CentOS.

For Fedora users like me:
$ sudo dnf install mecab-ipadic
$ sudo mkdir /usr/lib/mecab/dic/ -p
$ ln -s /usr/lib64/mecab/dic/ipadic /usr/lib/mecab/dic/mecab-ipadic-utf8

Hello, I'm the maintainer of mecab-python3.
Starting at the end of 2018, mecab-python3 began distributing wheels for Linux that included a bundled IPADic. So at that point the install of a system MeCab binary and dictionary was unecessary, and would in fact have been ignored. (That covers all posts in this thread, for the record.)
Starting with v1 last summer, wheels are available for Windows, OSX, and Linux, but the dictionary is bundled as an extra package. You seem to have already addressed this by depending on the ipadic package on PyPI, but your setup.py says a system mecab-dev package install is required, which is not the case.
If you have any issues with MeCab please feel free to @ me at any time.
As a side note, I notice you packaged mecab-ko-dic for use with pypi. I would recommend getting someone who speaks Korean to check the output of that - the main thing I understand about that dictionary is that it's intended for use with a patched MeCab binary that has special treatment of spaces. However I don't speak Korean and have never seen an English explanation of this so I haven't been able to figure out the details.

Hello polm, thanks for the info here.
The release of version 2.4.2 is, as you've noticed, intended to resolve the long-standing difficulty of dealing with mecab's dependencies. This bug can be closed, except I'll leave it open for a bit because we're having this conversation here.
I didn't know that the default behavior of mecab-python3 itself had changed to use ipadic (we were overriding the default dictionary path before, so that we could look for multiple dictionaries). And it's true that I left a comment behind in setup.py that no longer applies.
I'm going to start looking into the effects of the Korean dictionary. If there's a problem with handling spaces, it may not appear in typical use of wordfreq -- where the inputs usually come from another dictionary or tokenizer, and wouldn't include internal spaces -- but it would affect exquisite-corpus, the build pipeline that generates the word frequencies in wordfreq.
So if there's a problem, it would show up as unusual word frequencies in the data. Again, I'll look into it.


PYIC-1369: Set backend session expiry time

See this config PR for adding the ttl parameter: https://github.com/alphagov/di-ipv-config/pull/628

Proposed changes
What changed
Set backend session expiry time
Why did it change
We're going to start checking if a users session has expired. To do this
we need to create an expiry time when their session is created. This TTL
should be pulled from config.
Issue tracking

PYIC-1369

The sessionStart lambda will need read permission for the new ttl SSM param


Add CloudVisionAPI interface

lgtm if it works well with other code.

lgtm too, if this is how you plan to have your two classes that implement this interface, however, in my case, the method is:
public ProductDetectionData detectProductPhotoContent(byte[] imageBytes) throws PhotoDetectionException; so I send the image and return ProductDetectionData, which basically contains strings of labels, logos and colors. This is what works for my production & testing code


New package: NNHelferlein v0.9.2

Registering package: NNHelferlein
Repository: https://github.com/KnetML/NNHelferlein.jl
Created by: @andreasdominik
Version: v0.9.2
Commit: 197aff58966bbac02793f703e4d9ad799db149cd
Reviewed by: @andreasdominik
Reference: https://github.com/KnetML/NNHelferlein.jl/commit/197aff58966bbac02793f703e4d9ad799db149cd#commitcomment-73743424
Description: Little helpers for neural networks in the Knet ecosystem

A new version (1.0.0) is submitted and registration requested (#60320) with compat section added to project.toml.
This one (#60319) can be deleted!


Mathematical expression with logical negation

There seems to be a bug that prevents the expressions "not (p and q)" and "not (p or q)" as correct answers to a "mathematical expression" question. Numbas seems to expect (not p) and q (or (not p) or q) instead of the requested answer.
To reproduce:

Create a new part with a mathematical expression;
Enter the expression "not (p and q)" in the Correct Answer settings
Test run and enter not (p and q) and submit the answer: it will show the solution is incorrect;
Test run and enter (not p) and q; submit the answer: it will accept the solution.
Reveal answers: it shows the wrong expression.

Here's a link to an example question exhibiting this problem:
https://numbas.mathcentre.ac.uk/question/52550/propositional-algebra/preview/

Thanks for this comprehensive bug report. The logic to decide when to put brackets around an argument didn't consider prefix operators. I've now fixed it.


widget.CurrentLayoutIcon: icon disappearing with use_mask = True

If use_mask is set to True, the icon disappears.
Playing with foreground color doesn't fix the issue.
widget.CurrentLayoutIcon(scale = 0.66, use_mask = True, foreground=colors[2]),

Any error in your log?

For the colors [2]:
def init_colors():     return [["#cdd6f4", "#cdd6f4"], # color 0 Blue Catppuccin Mocha             ["#1e1e2e", "#1e1e2e"], # color 1 Base Catppuccin Mocha             ["#9399b2", "#9399b2"], # color 2 Overlay 2 Catppuccin Mocha             ["#f5c2e7", "#f5c2e7"], # color 3 Pink Catppuccin Mocha             ["00000000", "00000000"], # color 4 Transparent             ["#f3f4f5", "#f3f4f5"], # color 5 White             ["#45475a", "#45475a"], # color 6 Surface0 Catppuccin Mocha             ["#1e1e2ea9", "#1e1e2ea9"], # color 7 Base Catppuccin Mocha 66% transparency             ["#fab387", "#fab387"], # color 8 Peach Catppuccin Mocha             ["#313244", "#313244"]] # color 9 colors = init_colors()
Any color from this list provide the same results.
No foreground parameter specified + use_mask = True has the same results
I purged the qtile.log and restarted qtile.
It's empty.

Not sure what to say here. I can't recreate this at all. Using a list of colours works fine for me.
Do you have the latest version of qtile-extras installed? (although, if you didn't, you should just get the standard white icons).
What happens if you hardcode a single foreground value?

Qtile-extra version :
aur/qtile-extras-git r142.e522e59-1 [+1 ~0.56] [Installed: r152.a89e5b8-1]
widget.CurrentLayoutIcon(scale = 0.66, use_mask = True, foreground="#f5c2e7"),
Same behaviour.
With:
widget.CurrentLayoutIcon(scale = 0.66, use_mask = False, foreground="#f5c2e7"),
Icon reappears but of course the icon color is still white
Is there any other log i could check to find what's going on ?

Is this Wayland or X11?

X11
I have another list of bug for Wayland when i have more courage to write it down

Can you try creating a MaskWidget from here: https://qtile-extras.readthedocs.io/en/latest/manual/how_to/img-mask.html
You'll need to set a valid icon in the code but I'm interested to see if this works for you.

I did try. And it works !
I get three icons, red green and rainbow.

Ok. So the image mask does work for you. That's good but now I'm more confused!
I'm away from my computer at the moment but will take a look again at this later.

Going from qtile stable version to git version may have some the problem.
Now inputting true for use_mask has the correct behaviour.


fix: release trigger

Looks good, I would just move the bash scripts to a scripts folder in the root (like we do in the Go SDK)


PascalABCNET hash constantly breaks

Scoop Configuration

Installing 'PascalABCNET' (3.8.3_3167) [64bit] from ru-school-scoop bucket
PascalABCNETSetup.exe (19,6 MB) [=============================================================================] 100%
Checking hash of PascalABCNETSetup.exe ... ERROR Hash check failed!
App:         ru-school-scoop/PascalABCNET
URL:         http://pascalabc.net/downloads/PascalABCNETSetup.exe#/dl.7z
First bytes: 4D 5A 90 00 03 00 00 00
Expected:    68b364d49b40f3aa1af8a2f2d1a136741c3cab025d8f24f3cc010c66674da4dd
Actual:      2b3ee1bf2bc2b3f0587fb267fb4504f3c0ba4c70596ffb7300741407ec5eb5dc

Should be fixed now.


Release Google.Cloud.NetworkConnectivity.V1Alpha1 version 1.0.0-alpha01

Changes in this release:
Initial release.

Created release for Google.Cloud.NetworkConnectivity.V1Alpha1-1.0.0-alpha01


Use TypeVar for class/callable decorators

Codecov Report

Merging #129 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #129      +/-   ##
=========================================
+ Coverage   95.79%   95.8%   +0.01%
=========================================
  Files           2       2
  Lines         309     310       +1
=========================================
+ Hits          296     297       +1
  Misses         13      13

Impacted Files
Coverage Δ

loggo/loggo.py
95.77% <100%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


[Feature] Can milkdown support delete the "format" ?

Is your feature request related to a problem? Please describe.
For formats like # or ```, even if I keep pressing delete, the format will never go away unless I use menu or edit the raw markdown text.
Describe the solution you'd like
Now:
When I keep pressing delete, the format will stay there, eg. # and codeblock

https://user-images.githubusercontent.com/1879635/170180371-8c93fcd4-b2c4-4af2-aaf4-6735b3fd5467.mov

https://user-images.githubusercontent.com/1879635/170180230-d06c0019-fab0-4d9a-b744-96883d08774f.mov
Proposal:
When I keep pressing delete, the format will eventually be deleted.
From typora:
https://user-images.githubusercontent.com/1879635/170180678-e8428d8a-0ec9-42f6-8278-944bfcda4ae9.mov

That's due to the design of prosemirror. I don't have a plan to add workaround for this right now.
https://github.com/ProseMirror/prosemirror/issues/1018


feature to save scores

Added the backend logic for saving the top 3 high scores to leaderboard.txt

@bjuspi Great work on adding the backend logic. Could you try and make changes to UI to display the top 3 scores via
options->highscores

Do you have the buttons asset ready ? Would need the leaderboard.png as the button

Do you have the buttons asset ready ? Would need the leaderboard.png as the button


Better management of multiple stacks

Would be nice to:

list which stack is running (if any)
get a friendly error if you attempt to start a stack when another one is already running

Could also discuss if running multiple stacks simultaneously should be supported - right now every stack picks the same port numbers so they will always conflict.

I have a pull request on the first one, which introduces a ps command. The second would definitely throw an error and it possible to run more than one stack simutaneously now if the port was specified at init


endless loop in Fbe.iterate

@rultor release, tag is 0.0.84

@rultor release, tag is 0.0.84

@yegor256 OK, I will release it now. Please check the progress here.

@rultor release, tag is 0.0.84

@yegor256 Done! FYI, the full log is here (took me 2min).


Insert statement for meta data ok?

I understand that when working with lots of data it makes sense to not use single insert statements but import_/export_ statements.
But is it ok to execute the insert statement for managing master data with pyexasol sind import_from_iterable doesn't seem to be flexible enough?
Or should a different library like pyodbc be used in this case?

It is usually a good idea to run a tiny MySQL / PostgreSQL instance and store meta data there. It is especially true for statuses of ETL processes.
You may run some small inserts and updates, and it will be fine at first. But things tend to grow, and you'll start hitting various issues very soon. For example:

100 parallel queries limit;
locking issues, transaction isolation is too strong;
big queries receiving less resources than normal;
SQL audit logs contention with rubbish data;
protected and personal data potentially exposed in SQL audit logs;

It's highly sub-optimal to store meta data in Exasol, and we wanted to explicitly discourage this behaviour. At least in Badoo.
But nothing really stops you from writing raw INSERT or UPDATE query manually and execute it.


Homepage redesign

It will be better to have a revamp of the homepage
The revamped homepage will have -- a short bio, recent news
This change will also need some custom shortcodes to be added

Adding recent news on the homepage using emojis
Doing away with the idea of a custom "about" page and retaining the current layout.
No new short code is required.

Closing this issue, since there is no plans to make changes to the homepage in the current setup


Add a button to pop the ImageJ2 main window when not in headless mode

So that you can do a bunch of ImageJ stuff at once, in its native UI.

See also this thread on Zulip.

This has been done for a while, how did we not close this??
See #89


feature/Add the Book class methods

🚩Add the Book class methods | add-book-class-methods - >>branch<< -
Hi there! @HFG43   👋
This is the PR for the Ruby capstone project. The main objective of this task is to override the can_be_ archived? method of the parent class Item, to show whether the Book can be archived or not.
Here is a summary of what has been done:

can_be_ archived? method:

[x] Override the can_be_ archived? the method in the Book class
[x] Use the value returned by the can_be_ archived? method of the parent class (Item class) and the value of the cover_state property to check if the book can be archived or not.

⭐To the code reviewer 👨💻
💝 I really thank 🙇♂ you, dear code reviewer 👨🎨 for dedicating your precious 🥇 time ⌚ to check the PR 🧾 .
I kindly ask 🙏 dear code reviewer 🤵 that if there is any kind of issue 🦯 in this project, please do list 📃 them in a descriptive 💡 manner and give your best suggestions 🎁 if needed.
If you think some big issues are essential to be changed ♻ please kindly contact 📞 me through Zoom or Slack, I am available on Microverse UTC-6 time zone.

@HFG43 Thank you so much for your review!


Kaniko Build Broken

Hi there,
I know that Kaniko is not a supported environment but it was working until recently and is required to build ORT container images in Kubernetes.
7453ae00e9cf5d0ab2cd32325df9fe6b4cfdbfd4 broke the Kaniko build. Kaniko seems to not support to just mount the archive  yet as docker builtkit is able to do so.
This is the problem I ran into:
INFO[1224] RUN --mount=type=bind,from=build,source=/usr/local/src/ort/cli/build/distributions/ort-$ORT_VERSION.tar,target=/opt/ort.tar tar xf /opt/ort.tar -C /opt/ort --exclude="*.bat" --strip-components 1 && /opt/ort/bin/ort requirements
INFO[1224] Taking snapshot of full filesystem...
INFO[1262] cmd: /bin/sh
INFO[1262] args: [-c tar xf /opt/ort.tar -C /opt/ort --exclude="*.bat" --strip-components 1 && /opt/ort/bin/ort requirements]
INFO[1262] Running: [/bin/sh -c tar xf /opt/ort.tar -C /opt/ort --exclude="*.bat" --strip-components 1 && /opt/ort/bin/ort requirements]
tar: /opt/ort.tar: Cannot open: No such file or directory
tar: Error is not recoverable: exiting now
error building image: error building stage: failed to execute command: waiting for process to exit: exit status 2

I will probably have to maintain a fork until this is fixed.

Kaniko seems to not support to just mount the archive yet as docker builtkit is able to do so.

That's interesting, as our Docker build required BuildKit before, and we're just making use of yet another BuildKit feature. So is BuildKit available in Kaniko, but probably just an older version of BuildKit?

So is BuildKit available in Kaniko, but probably just an older version of BuildKit?

I can not really give you a qualified answer to this. What I know is that Kaniko was catching up with supporting the features needed to build ORT. I haven't found detailed information yet if this was by accident or not.

Hello @jens-erdmann
I do reproduce the same issue for the recent commit 4048e7f5481d335b598e4d6b68f5676955c8e68b
The cache cannot be mounted with --mount=type=bind as it is still not implemented in kaniko from my knowledge https://github.com/GoogleContainerTools/kaniko/issues/1568
Have you succeed in building ort image with kaniko?

Have you succeed in building ort image with kaniko?

Sadly no. We are back to square one again and are building it manually.

Update: I just built the latest version of ORT with v1.9.1 of Kaniko. Latest changes to the Dockerfile seem to enable it again.


SW-6407 Added photo types for observation plot photos

#2732  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 31, 5:33 PM EST: A user merged this pull request with Graphite.


Namespace Requirements incorrect.

It looks like the Namespace is incorrect. It should be rescap3.

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

ID: 9bc57c32-ceb3-65b6-3685-c22ac71bbb1c
Version Independent ID: c52c5de2-3c0f-beac-2786-d17786e38277
Content: rescap3:MigrationProgIds - UWP app developer
Content Source: winrt-related-src/schemas/appxpackage/uapmanifestschema/element-rescap3-migrationprogids.md
Product: windows
GitHub Login: @laurenhughes
Microsoft Alias: lahugh

Thanks for the catch! This will be updated shortly.


fix: handle breaking change in cargo-edit upgrade

:tada: This PR is included in version 1.0.14 :tada:
The release is available on:

v1.0.14
GitHub release

Your semantic-release bot :package::rocket:


CPU Memory Usage

Hi! Thanks for this great work, well done!
We are running this code in our project, but the CPU memory has been eaten drastically over 64GB. Have you ever found this problem? Any idea on how to solve it? Thanks!

Hi! I meet the same issue. We are running detection on our own dataset. The GPU is good, but CPU memory is not sufficient. Did you solve this problem?


Overriding file loader rules to add exceptions

I'm using Quill in a Vue component. To avoid loading everything from the distributable build, I have prepared a quill entry file (like this one).
When doing this, however, webpack moves the svg icons used by Quill theme in /fonts/vendor/quill and returns their path, instead of returning the svg content, as Quill expects. I end up with this:

This doesn't happen when using the distributable build in the npm package.
Quill's build guide and related webpack example say that icons should be loaded with html loader:
https://github.com/quilljs/webpack-example/blob/master/webpack.conf.js#L39
Now to my question: how can I replace/override Laravel Mix's svg loader so it doesn't match Quill icon paths, and instead setup html-loader just for that path?
Thanks
Edit: A past issue (closed unanswered) asks the same thing: #942.

@tpraxl, I remind you)))

I use a ckeditor and it requires an raw-loader for load svg icons. But I can not override the loader.

@tegola thanks for example, but listen is not a function(((

@ZapevalovAnton You need to use the Mix object (uppercase 'M'), so it's Mix.listen(), not mix.listen(). Mix is always available, it's instantiated by the default Laravel mix config.

@tegola Thanks!)

@ZapevalovAnton hehe:) I remember it very well. Yet, I still have time until the fortnight is over. Hopefully, I can look into it then, but it might as well take longer.

@tegola could you point me to a resource for the valid webpack schema? I'd like to research valid options first. If there's no such option, we could try to name the rules anyway and remove the names before we feed the rules to webpack. But I'd rather try to avoid complicated hacks if possible.

@tpraxl This seems like it:
https://github.com/webpack/webpack/blob/master/schemas/webpackOptionsSchema.json#L769

Actually, it looks as if it would be pretty easy to use an object with rule names instead of the rules, merge that with the user config and return an array of rules afterwards. Minimal changes required, all tests are green, but I want to test that manually as well, just to be sure. Just had about half an hour to tinker with the idea.

@tegola I'll have a look at it too, as soon as I've got some time.

Nope. This attempt introduces all sorts of problems. You could override a given rule, but:

This breaks the API (expected object in webpackConfig)
You could override a rules test, for example, but what if you wanted to exclude a loader (I'm currently attempting to merge)

I think we need another API method that allows us to explicitly override a rule. I'll think about it.

@tpraxl thanks for spending your time looking into this. I currently have no spare time to dedicate to the project, and don't even know well webpack and laravel mix config builder, actually. I'll keep a look at this thread and help in any way possible.

There's an early working version that allows to override, skip and customize (merge) the webpack rules:
https://github.com/tpraxl/laravel-mix/tree/feature/override-and-merge-rules
Still need to test that manually and to update the README before I can post a Pull-Request.

I tried to setup a quill example project, can't tell if it looks like it is supposed to and I don't want to spend too much time with quill.
You are not wasting your time. I promise that the customization will take effect. The question is: does it have the desired effect?
@tegola
You could help me out there in two alternative ways (might increase the chance of getting this feature PR ready and merged into laravel-mix)
A) Test this version of laravel-mix with your project setup
Just clone the sources, copy them into your node_modules folder to replace the official laravel-mix version. Then replace your webpack.mix.js workaround with this:
const quillIconsRegex = /node_modules\/quill\/assets\/icons\/(.*)\.svg$/;
Mix.customizeRule.images = (rule, Config) => {
  console.warn('using experimental mix version');
  rule.exclude = quillIconsRegex;
  return rule;
};
mix.webpackConfig({
  module: {
    rules: [{
      test: quillIconsRegex,
      use: [{
        loader: 'html-loader',
        options: {
          minimize: true
        }
      }]
    }]
  }
});

Does it show the warning (experimental) when you run it? Does it work?
B) Give me a working example for the failing quill setup
A complete sample project with quill that shows the problems, so that I can reproduce and tell if my changes work for your use-case.

Ok. Got it working with the instructions I gave above
Before:

After:

Will post the PR soon.

@tpraxl PR looks very nice, I'm gonna give it a spin tomorrow. Great work, thank you!

@tegola Just to avoid irritations: the PR is not merged yet.

If anyone winds up here, I wanted to use the react-svg-loader to compile in my svg icons. This results in no image pop-in and since they're svg icons, adds little to the build.
My solution was to clone the repo and customize the master rules builder. What would be extremely useful would be a component that could override the rules completely. Is this possible by writing a component?

I also need to be able to not use Laravel Mix's built-in SVG handling. I was able to do so building upon
@tegola's code.
First I did console.info() on the rules to find this:
{ rule: { test: /\.html$/, loaders: [ 'html-loader' ] } }
{ rule:
   { test: /(\.(png|jpe?g|gif|webp)$|^((?!font).)*\.svg$)/,
     loaders: [ [Object], [Object] ] } }
{ rule:
   { test: /(\.(woff2?|ttf|eot|otf)$|font.*\.svg$)/,
     loader: 'file-loader',
     options: { name: [Function: name], publicPath: '//<IP_ADDRESS>:8080/' } } }
...

It's the second rule that does the loading of SVGs along with images.
To be able to remove the built-in SVG loading, but keep the image loading, I had to modify the regex to remove the SVG part.
Based on @tegola's code, this is the code I wrote with which I was able to remove the built-in SVG loader so that I could use my own:
const mix = require('laravel-mix');

// Do not use Laravel mix's built-in SVG loader.
Mix.listen('configReady', function(config) {
  const rules = config.module.rules;
  const svgRegexPart = '|^((?!font).)*\\.svg';

  for (let rule of rules) {
    if (rule.test) {
      if (('' + rule.test).indexOf(svgRegexPart) > -1) {
        // Recreate the RegExp with just the SVG part removed from
        // the built-in mix RegExp for loading SVGs and images.
        rule.test = new RegExp(('' + rule.test).replace(svgRegexPart, ''));
      }
    }
  }
});

// ...


[services] Ensure services have env vars + cleanup

Summary
Ensures services have correct environment when called in shell. This fixes a particular case where a plugin is installed and then a service is started without restarting the shell.
Cleanup:

Removed feature flag
Simplified code by having plugin.Env return a slice instead of a map
Factored out service start/stop to make linter happy

How was it tested?

devbox shell
devbox add apacheHttpd && devbox service start apache

But could you add a todo next to the env var logic, that we would stop modifying the environment once we do the environment through nix itself?

will do


Doesn't seem to work on mobile even if forceDesktopMode: true

Hi @cantonalex thank you for the very useful screenshot..
Could you please recreate a test case on https://codesandbox.io/
Thank you

I couldn't determine if it was cloudflare or not but I solved the issue :)


Random Crash

My game occasionally  just randomly crashes when I am attacking Zombies or Skeletons.
This is a crash from a skeleton group, where 1 was on the same level as me, and the other two were in a hole right nearby.
Crash Log

Unfortunately, this is a clash between this mod and other mods that modify the skeleton. such as Twilight Forest.

Is there a way to tell what mod that might be from the crash log? I have 126+ mods installed, and any chance that I won't have to go through and test them all manually is greatly welcome.

There is no easy way. When this happens to me, I normally just drop herdcraft


Need confirmation if s3 buckets are required for PKS on AWS

In Create the Master Node Policy
http://docs-pcf-staging.cfapps.io/runtimes/pks/1-2/aws-prepare-env.html#master-policy
For this section of the Node Policy:
{
"Effect": "Allow",
"Action": "s3:*",
"Resource": [
"arn:aws:s3:::kubernetes-*"
]
},
Can we get dev to confirm if s3 buckets required by the operator?
Addn'l reference:  docs "Configuring AWS for PCF" and Create s3 Buckets

Adding: @outofmem0ry

Hey @neil-hickey would you be able to speak to the above question?

This is blocked on https://www.pivotaltracker.com/story/show/160729059 and https://www.pivotaltracker.com/n/projects/2093412/stories/160098582.

Hi @kroberts7
We've confirmed that S3 buckets are not required and removed the statement from the documentation set. Thanks!


Improve sort order Liquid example

The Liquid for tag has a built-in parameter for reversing direction. This removes the need for assigning the reversed collection to a separate variable.
Source: http://shopify.github.io/liquid/tags/iteration/#reversed

Thank you!


Update README.md

added links to the Simulation pages
Summary
Provide a general description of the code changes in your pull request... were there any bugs you had fixed? If so, mention them. If these bugs have open GitHub issues, be sure to tag them here as well, to keep the conversation linked together.
Other Information
If there's anything else that's important and relevant to your pull
request, mention that information here. This could include
benchmarks, or other information.
If you are updating any of the CHANGELOG files or are asked to update the
CHANGELOG files by reviewers, please add the CHANGELOG entry at the top of the file.
Finally, if your pull request affects documentation or any non-code
changes, guidelines for those changes are available
here
Thanks for contributing to Rails!
Description of the Change
We must be able to understand the design of your change from this description. We need can get a good idea of what the code will be doing from the description here. Keep in mind that the maintainer reviewing this PR may not be familiar with or have worked with the code here recently, so please walk us through the concepts.
Benefits
What benefits will be realized by the code change?
Possible Drawbacks
What are the possible side-effects or negative impacts of the code change?
Verification Process
What process did you follow to verify that your change has the desired effects?

How did you verify that all new functionality works as expected?
How did you verify that all changed functionality works as expected?
How did you verify that the change has not introduced any regressions?

Describe the devices you tested on (e.g., Arduino Uno Rev 3, Arduino Leonardo, sensor HC-SC04, etc.), and describe the results you observed.
Applicable Issues
Enter any applicable Issues here

Amazing!
Thank you very much @matititam!


637106: Outreach for article clean-up: chgranad (2)

This PR is created for the following user story

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

@cgranade , thank you for the feedback. I will address the above and resubmit.

Hi @vatmicro, @cgranade,
These commits were made in the public repo. Should they be made in the private repo instead?
In the private repo, PRs benefit from staging and validation.
Thanks!
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @vatmicro, @cgranade,
These commits were made in the public repo. Should they be made in the private repo instead?
In the private repo, PRs benefit from staging and validation.
Thanks! #label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

Yes, I believe so.

HI @cgranade , I have addressed the issues you pointed out above in various ways, however I need more information from you to resolve a few of the issues. Could you please supply the additional info that I need? Thank you.

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

Hi @cgranade , I have addressed all of your question with this build latest build, aside from those that I have asked for extra clarification on.
Please also note that our goal is to resolve all issues and receive your approval on the PR by end of day tomorrow June 29, since the project is coming to an end.
Thank you.


Can't clear layer filter after being set

You are currently not able to clear a layer filter after it has been set.
For example: you set a filter on a layer to show a subset of the features by setting the prop. At some point, you want to clear your filter to show all features in that layer again. Setting the filter prop back to null does nothing.
The issue lies in this part of the componentWillReceiveProps
if (props.filter && filter && !isEqual(props.filter, filter)) {
    map.setFilter(id, props.filter || [])
}

This would fix the issue
if (!isEqual(props.filter, filter)) {
    map.setFilter(id, props.filter || [])
}

Mapbox says that map.setFilter(null) or map.setFilter(undefined) is the intended way to remove any existing filters off of a layer. See: https://docs.mapbox.com/mapbox-gl-js/api/#map#setfilter

Also passing set filter [] is invalid. Mapbox throws console errors when you try to do that. The proper way to set to an empty filter is by passing null.

sorry for the delay & thanks for pointing this out – a fix was released under v4.8.2

sorry for the delay & thanks for pointing this out – a fix was released under v4.8.2

Much appreciated sir

Hello @mklopets,
seems geojson-layer has the same problem.
I've created a PR with a small fix.
Thanks


[SVCS-TBD] Fix resp.json() Warning

Ticket
TBD
Purpose
resp.json() will throw warning "Attempt to decode JSON with unexpected mimetype: %s" if the "Content-Type" is not of a json type.  This is a correct and expected warning.
However, providers (such as S3 and GoogleCloud) using XML API endpoint pollute the WB celery log with this warning when checking file existences before each copy/move.  In addition, some 404s responses have a string body regardless of the Content-Type.
Current fix is skip parsing the body for HEAD requests.  We cannot simply rely on the header "Content-Type" to decide whether to call .json() or .read().  The warning should be still logged at a level that it is useful but not annoying.
Changes
Skip parsing response body for HTTP HEAD requests.
Side effects
Fixed an issue where response are not released.
QA Notes
QA is not needed.
Deployment Notes
No

Coverage decreased (-0.008%) to 89.909% when pulling ab6bed6517aeb8f4c3bd7b49de54c55cc7c2cc53 on cslzchen:fix/resp-json-warning into 81bec1ad3602177bca177a1ca0e8c93adc65b5e4 on CenterForOpenScience:develop.


Fix false negatives for Rails/EnumUniqueness cop

Fixing Rails/EnumUniqueness cop per conversation in https://github.com/rubocop-hq/rubocop-rails/pull/96#issuecomment-515776639

Could you add a changelog entry to CHANGELOG.md what kind of false negative has been fixed?

@koic Rebased with master and added an entry to the changelog

Thanks!


[FEAT] Support for Strapi v5

It would be great if this plugin could be updated to support Strapi v5. Strapi v5 brings improvements in performance, stability, and new features, which makes it highly desirable for developers who want to keep their projects updated. Currently, the plugin is only compatible with Strapi v4, and upgrading to Strapi v5 causes compatibility issues.

@tobiratzberger it will be nice to have your contribution to this topic.
We are expecting @konstantinmuenster to get back on hand, there is a lot of PR waiting.


On Device with lot of Images, Image Picker is slow to load

When tested on a device with lot of images ~7000 it took about 5 seconds to display result. After removing the line File file = makeSafeFile(path)(Link) in the ImageFileLoader displayed results in about 2 seconds.

i had the same problem can u help us?

@edgarscr what I have done is, I cloned the repo and imported the library into my project as a module and removed those lines as well as made several other changes.

what lines we have to remove? i dont know how i can add this library as module can u help me?

To add a library as a module, on Android Studio select File -> New -> Import Module and import the  imagepicker module of this library. (There are 3 modules in the code of this library. 1. sample 2. rximagepicker 3. imagepicker)
Check the issue message to know which line has to be remove.

Hey guys, sorry for the late reply.
I could make this optional or configurable if you want.
Probably will make this configurable.


MSSQL for Windows Container

Hello,
Are there also instructions available how to setup a MSSQL 2019 in a Windows Container an a Windows 2019 host?
Johan

@Johan1974 -- Johan, please see this article and let me know if it helps address your question:

Deploy and connect to SQL Server Docker containers

Also, consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

Hi @Johan1974. SQL Server is not supported in Windows containers. You can deploy it using Docker on a Windows or Intel-based macOS host for testing and development purposes, but it is not a supported configuration. Only SQL Server on Linux containers is supported.
#please-close


fix: align WebDAV stat with RFC specification

closes: #1782

PTAL @Xuanwo

By the way, do you know if there is a WebDAV service accessible to us? I want to add it to our CI integration tests.

Thanks review.
I had been using jianguoyun for self-testing, but it is not suitable for this project.
Perhaps, you could consider setting up a WebDAV server like apache httpd1 in the GitHub action?

Thanks review.
I had been using jianguoyun for self-testing, but it is not suitable for this project.
Perhaps, you could consider setting up a WebDAV server like apache httpd1 in the GitHub action?

Thanks a lot!


Improve error logging for  in Azure Pipelines Scaler

Hi team,
This PR will improve the error logging in Azure Pipelines Scaler. Could you help to review it :pray: ?
Checklist

[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Changelog has been updated and is aligned with our changelog requirements
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #3996

/run-e2e pipeline*


TST: silence RuntimeWarning for nan test of stats.spearmanr

This is the only test warning I see with 0.19.0rc1, so good to backport.

Since this is newly introduced, maybe it's better to see if there's a way of rewriting the code itself so that it does not produce warnings in the first place.
I'll take a look.

Nah, it's fine. The warnings come from np.corrcoef being nan, which happens for zero-variance inputs. So it falls below my not worth bothering threshold too. LGTM, merging. Thanks Ralf

Backported in 8dbe7c5074d6e57da6a38a0b608588668d9ee40d


Display selected landmark in 'selectedPlaceWidget'

I would like to express my appreciation for this package! It's been incredibly helpful.
I have a feature request: When selecting a location from nearby landmarks, it would be great if the selected landmark could be displayed in the selectedPlaceWidget. Currently, after selecting a landmark, the widget doesn't update to show the selected landmark, and it would be a super useful enhancement if this could be implemented.
It would also be great to add cancel button above the search bar to close the popup
Thanks for your attention to this!

Hi @mostafasany, Thank you for the comments.
For the second one you could use the close icon to close the auto complete popup right?


posix: pthread: implement pthread_cleanup_push() / pop()

pthread_cleanup_push() and pthread_cleanup_pop() are required by the POSIX_THREADS_BASE Option Group as detailed in Section E.1 of IEEE-1003.1-2017.
The POSIX_THREADS_BASE Option Group is required for PSE51, PSE52, PSE53, and PSE54 conformance, and is otherwise mandatory for any POSIX conforming system as per Section A.2.1.3 of IEEE-1003-1.2017.
Fixes #59940
Fixes #59941
See also https://pubs.opengroup.org/onlinepubs/9699919799/functions/pthread_cleanup_pop.html

Cannot reproduce build errors locally. Will rebase.

Cannot reproduce build errors locally. Will rebase.

I can.I'm seeing an allocation failure in test_thrd_reuse  when i == 27 when  pthread_setspecific(__pthread_cleanup_key, &cleanup_head); is called. Is that allocation ever freed? Or is there some other memory leak?

Cannot reproduce build errors locally. Will rebase.

I can. I'm seeing an allocation failure in test_thrd_reuse when i == 27 when pthread_setspecific(__pthread_cleanup_key, &cleanup_head); is called. It doesn't look like that allocation is ever freed.

It's not ever freed. It doesn't actually count as one of the CONFIG_MAX_PTHREAD_KEY_COUNT keys that are user-allocatable.
I'm using a relatively old SDK version 0.16.0. Still not seeing failures (on qemu_x86 and qemu_riscv64).

Cannot reproduce build errors locally. Will rebase.

I can. I'm seeing an allocation failure in test_thrd_reuse when i == 27 when pthread_setspecific(__pthread_cleanup_key, &cleanup_head); is called. It doesn't look like that allocation is ever freed.

It's not ever freed. It doesn't actually count as one of the CONFIG_MAX_PTHREAD_KEY_COUNT keys that are user-allocatable.

No, the problem is that there is per-thread data allocated in this call and that never gets freed, so you end up running out of the kernel pool after you allocate 1024 bytes (by default) of these things.

I'm using a relatively old SDK version 0.16.0. Still not seeing failures (on qemu_x86 and qemu_riscv64).

I'm using 0.16.4 and getting failures. Might be good to give that a whirl?

No, the problem is that there is per-thread data allocated in this call and that never gets freed, so you end up running out of the kernel pool after you allocate 1024 bytes (by default) of these things.

Ah - I see it now. Yeah, that's kind of an annoyance of using k_malloc for this sort of thing. I know some other libcs allocate pthread keys in a dedicated soft-TLS area.
I wonder if there is a way to implement pthread_cleanup_push() / pop() without using pthread_setspecific() - e.g. just on the stack 🤔
Let me give this one another whirl.

@keith-packard - should be ready now.


Some questions about the method

Thanks four your great work! I got some questions about the method you proposed.
First, you treat keyframe and mapping-only frame seperately. If only tracking on keyframes, I want to know how do you get the pose of mapping-only frames?
Second, I don't understand the scale regularization in your paper very well, can you point out the related code?
Thank you very much!

Thanks for your interest in our work!
In our system, we estimate the camera pose on every frame. Keyframes are the subset of frames that are considered to contain essential Gaussians for tracking, and we add Gaussians in the keyframes to the 3DGS map.
Scale regularization is implemented in our submodule, fast_gicp. Please refer to calculating source covariances, and setting target covariances parts in fast_gicp.


Cannot find type Windows.UI.Xaml.Media.XamlLight in module Windows.winmd

@onovotny After Upgrading VS 2019 to 16.2
Upgrading my Multi-targting projects to MSBuild.Sdk.Extras/2.0.31
Using NetCore 3 preview
i get an error when using the Ambient Lights from WindowsCommunityToolkit within a UWP app.
Cannot find type Windows.UI.Xaml.Media.XamlLight in module Windows.winmd

Workaround:
To Solve my issue i actually had to directly reference the Windows.winmd file within my UWP Xamarin Forms App. C:\Program Files (x86)\Windows Kits\10\UnionMetadata\10.0.16299.0\Windows.winmd
Note: I didn't have to reference this file before in previous version of Visual Studio or MsBuildSDKExtras

Do you have a small repro project?

WindowsMDTestApp.zip

To be clear, you only had to reference it in the app? or the library and the app?
You also should not need to add a PackageReference to the Microsoft.NETCore.UniversalWindowsPlatform as the SDK brings that in in 2.x.

I only had to reference in the UWP app. Not in the Library. When try to build the UWP app. I get the erro.
Thank you for the info on Microsoft.NETCore.UniversalWindowsPlatform PackageReference. I follow the Xamarin Essentials Github repo https://github.com/xamarin/Essentials/blob/master/Xamarin.Essentials/Xamarin.Essentials.csproj and it still seems to have it referenced.

I'm going to close this as won't fix since this package really only targets building class libraries. Building app projects is out of scope and you very well may need to add additional references to those.


The wisdom of adding datatype to variable names

E.g. stacked_bar_chart_df versus stacked_bar_chart
See https://en.wikipedia.org/wiki/Hungarian_notation#Disadvantages
though there is
https://en.wikipedia.org/wiki/Hungarian_notation#Advantages
Personally I'm not a fan but maybe should depend on what is most common in R code.

Good point - have changed for here, will keep in mind everywhere but remind me 👍


[Bug]: Unexpected end of form at Multipart._final

Library
"hyper-express": "^6.8.0"
Repo Public
Repo Bug
Actual Behavior
Upload using method stream
await pipeline(field.file.stream, fs.createWriteStream(pathFile))
My code hyper-express.ts
import HyperExpress, { MiddlewareHandler } from "hyper-express";
const mime = require("mime-types")
const { v4: uuidv4 } = require('uuid');
const { pipeline } = require("node:stream/promises");
const fs = require("node:fs");
import path from "path";

const webserver = new HyperExpress.Server({
  max_body_length: Number.MAX_SAFE_INTEGER,
});

// Create GET route to serve 'Hello World'
webserver.get("/", (req, res) => {
  res.send("ok");
});

webserver.any("/upload", async (req, res) => {
  await req.multipart(async (field) => {
    if (field.file) {
      let pathFile = path.join(
        "./storage",
        "images",
        uuidv4() + path.extname(field.file.name || ''),

      );
      console.log(pathFile)
      try {
        await pipeline(field.file.stream, fs.createWriteStream(pathFile));
      } catch (err) {
        console.log("error", err);
      }
    }
  });
  res.send("ok");
});

// Activate webserver by calling .listen(port, callback);
webserver
  .listen(12300)
  .then(() => console.log(`Webserver started on port ${'http://localhost:12300'}`))
  .catch(() => console.log("Failed to start webserver on port 80"));

When I upload files that are less than 100kb, upload success. But, i got error when upload file with size 1mb.
Error: Unexpected end of form
    at Multipart._final

Using postman for upload
Screenshoot
Input File
File 1 MB
Output File
File 192 KB
Expected Behavior
No error
Logs
error Error: Unexpected end of form
    at Multipart._final (C:\Project\xxx\server-linux\node_modules\busboy\lib\types\multipart.js:588:17)
    at callFinal (node:internal/streams/writable:698:12)
    at prefinish (node:internal/streams/writable:710:7)
    at finishMaybe (node:internal/streams/writable:720:5)
    at Multipart.Writable.end (node:internal/streams/writable:634:5)
    at Readable.onend (node:internal/streams/legacy:47:10)
    at Readable.emit (node:events:525:35)
    at Readable.emit (node:domain:489:12)
    at endReadableNT (node:internal/streams/readable:1359:12)
    at processTicksAndRejections (node:internal/process/task_queues:82:21)
Error: uWS.HttpResponse must not be accessed after uWS.HttpResponse.onAborted callback, or after a successful response. See documentation for uWS.HttpResponse and consult the user manual.
    at Response.send (C:\Project\xxx\server-linux\node_modules\hyper-express\src\components\http\Response.js:491:39)
    at C:\Project\Andromedia\server-linux\src\hyper-express.ts:34:7
    at Generator.next (<anonymous>)
    at fulfilled (C:\Project\xxx\server-linux\src\hyper-express.ts:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

Hi, hyper-express limits the HTTP response body size across and Server and even on a per-route basis. Can you try to see and set this limit to just above your testing amount, so something like 5mb? You must specify this limit in bytes and can do so when constructing the Server() or creating a route through the RouteOptions argument. See docs for more specifics.
I think the error is being triggered because hyper-express will immediately terminate request when a body too large is received and the parser attempts to read more bytes leading to the uWS forbidden access exception.

i set max_body_length is<PHONE_NUMBER>000 Bytes or 1tb, when uploading file with size 1566kb, output file not all content will be 20% of content and size 576kb. I have no idea what i can do with this error.
const webserver = new HyperExpress.Server({
  max_body_length:<PHONE_NUMBER>000, // limit 1tb
});

@faisolavolut Can you try again with v6.8.1 of hyper-express and also can you see If you receive an HTTP response to your POST request and what the HTTP status code is?

@kartikk221 still get error, if i use postman respond like this.
POST http://localhost:12300/upload
Error: read ECONNRESET
Request Headers
User-Agent: PostmanRuntime/7.32.3
Accept: */*
Postman-Token: dea0ab74-fe2a-49e5-9278-c25ee53633e9
Host: localhost:12300
Accept-Encoding: gzip, deflate, br
Connection: keep-alive

If i use method buffer in websocket like this, i can upload full size file and no error.
my code
const header = req.getHeader('content-type');
  let buffer = Buffer.from('');
  await res.onData(async (chunk:any, isLast:any) => {
    /* Buffer this anywhere you want to */
    console.log('Got chunk of data with length ' + chunk.byteLength + ', isLast: ' + isLast);
    buffer = Buffer.concat([buffer, Buffer.from(chunk)]);
    /* We respond when we are done */
    if (isLast) {
      const data = getParts(buffer, header);
      let pathFile = join(
        "./storage",
        "images",
        uuidv4() + extname(data[0].filename || ''),
      );
      const fileStream = fs.createWriteStream(pathFile)
      await fileStream.write(Buffer.from(data[0].data))
      res.end('Thanks for the data!');
    }
  });

Full code websocket.ts

@kartikk221 i dont know if my code is right or no. I success upload full file, if i used data buffeer in hyper express.
const header = req.header('content-type');
  let buff = await req.buffer()
  const data = getParts(buff, header);
  let pathFile = path.join(
    "./storage",
    "images",
    uuidv4() + path.extname(data[0].filename || ''),
  );
  const fileStream = fs.createWriteStream(pathFile)
  await fileStream.write(Buffer.from(data[0].data))

its weird, i didn't do anything in req.multipart just console in stream
webserver.any("/upload", async (req, res) => {
  await req.multipart(async (field) => {
    if (field.file) {
      console.log('in stream')
    }
  });

  console.log('finish')
  res.send("ok");
});

log terminal
in stream
finish
> respond Error: read ECONNRESET

@faisolavolut This bug has been fixed in the recent v6.8.2 release of hyper-express. Hope that helps!

[0]     Abdullah Faisol reacted to your message:

From: Kartik @.>
Sent: Monday, July 31, 2023 4:57:13 PM
To: kartikk221/hyper-express @.>
Cc: Abdullah Faisol @.>; Mention @.>
Subject: Re: [kartikk221/hyper-express] [Bug]: Unexpected end of form at Multipart._final (Issue #183)
Closed #183https://github.com/kartikk221/hyper-express/issues/183 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/kartikk221/hyper-express/issues/183#event-9966514023, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BBDI2SPTRQUP2L52ACPAQTLXS7PWTANCNFSM6AAAAAA2YG2WWQ.
You are receiving this because you were mentioned.Message ID: @.***>

[heart]         Abdullah Faisol reacted to your message:

From: Kartik @.>
Sent: Monday, July 31, 2023 4:57:13 PM
To: kartikk221/hyper-express @.>
Cc: Abdullah Faisol @.>; Mention @.>
Subject: Re: [kartikk221/hyper-express] [Bug]: Unexpected end of form at Multipart._final (Issue #183)
Closed #183https://github.com/kartikk221/hyper-express/issues/183 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/kartikk221/hyper-express/issues/183#event-9966514023, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BBDI2SPTRQUP2L52ACPAQTLXS7PWTANCNFSM6AAAAAA2YG2WWQ.
You are receiving this because you were mentioned.Message ID: @.***>


refactor(common): fix project name validation

Description of change

Merge the two implementations of ProjectName in gateway and common.
Improve project name validation with changes from PR #418.

Closes #1223
How has this been tested? (if applicable)
The existing tests were updated to reflect the new constraints added to the implementation in common from the deleted implementation in gateway and from PR #418.

I've thought about this change, and I must say that the ProjectName struct feels a bit over-engineered.
What we want to do:

validate project name on incoming API endpoints in t he backend(s)

What we do:

Wrap it in a struct, forcing us to derive and implement lots of traits + causing the sqlx derive issue
Validate it on every Deser, which means it is re-validated every time it passes a backend border, but also...
validate it when coming out of the gateway database (it should already be valid when being put in)
Pass around this struct all over the codebase, when we already know it is a plain string that has already been validated
Expose it in user-facing API (shuttle-metadata) for not much reason.

Therefore we might just:

Replace all occurrences of it with String
Make is_valid a pure function valid_project_name
Explicitly validate it in each endpoint that takes it in

Or we make some sort of compromise.
Or we use Deref?
I realize that this is pretty substantial and can be carried over to another PR, if wanted.
What do you think @oddgrd?

@d4ckard There is one conflict and then the ordering nitpick :)

I've thought about this change, and I must say that the ProjectName struct feels a bit over-engineered.
What we want to do:
* validate project name on incoming API endpoints in t he backend(s)

What we do:
* Wrap it in a struct, forcing us to derive and implement lots of traits + causing the sqlx derive issue

* Validate it on every Deser, which means it is re-validated every time it passes a backend border, but also...

* validate it when coming out of the gateway database (it should already be valid when being put in)

* Pass around this struct all over the codebase, when we already know it is a plain string that has already been validated

* Expose it in user-facing API (shuttle-metadata) for not much reason.

Therefore we might just:
* Replace all occurrences of it with String

* Make `is_valid` a pure function `valid_project_name`

* Explicitly validate it in each endpoint that takes it in

Or we make some sort of compromise. Or we use Deref?
I realize that this is pretty substantial and can be carried over to another PR, if wanted. What do you think @oddgrd?

I think using the type system to ensure we can't accidentally use an invalid project name anywhere is a good design pattern, but I'm not sure about doing it on deserialize, it feels a bit obfuscated to me. But that may just be my personal preference. The way I've done this in the past (thanks zero2prod!) is to have a parse method on the struct, and that is the only way you can create an instance of the struct. It may seem like overkill now, but as the codebase grows it could prevent bugs. That being said, there might be reasons that strategy isn't a good fit here, so I'm very open to discussion.
By the way, am I mistaken in reading this as applying the stricter requirements we had in the gateway validation to the CLI as well? That will break older projects, we'd have to look into how many and how much it would take to migrate them, but it may block merging this for a bit.

@oddgrd The only rule that was missing from gateway was the profanity check (which was checked in CLI only). Otherwise, all projects currently in gateway should be valid, depending on if anyone has created projects with manual requests. (They would be unusable via CLI though?) So there should not be a big blocker for merging this.

@oddgrd The only rule that was missing from gateway was the profanity check (which was checked in CLI only). Otherwise, all projects currently in gateway should be valid, depending on if anyone has created projects with manual requests. (They would be unusable via CLI though?) So there should not be a big blocker for merging this.

We also disallowed underscores in the gateway, but I'll try to make time this week and see how many projects would be affected by this.

By the way, am I mistaken in reading this as applying the stricter requirements we had in the gateway validation to the CLI as well? That will break older projects, we'd have to look into how many and how much it would take to migrate them, but it may block merging this for a bit.

While digging through the old implementation, to find out what to change, I found that it was quite hard to figure out what exactly the constraints where, and in which situations they applied. However, I believe that some of the checks were duplicated over multiple places, and that this new validation does not reject anything that would  have passed all checks before. The only thing that might have changed is where the check that filters out specific invalid requests fails.

[What we do:] Pass around this struct all over the codebase, when we already know it is a plain string that has already been validated

Regarding this point, I really don't think that this matters much. The code generated for the current iteration of the ProjectName struct is exactly the same that would be generated for a plain string (I disassembled a basic example, this  is just what the newtype pattern is supposed to work like). I agree that it does not make sense to give up the type safety that Rust gives us.
I don't know which one is the best way to go here.

Removing  Deser from ProjectName might eventually lead to strings creeping everywhere, because validation could be forgotten. It might work though, if there relevant functions (e.g. create_project all mandate  ProjectNames).
Allowing to construct ProjectName without validating it makes the entire domain model less secure. At least it keeps the semantic information attached to the type. That's reason enough to keep  some sort of ProjectName type, no matter how simple it is.
Keep ProjectName as is to get all the correctness guarantees attached to it. This makes all development down the line easier, because one doesn't need to worry about ever breaking the validation process: all ProjectNames must be valid.

I tend towards 3. Why give up the safety and correctness guarantees that Rust provides? Potentially, the validation procedure itself could be optimized a bit more, thus decreasing the performance hit (that might exist).

Me and O did some digging, and found this:

Axum extractors don't seem to call the custom Deser, so the only place gateway does validate the name is in create_project when creating a new project (which is kind of the only place it is needed). This realization kind of throws out the "safety" that the Deser was thought to give us. So, if it turns out that this is the only place where gateway ever validated names, then we should survive with keeping it that way, and maybe remove that type.
47 projects have _ in their name, where at most one of them has recent activity. (remains to check other rule violations)

So, we will need to check some more things to make sure this change is not too breaking before it can be merged.

Hey @d4ckard, I just wanted to update you on the state of things. In our investigation of which projects this would break, we determined that we can go ahead with this PR, since very few active projects are affected. However, we are heading to EuroRust next week, so we won't be able to get around to migrating those projects before that. We are happy with this PR, though, and we aim to get it merged as soon as we can after EuroRust. Apologies for the delay!

No problem, I'm glad to hear that the changes are OK 😄.

Hey @d4ckard, do you have time to address the failing CI and resolve the conflicts this week? If you're busy, I can also take care of it. :slightly_smiling_face:

I'm a bit full this week so I'd be grateful if you could fix it.

Sure thing!

I tried removing FromStr and Deser. It became very messy, as they are implicitly used in a lot of places.
Now I have removed all validation form cargo-shuttle.
Shall we remove the type from DeploymentMetadata, which is given to users in the metadata plugin?

@d4ckard I intended to get this merged alongside #1354, but after discussing this we want to keep the main branch clean and have #1354 with all changes to make it easier to review. Thanks for the work on this, your commits live on in #1354 :) (I hope it gets merged very soon!)

Thanks for letting me know. I'm glad I could help 😊


Allow running the program multiple times

At the end of the program, some input should be accepted to determine whether main should be called again. If the input matches something, main should be called again.

i would like to work on it


Support webpack2?

Hi! Do this package supports webpack2 ?

We use it successfully with<EMAIL_ADDRESS>in our environment.

I use it successfully with<EMAIL_ADDRESS>

Seems to be answered. Feel free to ask more.


All command should be accessible from command palette [WIP]

Fix: #288

Codecov Report

Merging #318 into master will increase coverage by 0.44%.
The diff coverage is 69.27%.

@@            Coverage Diff             @@
##           master     #318      +/-   ##
==========================================
+ Coverage   73.65%   74.09%   +0.44%
==========================================
  Files          44       44
  Lines        3245     3347     +102
  Branches      591      622      +31
==========================================
+ Hits         2390     2480      +90
- Misses        855      867      +12

Impacted Files
Coverage Δ

src/tekton/task.ts
39.58% <9.67%> (-0.42%)
:arrow_down:

src/tekton/pipeline.ts
68.18% <47.50%> (+2.11%)
:arrow_up:

src/tekton/clustertask.ts
94.73% <80.00%> (-5.27%)
:arrow_down:

src/tkn.ts
65.45% <80.00%> (+0.19%)
:arrow_up:

src/tekton/taskrun.ts
94.64% <84.21%> (-5.36%)
:arrow_down:

src/tekton/pipelineresource.ts
95.83% <90.90%> (-1.67%)
:arrow_down:

src/tekton/pipelinerun.ts
95.23% <91.30%> (-4.77%)
:arrow_down:

src/tekton/tektonitem.ts
85.93% <93.10%> (-0.73%)
:arrow_down:

src/extension.ts
89.62% <100.00%> (+2.90%)
:arrow_up:

src/tekton/eventlistener.ts
100.00% <100.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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


Fix storage seconds

Fixes #1560, fixes #1563

Hi @oakeyc, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (t-cooka). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Still have Pylint errors


4.3.3 : no payment available anymore

Hello,
Since we have upgraded to 4.3.3, customers can't place an order, as there is no more payment method available. Thank you for your help.
Best regards
Expected behavior
get as before the update all the payment methods like bankwire, ...
Actual behavior
Since the upgrade to 4.3.3, no payment methods available
Steps to reproduce the behavior
Upgrade the module to 4.3.3
Environment

PrestaShop version: <IP_ADDRESS>, or <IP_ADDRESS>
Mollie module version: 4.3.3
PHP version: 7.3

Hi @Ditwo ,
Thank you for your message.
Could you clarify where exactly you can't see the payment methods? Are they not showing in the front office? Or in the Mollie module itself? Have you tried checking this guide out here - https://github.com/mollie/PrestaShop/wiki/Troubleshooting#payment-method-does-not-appear-in-front-office ?
Looking forward to your reply!
--
Best regards,
Invertus Support team

Hello Team,
Thanks for your reply. Actually I meant that in Front Office, when checking out, there is no method anymore, even those that have nothing to see with Mollie, like bankwire.
With Mollie 4.3.3 deactivated, you can see Paypal and bankwire :

With Mollie 4.3.3 activated, no payment methods anymore :

And when I reinstall 4.3.2, it works again :

--> it happens on 2 differents sites. I think it is due to the 4.3.3 release.
Thank you for your help,
Denis

Thanks for the clarification.
As I can see the guide should help you resolve the issue - https://github.com/mollie/PrestaShop/wiki/Troubleshooting#payment-method-does-not-appear-in-front-office
Please follow all the steps, as we did not encounter any bugs on this release that might block the payment methods.
--
Best regards,
Invertus Support team

Hello,
I followed these steps but the issue is still there.  Anyway I'm not surprised because the wiki deals with Mollie payments, and not the other payments like bankwire and paypal.
What can I do ?

Hi,
We would need your back office + FTP credentials sent to -<EMAIL_ADDRESS>for further investigation, if the guide did not help.
Looking forward to your reply!
--
Best regards,
Invertus Support team

Hello Ditwo, hello Saimis777,
I have the same problem:
PrestaShop version: <IP_ADDRESS>
Mollie module version: 4.3.3
PHP version: 7.3

Since the upgrade of Mollie module to 4.3.3, no payment methods available.
Have you already found a solution for the problem?
Best regards,
Bernhard

Hi @ber-nie ,
We could not reproduce the issue on our end, with a default install, therefore asked for credentials to check, but have yet to receive them.
Did the wiki steps not helped you as well?
Looking forward to your reply!
--
Best regards,
Invertus Support team

Same Error here.
Payment Methods do not show in Frontoffice.
PS: <IP_ADDRESS>
Mollie Version 4.3.3
PHP Version: 7.2.34

Hi @waschier-design ,
Maybe you could provide us with your BO + FTP credentials to -<EMAIL_ADDRESS>for further investigation please?
Looking forward to your reply!
--
Best regards,
Invertus Support team

Thank you, @Saimis777.
This is also the solution for me.
Mollie-Payment works again after changing the language-code from "de" to "de-de".

Thank you for the clarification @ber-nie ,
Since the issue is resolved for everyone, I am closing it for now.
Let me know if you have any other questions!
--
Best regards,
Invertus Support team


Fix race condition in TestFlowAggregator_Run

There was a race condition caused by assigning mock exporter expectations while running exporter.
Fixes #4913.

/skip-all


the pt command built by go get -u does nothing

I might be wrong, but
go get -u github.com/monochromegane/the_platinum_searcher/...
resulted in a pt command without doing anything.
I had to do
go build -o pt cmd/pt/main.go
instead to build a workable pt command in the source directory.
I'm not familiar with the golang yet, but I guess you need to add something for building a workable pt command with the go get -u command sequence.
The execution environment is based on FreeBSD 11.0-STABLE and Ports (go-1.8_1,1).

Thank you for reporting.
I try the go get -u github.com/monochromegane/the_platinum_searcher/...  on my macOS, and run successfully.
I don't have FreeBSD environment, so please execute the following commands, and tell me the output.
$ go install -v github.com/monochromegane/the_platinum_searcher/...

$ ls -l `which pt`

@monochromegane
go install -v github.com/monochromegane/the_platinum_searcher/...
generated no output.
And the ls command generates the following:
ls -l `which pt`
-rwx------  1 kenji  k2r  4362184 Apr  9 13:19 /home/kenji/golang/bin/pt*

BTW On FreeBSD the env variables are:
GOPATH=/home/kenji/golang
GOROOT=/usr/local/go

Thank you. It was reproduced on my environment.
$ go install -v github.com/monochromegane/the_platinum_searcher/...
github.com/monochromegane/the_platinum_searcher
github.com/monochromegane/the_platinum_searcher/cmd/pt
github.com/monochromegane/the_platinum_searcher/lib/pt

$ pt
$

In this case, go install put lib/pt to our bin directory. The lib/pt is C binding program, so nothing to do.
If go install choise cmd/pt after lib/pt, run scccessfully.
I will extract lib/pt (it's beta feature) from this repository.

@monochromegane
I think giving lib/pt an other name than pt will make things less confusing.
And I've also found the

github.com/monochromegane/the_platinum_searcher
github.com/monochromegane/the_platinum_searcher/cmd/pt
github.com/monochromegane/the_platinum_searcher/lib/pt

sequence was followed in the go test command.
I'll stick to the go build -o pt cmd/pt/main.go for the time being.

I think giving lib/pt an other name than pt will make things less confusing.

Yes, but I thought it was not good to install the unuse command (only C binding binary) when go get pt. So, I remove lib/pt from this repo.

@monochromegane Fair enough. Thx for checking this out! :)

@monochromegane Rebuilding https://github.com/monochromegane/the_platinum_searcher/commit/c7d8eec66dca50773e6b4ee7dfdad2174860b9b1 with go get -u solved the issue. Thx a lot!


Lua Error 3.1.22

🔵 LUA ERROR:
A clear and concise description of what the bug is.
🔵 Just Login: (Steps to reproduce the behavior)

Go to '...'
Click on '....'
Scroll down to '....'
See error

🔵 **Message: Interface/AddOns/GSE/API/Serialisation.lua:286: attempt to compare nil with number
Time: Sat Nov 26 14:07:04 2022
Count: 15
Stack: Interface/AddOns/GSE/API/Serialisation.lua:286: attempt to compare nil with number
[string "@Interface/AddOns/GSE/API/Serialisation.lua"]:286: in function performVersionCheck' [string "@Interface/AddOns/GSE/API/Serialisation.lua"]:411: in function ?'
[string "@Interface/AddOns/DBM-Core/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua"]:119: in function <...ore/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:119>
[string "=[C]"]: ?
[string "@Interface/AddOns/DBM-Core/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua"]:29: in function <...ore/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:25>
[string "@Interface/AddOns/DBM-Core/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua"]:64: in function `Fire'
[string "@Interface/AddOns/Details/Libs/AceComm-3.0/AceComm-3.0.lua"]:260: in function <...face/AddOns/Details/Libs/AceComm-3.0/AceComm-3.0.lua:242>
Locals: :**
Paste the error message in this blockquote.

🔵 **

:**
If applicable, add screenshots to help explain your problem.
🔵 Expected behavior:
A clear and concise description of what you expected to happen.
🔵 GSE.lua file:
Please provide your GSE.lua file or the export string for the specific macro that is causing an issue.

To find the GSE.lua file:

First, make sure you have enabled the "File Name Extensions" checkbox in Explorer (View tab)
Browse to C:\path\to\wow\_retail_\WTF\Account\YourAccountName\SavedVariables\ or /path/to/WoW/_retail_/WTF/Account/YourAccountName/SavedVariables
Copy the GSE.lua file to your Desktop
Rename this copy to GSA.lua.txt
Then just drag it here in your message

Note: If you do not include your GSE.lua file you are wasting your time and mine. 90% of the time, the first thing I will ask for is "Can you please upload your GSE.lua file?".  This file is needed with the error to work out Why that error occured for you.

🔵 Desktop (please complete the following information):

OS: [e.g. Windows 10 64bit]
Game Version [e.g. classic/bcc/retail (9.1.0)]

🔵 3.1.22:

Version: [e.g. 3.x.y]
Downloaded From: [e.g. https://www.curseforge.com]

🔵 Additional context:
Add any other context about the problem here.

Moving this to a discussion
Apart from going you have an error most likely with what you are trying to import - I need the actual what you are trying g to inport


Update MD file on new branch 2017-11-13_02-35-57

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/content1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


org-version is N/A on native-comp, even after trying hacks

Hi,
First off: great work on this package! Both the documentation and the functionality are stellar.
However, as as documented, there are certain issues with org-mode. I'm trying to set this up on the feature/native-comp branch (using NixOS), but neither the old workaround and the new one seem to fix the issue. It seems that whatever I try, Emacs chokes on the org setup, reporting that "N/A" is not a valid version.
I can reproduce it with the following minimal configuration:

Emacs config that loads straight, use-package, and
org
(setq straight-use-package-by-default t
      straight-fix-flycheck t)

(defvar bootstrap-version)
(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
      (bootstrap-version 5))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))
(message "Configured straight.el")

(straight-use-package 'use-package)

(use-package git)
(use-package org
  :init

  ;; The following is a temporary hack until straight.el supports
  ;; building Org, see:
  ;;
  ;; * https://github.com/raxod502/straight.el/issues/211
  ;; * https://github.com/raxod502/radian/issues/410
  ;;
  ;; There are three things missing from our version of Org: the
  ;; functions `org-git-version' and `org-release', and the feature
  ;; `org-version'. We provide all three of those ourself, therefore.

  (defun org-git-version ()
    "The Git version of org-mode.
  Inserted by installing org-mode or when a release is made."
    (require 'git)
    (let ((git-repo (expand-file-name
                     "straight/repos/org/" user-emacs-directory)))
      (string-trim
       (git-run "describe"
                "--match=release\*"
                "--abbrev=6"
                "HEAD"))))

  (defun org-release ()
    "The release version of org-mode.
  Inserted by installing org-mode or when a release is made."
    (require 'git)
    (let ((git-repo (expand-file-name
                     "straight/repos/org/" user-emacs-directory)))
      (string-trim
       (string-remove-prefix
        "release_"
        (git-run "describe"
                 "--match=release\*"
                 "--abbrev=0"
                 "HEAD")))))

  (provide 'org-version))

(provide 'init)
;;; init.el ends here

Is there anything else to this workaround (that I'm missing), or
should this work in theory?
Versions

Emacs version: GNU Emacs 28.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.24, cairo version 1.16.0)
straight-version: prerelease (HEAD -> master, origin/master, origin/HEAD) 2d407bc 2020-12-20
uname -a: Linux nixos 5.9.16 #1-NixOS SMP Mon Dec 21 12:28:21 UTC 2020 x86_64 GNU/Linux

This is all the info I can think to provide right now. If you need anything else, please don't hesitate to ask.
Again, thanks for all your work and time on this!

Oops, sorry. This should have been on straight.el. I'll close this one and open a new one there. My bad!

Oops, sorry. This should have been on straight.el. I'll close this one and open a new one there. My bad!


Don't use LocalQuorum as default consistency for SimpleStrategy keyspaces

By default the driver performs each query with the LocalQuorum consistency.
This choice of consistency causes trouble for multi-dc clusters with SimpleStrategy.
Imagine a cluster with 3 datacenters (dc1, dc2, dc3), with 3 nodes in each datacenter.
Now let's say there's a keyspace:
CREATE KEYSPACE ks WITH replication = {'class': 'SimpleStrategy', 'replication_factor': 3};

'replication_factor': 3 means that for each partition there are 3 nodes in the whole cluster that are replicas for this partition.
Let's say the replicas are: A (in dc1), B (in dc1) and C (in dc2).
Now the user performs a query, and let's say that the load balancing chooses C to be the coordinator. Because the query uses LocalQuorum as the default consistency, the query will fail - there aren't enough nodes in dc2 to fulfill the consistency requirement, there would need to be at least 2.
I think it would be generally better to have the default be Quorum instead of LocalQuorum. It's the safest option.
Sadly changing this would count as a breaking change.

Note that we'd like to deprecate SimpleStrategy (see https://github.com/scylladb/scylladb/issues/5224 and others).

@Lorak-mmk / @wprzytula since we shouldn't support SimpleStartegy with multiDC, can we close this one?

@Lorak-mmk / @wprzytula since we shouldn't support SimpleStartegy with multiDC, can we close this one?

If the reason for that is that Scylla won't support SimpleStrategy with multiDC, then remember that Cassandra will still support it, and we want to correctly support Cassandra as well. Right?

Yes, but also for Cassandra it's highly advisable not using SimpleStrategy with Multi DC configuration.


Redeploying webscript with formatStyle in URI annotation is not effective

I tried to update a jar with a webscript without a formatStyle in to one with formatStyle = FormatStyle.ARGUMENT. The change was not effective until I restarted the OSGI container.

The url parsing pattern is cached in the Alfresco Webscript cache.
That is why you will still see your extension as part of a UriVariable or vice versa.
The fastest way to resolve this is refreshing the webscript index at /alfresco/service.
Resetting the OSGi container does a lot more then that.


no checkpoint files found in ./ckpts

The problem is that no inference has been made to the model to produce a checkpoint? Or do you need something
The steps I take are as follows：

(mixtralkit) root@fb05uge0jt0fc-0:/T106/model_main/MixtralKit-main/MixtralKit-main# python tools/example.py -m ./ckpts -t ckpts/tokenizer.model --num-gpus 2
Traceback (most recent call last):
File "/T106/model_main/MixtralKit-main/MixtralKit-main/tools/example.py", line 63, in
main()
File "/T106/model_main/MixtralKit-main/MixtralKit-main/tools/example.py", line 42, in main
generator = Mixtral.build(
File "/T106/model_main/MixtralKit-main/MixtralKit-main/mixtralkit/mixtral/generation.py", line 76, in build
assert len(checkpoints) > 0, f"no checkpoint files found in {ckpt_dir}"
AssertionError: no checkpoint files found in ./ckpts
What should I do about it
@tonysy

You need to download the checkpoints mannually.

Thanks you
I didn't replace the model path, it should be “ python tools/example.py -m./model -- path-t model -- path/tokenizer.model --num-gpus 2”, right？

Thanks you I didn't replace the model path, it should be “ python tools/example.py -m./model -- path-t model -- path/tokenizer.model --num-gpus 2”, right？

Yes, you can try it.

Feel free to re-open if needed.


bzip2 release has broken links in bin directory

Describe the bug
bzip2 1.0.8-1 release has broken links in bin directory which causes problems while assembling a Python package with it.
$ ls -al bzip2-1.0.8-1-aarch64-linux-android/bin
total 632
drwxr-xr-x@ 13 feodor  staff    416 14 Feb 14:06 .
drwx------@  6 feodor  staff    192 26 Apr 14:18 ..
-rwxr-xr-x@  1 feodor  staff  98160 14 Feb 14:06 bunzip2
-rwxr-xr-x@  1 feodor  staff  98160 14 Feb 14:06 bzcat
lrwxrwxrwx@  1 feodor  staff    131 14 Feb 14:06 bzcmp -> /home/runner/work/cpython-android-source-deps/cpython-android-source-deps/bzip2/build/1.0.8/aarch64-linux-android/prefix/bin/bzdiff
-rwxr-xr-x@  1 feodor  staff   2140 14 Feb 14:06 bzdiff
lrwxrwxrwx@  1 feodor  staff    131 14 Feb 14:06 bzegrep -> /home/runner/work/cpython-android-source-deps/cpython-android-source-deps/bzip2/build/1.0.8/aarch64-linux-android/prefix/bin/bzgrep
lrwxrwxrwx@  1 feodor  staff    131 14 Feb 14:06 bzfgrep -> /home/runner/work/cpython-android-source-deps/cpython-android-source-deps/bzip2/build/1.0.8/aarch64-linux-android/prefix/bin/bzgrep
-rwxr-xr-x@  1 feodor  staff   2054 14 Feb 14:06 bzgrep
-rwxr-xr-x@  1 feodor  staff  98160 14 Feb 14:06 bzip2
-rwxr-xr-x@  1 feodor  staff  13984 14 Feb 14:06 bzip2recover
lrwxrwxrwx@  1 feodor  staff    131 14 Feb 14:06 bzless -> /home/runner/work/cpython-android-source-deps/cpython-android-source-deps/bzip2/build/1.0.8/aarch64-linux-android/prefix/bin/bzmore
-rwxr-xr-x@  1 feodor  staff   1259 14 Feb 14:06 bzmore

Other distros seem to be fine, for example xz-5.4.6-0 has relative links:
drwxr-xr-x  25 feodor  staff     800 14 Feb 14:10 .
drwxr-xr-x   6 feodor  staff     192 26 Apr 14:01 ..
lrwxr-xr-x   1 feodor  staff       2 14 Feb 14:10 lzcat -> xz
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 lzcmp -> xzdiff
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 lzdiff -> xzdiff
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 lzegrep -> xzgrep
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 lzfgrep -> xzgrep
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 lzgrep -> xzgrep
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 lzless -> xzless
lrwxr-xr-x   1 feodor  staff       2 14 Feb 14:10 lzma -> xz
-rwxr-xr-x   1 feodor  staff   63552 14 Feb 14:10 lzmadec
-rwxr-xr-x   1 feodor  staff   84472 14 Feb 14:10 lzmainfo
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 lzmore -> xzmore
lrwxr-xr-x   1 feodor  staff       2 14 Feb 14:10 unlzma -> xz
lrwxr-xr-x   1 feodor  staff       2 14 Feb 14:10 unxz -> xz
-rwxr-xr-x   1 feodor  staff  404096 14 Feb 14:10 xz
lrwxr-xr-x   1 feodor  staff       2 14 Feb 14:10 xzcat -> xz
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 xzcmp -> xzdiff
-rwxr-xr-x   1 feodor  staff  135976 14 Feb 14:10 xzdec
-rwxr-xr-x   1 feodor  staff    7424 14 Feb 14:10 xzdiff
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 xzegrep -> xzgrep
lrwxr-xr-x   1 feodor  staff       6 14 Feb 14:10 xzfgrep -> xzgrep
-rwxr-xr-x   1 feodor  staff   10335 14 Feb 14:10 xzgrep
-rwxr-xr-x   1 feodor  staff    1815 14 Feb 14:10 xzless
-rwxr-xr-x   1 feodor  staff    2192 14 Feb 14:10 xzmore

Steps to reproduce
List detailed contents of any archive from this release: https://github.com/beeware/cpython-android-source-deps/releases/tag/bzip2-1.0.8-1
Expected behavior
Relative links.
Screenshots
No response
Environment

Operating System: macOS
Python version: 3.12

Logs
No response
Additional context
No response

Thanks for the report; I would accept a pull request that fixes this. But what problems does it cause when building Python? That should only need the include and lib directories.

It's shutil.copytree() that failing while trying to copy those links, but it could be fixed by adding ignore_dangling_symlinks=True.
So, what would be the fix:

fixing links;
removing links or bin directory all together?

Is this error coming from Python's own build system, or one of your own scripts?

If it's your own script, you can probably just remove the bin directory before copying anything.
If it's Python's own build system, please let me know which version you're building, and post the relevant part of the build log.

The proper fix would be to make the bzip2 build script produce relative links. For example, the last line of the script is currently:
make install PREFIX=$prefix

Maybe this could be changed to:
cd $prefix
make install PREFIX=.

That's my own script to build Python with the structure to work with mobile-forge. Mobile forge doesn't need bin though.

Mobile-forge doesn't really have Android support yet. You should use Chaquopy's build-wheel tool instead.

I'm working on a proof-of-concept of Android support in mobile-forge and it looks promising so far.


Feature Request: Clean up assertions

Feature Idea
The assertion functions should be reviewed and duplicates and functions which are no longer used should be removed. For example assert_function() covers assert_function_params().
Relevant Input
No response
Relevant Output
No response
Reproducible Example/Pseudo Code
No response

0
asserts

1
assert_character_scalar

2
assert_vars

3
assert_logical_scalar

4
assert_data_frame

5
assert_s3_class

6
assert_list_of

7
assert_function_param

8
assert_numeric_vector

9
assert_character_vector

10
assert_date_vector

11
assert_integer_scalar

12
assert_symbol

13
assert_function

14
`assert_db_requirements

15
`assert_terms

16
assert_db_requirements

17
assert_terms

18
assert_varval_list

19
assert_param_does_not_exist

20
assert_expr

21
assert_filter_cond

22
assert_unit

23
assert_expr_list

24
assert_same_type

25
assert_list_element

26
assert_one_to_one

27
assert_date_var

28
!!assert_expr

29
assert_named_exprs

30
[assert_valid_queries

31
assert_valid_queries

32
assert_has_variables

33
`assert_one_to_one

Is assert_s3_class() redundant to assert_function_param() and assert_list_of() based on these instances? @bundfussr

@pharmaverse/admiral
Looks like

assert_function() covers assert_function_params()
assert_date_var() and assert_date_vector() do relatively similar things, ie check for is.instant() most likely can be consolidated?
assert_date_frame() 's argument required_vars looks like it covers assert_has_variables()
assert_named_expr() is only used in admiral::derive_basetype_records(), but looks like it can be swapped with assert_expr_list()

Proposed decision is the deprecate:

assert_function_params()
one of assert_date_var()/assert_date_vector() (based on naming convention I like _vector more)
assert_has_variables()
assert_named_expr()

@pharmaverse/admiral, what do you all think? Ping me for mural access (didn't want to link it on this github issue)
Looks like

assert_function() covers assert_function_params()
assert_date_var() and assert_date_vector() do relatively similar things, ie check for is.instant() most likely can be consolidated?
assert_date_frame() 's argument required_vars looks like it covers assert_has_variables()
assert_named_expr() is only used in admiral::derive_basetype_records(), but looks like it can be swapped with assert_expr_list()

Proposed decision is the deprecate:

assert_function_params()
sounds good to me
one of assert_date_var()/assert_date_vector() (based on naming convention I like _vector more)
I like vector more as well
assert_has_variables()
umm...do you think we might need this...i can't think of any reason...
assert_named_expr()
sounds good to me

@pharmaverse/admiral After investigating a little more into how each of assert_date_var/assert_date_vector is used in admiral, there are slightly different use cases, side-by-side screenshot of code for reference above.

assert_date_var is used frequently in loops where there are multiple source datasets, in which the assertion becomes useful in providing messaging for indexing which columns from which dataframes there might be an issue with date/datetime
assert_date_vector is used frequently in compute_ function where there's no direct tie to a dataframe in the arguments

As such, I'm unsure if there is a need to deprecate the assert_date_* functions although I can't help but think these two still can be combined somehow, perhaps a better task would be to leave better documentation when to use each one?

@zdz2101 , what about assert_varval_list()? I wonder if it should be deprecated in favor of assert_expr_list().


Free Code Camp: Challenge [Nest an Anchor Element within a Paragraph

I think there is a bug here because I followed the format of the example in my answer. The example code was:
Here's a  link to Free Code Camp for you to follow.
The code I entered, one line below the  tags was:
View more cat photos
There were 8 tests run for this challenge, but only one test failed. The test that failed was “your p element should have the text “View more “ (with a space after it). However, I included the exact words "View more " with a space after it. I would like to pass this challenge and move on to the next one. I would like this problem resolved. Thank you.

@jimawofadeju I cannot see the rest of your code but I think I know what the "issue" is. A hint that's in the instructions is:

Now nest your existing a element within a new p element (just after the existing h2 element)...

If that doesn't help, please copy-paste all of your code for us to determine if it is a bug in the site or there's an issue with your code. Thanks!

I just copied and pasted all of my code for this challenge. I submitted a new issue with the same title as the old issue.

On Apr 24, 2016, at 9:08 PM, Eric Leung<EMAIL_ADDRESS>wrote:
@jimawofadeju I cannot see the rest of your code but I think I know what the "issue" is. A hint that's in the instructions is:
Now nest your existing a element within a new p element (just after the existing h2 element)...
If that doesn't help, please copy-paste all of your code for us to determine if it is a bug in the site or there's an issue with your code. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@jimawofadeju looking at your code you've posted in #8277, my guess was right. There isn't anything wrong with the site. Please read the instructions carefully. The answer to fix your issue is in the instructions. But if you are still stuck with this challenge, please ask questions to the Help Room. Happy coding!

I did ask twice if anyone was able to pass the nesting an anchor element within a paragraph. No one responded and it has been 7 or 8 hours since I asked.

On Apr 24, 2016, at 9:20 PM, Eric Leung<EMAIL_ADDRESS>wrote:
@jimawofadeju looking at your code you've posted in #8277, my guess was right. There isn't anything wrong with the site. Please read the instructions carefully. The answer to fix your issue is in the instructions. But if you are still stuck with this challenge, please ask questions to the Help Room. Happy coding!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

I figured it out. I had two of the same anchor elements. I only need one. I must have copied and pasted the anchor element without removing the one I initially copied.

On Apr 24, 2016, at 9:20 PM, Eric Leung<EMAIL_ADDRESS>wrote:
@jimawofadeju looking at your code you've posted in #8277, my guess was right. There isn't anything wrong with the site. Please read the instructions carefully. The answer to fix your issue is in the instructions. But if you are still stuck with this challenge, please ask questions to the Help Room. Happy coding!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


feat: add jaas offer access resource

Description
This PR adds a new resource juju_jaas_access_offer building on top of the generic access structure.
Note that there is a minor issue with Juju application offer tags. In the Juju names/v4 package, an applicationoffer tag was based on an offer URL. In the names/v5 package this changed to an offer UUID. JAAS accepts both but currently the provider does not store the offer UUID (proposed change in #583). Until then I've opted to use the Juju names/v4 package to construct application offers based on an offer URL.
This PR is based on top of #581. See the last commit for the changes.
Partially resolves:  CSS-10638
Type of change

Add new resource

/merge


Upgrade from go2rtc version 1.2.0 linux/amd64 to go2rtc 1.7.0 on latest Frigate breaks MSE

Not sure when this happened, but i just upgraded my frigate from 12.0 to 13.0-beta2 and it does not show MSE streams in go2rtc anymore.
I logged this on frigate GitHub as well on https://github.com/blakeblackshear/frigate/issues/8056.

@AlexxIT  Ok, so I loaded the config from go2rtc in frigate into a stand-alone go2rtc so it’s using the direct rtsp streams.
same behaviour, only webrtc works. Not sure if there is a workaround available by setting it so that it always opens in webrtc ?

Not sure if helpful, but I’m using these cameras:
https://www.amazon.co.uk/gp/aw/d/B00KPSARL4?psc=1&ref=ppx_pop_mob_b_asin_title

@AlexxIT Any update on this? From other issues here and in Webrtc as well it seems whatever changed in MSE has been a problem for many users. Can we identify and rollback those changes ?

Well. No any useful info about this issue that can helps to find problem.

I guess you can compare the go2rtc (1.2.0) used in frigate 0.12.0 and see what changed there for MSE ?
Is there any debug mode which can be turned on which would be useful ?

Almost whole code base was rewritten since 1.2.0

But it still must be limited to MSE. What changes were made there?

From: Alex X @.>
Sent: 18 November 2023 16:07
To: AlexxIT/go2rtc @.>
Cc: atv2016 @.>; Author @.>
Subject: Re: [AlexxIT/go2rtc] Upgrade from go2rtc version 1.2.0 linux/amd64 to go2rtc 1.7.0 on latest Frigate breaks MSE (Issue #671)
Almost whole code base was rewritten since 1.2.0
—
Reply to this email directly, view it on GitHubhttps://github.com/AlexxIT/go2rtc/issues/671#issuecomment-1817546963, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEBCFU5Q32LQKL6IIEV5EUTYFDMKRAVCNFSM6AAAAAA5SP32QWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXGU2DMOJWGM.
You are receiving this because you authored the thread.Message ID: @.***>

Please let me know if the problem is relevant

Still an issue AFAIK. My Reolink E1 Zoom shows a black screen in preview and live mode, but streams fine in debug. Seems to be caused by this issue.

Same advice https://github.com/AlexxIT/go2rtc/issues/671#issuecomment-1748846171


The example presented for the challenge "Target the Parent of an Element Using jQuery" does not work

Target the Parent of an Element Using jQuery
The challenge presents the user with the example $("#left-well").parent().css("background-color", "blue") but that example does not create a background color. By looking at the picture you can tell it does not work which begs the question, why even add the example to the challenge?
PS. I found the correct method for the challenge, but I still don't understand why you would show a different method to the user when the method does not work correctly.

The example code does work. What you have in the editor does not match the example code.

@sludge256 How is it wrong? I did exactly what it asks, I have typed the layout multiple times and I even copied the example to compare and check what I am doing wrong. I have gone over this multiple times and yet I don't see anything wrong.

If you look at the the example code there is nothing within the parenthesis of parent() while your code does.

I hate to break it to you but the parent() is between ("#left-well") the .css
PS. the example is as follows $("#left-well").PARENT().css("background-color", "blue")
(the parent() is capitalized so you can see what I am talking about).

The code you used in your screenshot is:
$("#left-well").parent("#target1").css("background-color", "red")

This is what my answer is but the challenge is not accepting it. And what about the .parent() method? I still need it just like the challenge says.

You are calling the parent() method on #left-well when the instructions ask for it to be called on #target1.

The way I am reading it is like this it's saying to put #target1 between the empty parenthesis.


Refactor conformance test infrastructure a little

Use IsTest to define tests
Provide a helper for discovering a hierarchy of tests
Fix the issue with the "safe" eval function throwing exceptions. As it
says in its Haddock, it's partial if the input has free variables, so we
need to check that.
Use the version from the input program, not the default one. The
previous behaviour seems wrong. This required re-generating the output
files.

Fixed merge conflicts.

Added a little more docs.


Intrinsicsfix

I'm unsure, but is having it check for either x86 or x64 redundant? It's always going to be one of those two for standard hardware, right? Removing those checks completely strikes me as being the simpler approach if it's always going to pass on one of those.
Otherwise, this looks pretty awesome.

@Areloch Possibly for files that are specific to the target architecture, but not for files like oggTheoraDecoder where it needs to know what the target platform is. Especially if T3D were to be ported to Android (ARM majority) or other platforms/devices.

Muh PowerPC support.


apromise/catch-or-return will cause many empty callback of match

when i start promise/catch-or-return rule, i must add catch method, although i don't care about error logic .
empty catch callback makes me feel weird.

That's basically the purpose of the rule. Are you proposing taking this out of the recommended config?


feat: 🎸 Copy button for format Text & JSON

✅ Closes: #98

I have miss file .env.example. Can you add it for me? I'm sorry about :( It's first my PR @Balastrong

@all-contributors please add @K1ethoang for code


Update AVClient to Foundry VTT 0.7.9

Which class should be updated?
AVClient
For which version should the class be updated?
Foundry VTT 0.7.9

Taking this

Fixed by #291


Added Endpoint type name example name

I was a bit confused when setting up my custom domain name for the generated API. As per the instructions in the readme, I was looking for my Gateway API endpoint ending with .cloudfront.net. The instructions may be clear for the more experienced AWS users but it might be a bit confusing for novice developers.
P.S.: Thank you for the awesome script! I am very happy that I could shut down my EC2 bitwarden instance and have everything in a serverless form.

The CI complains about AWS provider credentials not found which is not related to my PR

Thanks!