[BUG] Gmod.NET is not working on Ubuntu 18.04 and older

Bug description
Garry's Mod Dedicated Server is not able to load Gmod.NET on Ubuntu 18.04 (newer versions work just fine).
Error message:
[ERROR] lua/autorun/start.lua:1: Couldn't load module library!

  1. require - [C]:-1

   2. unknown - lua/autorun/start.lua:1

Can be reproduced with the following Dockerfile:
FROM ubuntu:18.04
WORKDIR /steam
RUN apt-get update
RUN apt-get install -y lib32gcc1
RUN apt-get install -y curl
RUN curl -sqL "https://steamcdn-a.akamaihd.net/client/installer/steamcmd_linux.tar.gz" | tar zxvf -
RUN ./steamcmd.sh +login anonymous +force_install_dir /gmod "+app_update 4020 -beta x86-64 validate" +quit
WORKDIR /gmod/garrysmod/lua/bin
RUN curl -sqL "https://github.com/GmodNET/GmodDotNet/releases/download/0.7.1/gmod-dot-net-linux.0.7.1.tar.gz" | tar zxvf -
COPY start.lua /gmod/garrysmod/lua/autorun
RUN apt-get install -y libc6 libgcc1 libgssapi-krb5-2 libicu60 libssl1.1 libstdc++6 zlib1g
WORKDIR /gmod

ENTRYPOINT ./srcds_run_x64 -game garrysmod -systemtest -condebug +sv_hibernate_think 1

Posible cause
Since we are building C++ code of the Gmod.NET on Ubuntu 20.04, it is likely that we are implicitly linking to something (like newer libc version) which is not available on Ubuntu 18.04.

Can we update some library or link to library without version in name?

The problem is indeed an older version of libc and libstdc++. Here is an Ubuntu 18.04 dockerfile with a fix:
FROM ubuntu:18.04
WORKDIR /steam
RUN apt-get update
RUN apt-get install -y lib32gcc1
RUN apt-get install -y curl
RUN curl -sqL "https://steamcdn-a.akamaihd.net/client/installer/steamcmd_linux.tar.gz" | tar zxvf -
RUN ./steamcmd.sh +login anonymous +force_install_dir /gmod "+app_update 4020 -beta x86-64 validate" +quit
WORKDIR /gmod/garrysmod/lua/bin
RUN curl -sqL "https://github.com/GmodNET/GmodDotNet/releases/download/0.7.1/gmod-dot-net-linux.0.7.1.tar.gz" | tar zxvf -
COPY start.lua /gmod/garrysmod/lua/autorun
RUN apt-get install -y libc6 libgcc1 libgssapi-krb5-2 libicu60 libssl1.1 libstdc++6 zlib1g
RUN apt-get install -y software-properties-common
RUN add-apt-repository ppa:ubuntu-toolchain-r/test
RUN apt-get update
RUN apt-get install -y gcc-9 g++-9
WORKDIR /gmod

ENTRYPOINT ./srcds_run_x64 -game garrysmod -systemtest -condebug +sv_hibernate_think 1

TLDR if we install a gcc version 9 from experimental channel on Ubuntu 18.04, it brings newer libc and libstdc++ versions to system, and Gmod.NET works just fine.
Since we are using C++ 17 features in our bootstrapping module, we can't downgrade our build pipeline to Ubuntu 18.04 and GCC 7.
We need to create a docs article which will state that Linux distribution should be equipped with GCC 9 and newer (Ubuntu 20.04) and if upgrade of the OS is not available, install newer version of GCC manually.

Hello guys, this is quite a common issue with Gmod modules. For my modules I build using a manylinux2014 Docker container which uses the ancient CentOS 7, and works even on the shittiest of Russian hosts.
Here is the github actions yml, hopefully it will help you
https://github.com/WilliamVenner/gmsv_workshop/blob/master/.github/workflows/build.yml


Making some internal fields public.

Hi!
I'd like to access the information on where the glyf::Table data starts and where it ends. I understand that I could do it by accessing the glyf::Table::data. Unfortunately, this field is pub(crate) only. Would it be possible to make it fully public, please? :)

I think you should raise this at ttf-parser this repo just provides owned data support on top of that crate.

Oh, you are right. I was planning to raise it there, but somehow I probably switched my tabs. Thanks! :)


Jittery on Mobile

I'm having issues where the scroll is jittery on my mobile device. Do you have similar issues? If so, got a solution? Love the plugin though—works great on desktop.

On the demo too, or your specific implementation? What device/browser. There’s a few things this could be.

It's on my own project. I put in a javascript filter to deactivate it on mobile and keep it activate on desktop. It gets the job done, but it would be nice if it worked on both.

I've seen this happen before. The cause is usually a third-party script not debouncing their scroll events. I'm assuming you're using some other scripts on the page, too.
Modern desktop browsers "debounce" scroll events so that they don't fire too frequently when scrolling quickly down a page. This prevents jank. Many mobile browsers, esp on slightly older devices, don't.
Smooth scroll, by the very nature of scrolling down the page, triggers scroll events.
If a third party script hasn't added debouncing for scroll events, the event callbacks try to fire thousands of times a second and bog down the entire UI.
Since the demo doesn't have the problem and I can't provide support for third-party scripts, I'm closing this one. Sorry!

Sounds good! I deactivated all the scripts besides standard jQuery and SmoothScroll.js and it is still being jittery. I'm using an Iphone8 X with Chrome. Again, detecting whether or not you're viewing the site on a mobile device then activating/deactivating it accordingly is a good workaround. Appreciate the help!
For anyone else who might be having this issue, this is what I added to the end to deactivate it on mobile.
var isMobile = /iPhone|iPad|iPod|Android/i.test(navigator.userAgent);
if (isMobile) {
} else {
new SmoothScroll('.smooth', params)
}


Incompatibility with @opentelemetry/api 1.8.0

We’ve lost a week of tracing because these don’t play nice:
"@honeycombio/opentelemetry-node": "^0.7.0"
"@opentelemetry/api": "^1.8.0",

This error is logged:
Error: @opentelemetry/api: Registration of version v1.8.0 for trace does not match previously registered API v1.7.0

I’ve traced it back to this commit…
The stated intention of PR#268 was to avoid:

Experimental packages can have bigger (even breaking) changes in a minor version, which makes a caret (minor) unpredictable. A tilde can be used for allowing patches only.

However, I think the @opentelemetry/api can be trusted for proper versioning.
In the meantime, I need to either pin @opentelemetry/api to 1.7.0 until PR#286 is resolved, or remove my explicit dependency.

Thanks for reporting! Release v0.7.1 is now available that should help resolve this problem!

Thanks for the rapid response.  0.7.1 works for us and resolves this issue.
I'm still a little concerned about the use of the ~ for the api version... it's essentially saying "we don't trust @opentelemetry/api to properly use semantic versioning."
As it stands, this specific problem is quite likely to occur on the next version bump.

I'd like to also request reconsidering the use of ~ for versions - or, alternatively, make loud and obvious error messages in case of version mismatch.
In our case, we're trying to get up and running with Honeycomb and OpenTelemetry, but none of our parent span relationships were working. The problem was the result of @honeycombio/opentelemetry-node pinning to different versions of @opentelemetry packages - that resulted in two different installations of @opentelemetry packages and two different instances of the global OTel context manager, which broke span resolution.

More information, in case it helps: Changing @honeycombio/opentelemetry-node's dependencies for @opentelemetry/api only from ~ to ^ appears sufficient to fix the incompatibility we noticed. So, at least for that issue, dependencies on other OTel packages can continue to use ~ as in #268. (All my tests were done with the Yarn 4.x package manager.)
Another possible solution, as I understand it, would be to specify "@opentelemetry/api": "~1.7.0" (etc.) under  @honeycombio/opentelemetry-node's peerDependencies - then users' package managers can warn them about potential incompatibilities.

Thanks for the input @joshkel and @zaq42 ! I opened PR #299 to at least ease the restriction on the API version. I'm not positive on the definite best route to take, as the overall intention is indeed to minimize incompatibilities. The peerDependencies might be something to consider as well, though I believe that will require installs if those packages are not installed by the end user (which they may not need to be). Perhaps peerDependenciesMeta is the solution 🤔  as it allows peer dependencies to be marked optional.


get blog post from db, display on blog post page

comment text formatting is not working, probably a better way to store comments than as a JSON string of an array of objects
most other things are working fine and come from the db

Can't test.
I went to localhost:3000/test/new
Created a new post and its stuck at 'loading...' indefinitely
I see that your image has a slug tho.
I'm not getting any errors at the moment.

Can't test. I went to localhost:3000/test/new Created a new post and its stuck at 'loading...' indefinitely
I see that your image has a slug tho.
I'm not getting any errors at the moment.

i'll have to merge the changes from the create blog PR to get that working completely, but right now if you use the mock function on the /blog/new page, http://localhost:3000/TestBlog/new?user=TestBlog, then it'll use that in the redirection and if the post you're making already exists in the db, you'll be redirected there.


A 4th PNG decoding crate: imagine

Hi you don't know me but I was staring at the super long docs.rs releases queue right now and then i saw png_pong in the list and I have a PNG decoder crate so I thought "hey neat" and then in your crate it says that you know of 3 others, but I'm not on your list.
So, you can add https://github.com/Lokathor/imagine to your list if you like.
The version out on crates.io works, but forces allocation. The next draft has been revised many times and it's mostly working (will be finished "soon" tm), and it won't force allocations. Though it will have helper functions that can do the allocation for you.

Sure, why not?

Added to the list


[RayJob][Refactor] use ray job logs to capture logs and be tolerant of duplicated job submissions

Why are these changes needed?
This is a PoC of using ray job submit --no-wait + ray job logs --follow to be tolerant of duplicated job submissions.

Related issue number

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

I used a similar workaround in here https://github.com/ray-project/kuberay/blob/master/benchmark/perf-tests/10000-rayjob/pytorch-mnist-rayjob.yaml#L23-L24
The problem is that with either solution, either ray job submit or ray job logs will raise an exception and I'm concerned users will be misled to think their job failed

To summarize:
If ray job submit really fails, then ray job logs will not succeed either and k8s will retry the job.
If ray job submit fails with duplication submissions, then users can see a message like ValueError: Job with submission_id rayjob-sample-6tqdm already exists. Please use a different submission_id.. I think it is less confusing.

This lines up with the results I had. The problem is still dumping a stack trace from an exception, I think users will find that really confusing. I wonder if there's an option where we can run the first command but just pipe the output to /dev/null and use the exit code to determine what command to run next?

To summarize:
If ray job submit really fails, then ray job logs will not succeed either and k8s will retry the job.
If ray job submit fails with duplication submissions, then users can see a message like ValueError: Job with submission_id rayjob-sample-6tqdm already exists. Please use a different submission_id.. I think it is less confusing.

This lines up with the results I had. The problem is still dumping a stack trace from an exception, I think users will find that really confusing. I wonder if there's an option where we can run the first command but just pipe the output to /dev/null and use the exit code to determine what command to run next?

I think we can pipe to grep -zv 'Please use a different submission_id'. That can hide the stack trace of a duplication submission.

Worth mentioning that this may not be required if https://github.com/ray-project/ray/pull/45498 was merged, but I think there was pushback

Worth mentioning that this may not be required if https://github.com/ray-project/ray/pull/45498 was merged

We still need this PR if we want to maintain the same compatibility matrix between KubeRay and Ray.

@andrewsykim, I prefer the first one:
ray job list | grep <submission_id> && ray job logs --follow <submission_id> || ray job submit --submission-id <submission_id> -- python job.py

The second one, ray job status will print out partial job logs which I think is unnecessary unless we discard the output of ray job status anyway.

The nice thing about ray job status is that the exit code can be used to decide if the job exists or not, in which case we can discard the entire output, this seems a bit cleaner than ray job logs | grep ...

The nice thing about ray job status is that the exit code can be used to decide if the job exists or not (this needs further testing / validation), in which case we can discard the entire output, this seems a bit cleaner than ray job logs | grep ...

Updated. Now the full command looks like:
if ray job status --address http://test-url test-job-id >/dev/null 2>&1 ;
then ray job logs --address http://test-url --follow test-job-id ;
else ray job submit --address http://test-url --submission-id test-job-id -- echo hello world ;
fi

Thanks, that seems much cleaner IMO. Can you confirm that ray job status --address http://test-url test-job-id returns exit 0 even if a submission ID exists, but the job failed?

Thanks, that seems much cleaner IMO. Can you confirm that ray job status --address http://test-url test-job-id returns exit 0 even if a submission ID exists, but the job failed?

Yes, it still exits with 0 if the job failed.

Thanks @rueian, can you think of any other reason why ray job status would fail with exit 1? I guess if Ray head is not reachable, it would fail and run ray job submit, but worse case scenario this would result in the same issue as before with duplication submission IDs right?

Thanks @rueian, can you think of any other reason why ray job status would fail with exit 1? I guess if Ray head is not reachable, it would fail and run ray job submit, but worse case scenario this would result in the same issue as before with duplication submission IDs right?

Yes, if the Ray head is not reachable, it will fail and run ray job submit and will fail again.

LGTM, thanks @rueian -- will defer final review to @kevin85421


Update warnings for minimatch and gulp-util

Fixes warnings from installing your plugin due to next gulp version and deprecation of gulp-util.
Also updates minimatch (regexp dos issue) and other cleanups

Hi @sirlantis you need help maintaining this repo or how can I help getting this change into the code?

Hey @rejas - thanks for the contribution!
I'm indeed not actively maintaining this repo anymore, but will try this weekend to get this change in.

Thx for the invite. Maybe moving this plugin to https://github.com/gulp-community would be an option?
@phated @demurgos : thoughts about this plugin?

@rejas If that's an option, certainly willing to do so.

@sirlantis no response yet from @demurgos regarding the gulp-community move. maybe transfering it to me for now especially the npm rights would be an option?

Gulp-community discussion: https://github.com/gulp-community/meta/issues/1#issuecomment-399016804

so, what do you say @sirlantis? ready to move the repo?

Add me as a contributor to this repo and I'll do the transfer if it's OK for you.

I havent got the right but I hope @sirlantis reads this :-)


set up mono_repo configs

Closes https://github.com/dart-lang/macros/issues/2
I also added tool/dart_model_generator to the workspace config.

Neat, thanks.
dart_model_generator doesn't have any test yet, let me see if I can add a golden test... feel free to merge before, to wait, to add an empty test, or to tweak the config so it doesn't test, as you like :)

#13 :)

cc @davidmorgan we are somehow getting carriage returns on windows, I just changed it to use equalsIgnoringWhitespace for now, which will only capture meaningful changes... but we could end up swapping between carriage returns and not, if people generate on windows versus other platforms...

Curious, maybe add a TODO / issue somewhere? We probably don't want to accidentally publish with different line endings.


user management fixes

use latest STS to bring in user management bug fixes

handle the scenario that connection uri cannot be retrieved.

close the dialog if something goes wrong when initializing the dialog instead of keeping the loading dialog open forever.

fix a typo in the management service.

Are you also planning to disable must_change option for existing logins?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 41.964%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/dialog/configurePython/configurePathPage.ts
1
57.02%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
28812

Relevant Lines:
63749

💛  - Coveralls

Are you also planning to disable must_change option for existing logins?

not going to touch it for March.


Image files don't open

For example here, saying tree harp doesn't open the image:

I think instead of this
await vscode.window.showTextDocument(uri);

we should use this
await vscode.commands.executeCommand("vscode.open", uri);

I can include this in #13 if you want.

closed by #13


Please add release tag

Can you please create a release tag. I am using your library in other projects and would like to release based on tagged / tested versions.
Thanks for supplying the library.

Just created tag v0.1 for the current latest version. I will try to remember to update tags when I push new versions. Thank you for this issue report.


Duplicate and missing accelerators in localization

This is a tracking issue for the following related issues filed internally on VSO
It seems that the accelerators were not preserved from previous releases in the translation recycling process.

368010 [CHS] The ‘Default namespace’ & ‘Output type’ items have the same hot-key 'Alt + I' in Application page
368022 [KOR] The ‘Neutral language’ option have no hot-key just have ‘(_)’ in Assembly Information dialog
368870 [ESN] The ‘Output type’ & ‘Resource file’ items have the same hot-key: <Alt + S> in application page
368872 [FRA] The ‘Output type’ & ‘Startup object’ items have the same hot-key: <Alt + O> in Application page
368875 [ITA] The ‘Description’ and ‘Company’ option have the same hot-key: <ALT + S> in Assembly Information dialog
368881 [PLK] The ‘Assembly version’ and ‘File version’ option have the same hot-key: <ALT + W> in Assembly Information dialog

@janbaltus

All HKs issue fixed.

fixed with PR https://github.com/dotnet/roslyn-project-system/pull/1336


test: refactor git portforwarder under gitprovider

This refactors the git port forwarder abstraction to be underneath the git provider. The git port forwarder is only relevant with the in-cluster git server, and thus fits as an implementation detail of the LocalProvider.

/assign @karlkfi

/retest

/retest


Version 1.1.6 bug of checkbox and master version dictionaries initialize failure

There is a bug in Release version 1.1.6 that causes the system to crash if you try to uncheck your first dictionary by clicking the checkbox. When this happens, you will receive a 500 error, and you will need to delete the /flask-mdict-master/content/flask_mdict.db file and restart it.
The Master version (as of March 16th, 2023) has fixed this bug. However, it has a problem with initializing dictionaries correctly, which can cause users to get stuck on the first page. To use the Master version, you can first initialize your dictionaries with version 1.1.6 and then switch to the Master version.
For example, if you use O8C dictionary and you put O8C.jpg, O8C.mdd, O8C.mdx, O8C.css in /flask-mdict-master/content, after initializing you should find O8C.mdd.db, O8C.mdx.db, about_O8C.html, flask_mdict.db have been created correctly in the same folder.

Which version is your MDD dictionary? flask-mdict only support MDICT version 2.0.
May you put some log messages?

I don't know my version of MDD dictionary. Could you advise how to check it?
However, when I did the same command on v.1.1.6, it worked.
Please see the log messages as below.
Master version
sudo flask run --host=<IP_ADDRESS>
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
Initialize MDICT "O8C" {21D06180-EC2A-309F-B72E-22F973C49875} [Enable]...
Traceback (most recent call last):
  File "/usr/bin/flask", line 33, in <module>
    sys.exit(load_entry_point('Flask==1.1.2', 'console_scripts', 'flask')())
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 967, in main
    cli.main(args=sys.argv[1:], prog_name="python -m flask" if as_module else None)
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 586, in main
    return super(FlaskGroup, self).main(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3/dist-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3/dist-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3/dist-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/click/decorators.py", line 73, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "/usr/lib/python3/dist-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 848, in run_command
    app = DispatchingApp(info.load_app, use_eager_loading=eager_loading)
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 305, in __init__
    self._load_unlocked()
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 330, in _load_unlocked
    self._app = rv = self.loader()
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 392, in load_app
    app = locate_app(self, import_name, None, raise_if_not_found=False)
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 257, in locate_app
    return find_best_app(script_info, module)
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 83, in find_best_app
    app = call_factory(script_info, app_factory)
  File "/usr/lib/python3/dist-packages/flask/cli.py", line 119, in call_factory
    return app_factory()
  File "/home/pi/flask-mdict-test/flask-mdict-master/app.py", line 30, in create_app
    init_app(app, url_prefix='/')
  File "/home/pi/flask-mdict-test/flask-mdict-master/flask_mdict/__init__.py", line 45, in init_app
    mdicts, db_names = helper.init_mdict(Config.MDICT_DIR, Config.INDEX_DIR)
  File "/home/pi/flask-mdict-test/flask-mdict-master/flask_mdict/helper.py", line 292, in init_mdict
    idx = IndexBuilder2(mdx_file, index_dir=mdict_index_dir)
  File "/home/pi/flask-mdict-test/flask-mdict-master/flask_mdict/mdict_query2.py", line 49, in __init__
    self._make_mdx_index(self.get_index_db(self._mdx_file, self._index_dir))
  File "/home/pi/flask-mdict-test/flask-mdict-master/flask_mdict/mdict_query2.py", line 112, in _make_mdx_index
    super(IndexBuilder2, self)._make_mdx_index(db_name)
  File "/home/pi/flask-mdict-test/flask-mdict-master/flask_mdict/word_query/mdict_query.py", line 130, in _make_mdx_index
    returned_index = mdx.get_index(check_block = self._check)
AttributeError: 'MDX' object has no attribute 'get_index'

v 1.1.6
sudo flask run --host=<IP_ADDRESS>
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
Initialize MDICT "O8C" {261481B1-8127-31ED-94C4-EF953C26D839} [Enable]...
	+ O8C.mdx
	+ O8C.mdd
Add "Google 翻译" [Enable]...
 Pls update your local code.
git pull

Thank you. It works now.


[cxx-interop] Add std::set initializer that takes a Swift Sequence

This got unblocked by https://github.com/apple/swift/pull/64897.
rdar://107909624

@swift-ci please smoke test

@swift-ci please test

Nice, now we just need an insert as well :)

Oh good point! We'll need to decide on the return type though, so I'd prefer to add func insert in a separate PR.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


Tabs: deep linking

Yet another feature missing from Tabs in foundation 5: deep linking via url hash tag. Any plans to bring it back?
I've temporarily hacked a fix for current project, but more permanent solution is welcome!

Can't make tabs work. The solution? Make your own tabs .


[WIP] Column search

Add search inputs for each column header.
Things to do before merging

[ ] Move column search demo module to src/app/demos/
[ ] Make styling more consistent with the rest of the app

Closes #99

I've labeled commits that shouldn't be merged into master with DO NOT MERGE:, to make them easier to find later. These include things like changing travis' build scripts to be specific to this PR, and other minor things. When this PR is ready, I'll rebase those out.

For some reason, I can't seem to get it to work on angular 4, but if I upgrade to angular 5 it works. ng-packgr is failing silently when running npm run build:core.
I'm going to clean this branch up so that others can contribute to it more easily. (That way I can rebase only when master get's updated).

Hi @mklein994 nice work, I'm also in the middle of refactoring the examples and how they are generated as that's one of the main things that have prevented me from bumping the code to angular 5, once I'm done with that it should be easier for others to contribute I hope. I haven't done anything changes to the lib so it should be pretty easy to merge, feel free to check it out if you like: https://github.com/hjalmers/angular-generic-table/tree/feat/angular-5-update
I've just added one example using the new "format" which is available under '/demo/basic'.

@hjalmers I left a comment over here about some of those updates.

I rebased onto master, updating each commit to match the new style.

Rebase onto master (cypress tests).

@hjalmers OK: so this works... when the table is small. I took your advice wrote it using pipes, because that seemed to be the simplest option to extend it. Everything is performed synchronously, so when I enabled searchBox for some fields in the advanced demo, performance was taking a big hit, as you predicted. (It was also failing because I haven't implemented the search or value transformations yet).
If I were to re-implement this using observables and triggering it asynchronously, what would need to be updated?

Yeah, I first wrote the search function for AngularJs which had a lot of issues with performance due to two-way binding so I tried to minimize the amount of operations performed and just trigger one change and it becomes even trickier when involving multiple columns. I think a lot of the code from the global search could be reused however, but instead of merging all the columns into one string, it could go through the data one column at a time and only go through columns that are included in the search "object". The next iteration should only have to go through the rows that pass the first column check and so on, it shouldn't have to have have that big impact on performance.
It would be a lot nicer to use observables however but I think it would be hard to use observables just for this with out rewriting the other pipes too, I think it would be better to start from scratch and think about the flow which should be something like this:

filter
global search
column search (might be performed at the same time as global search)
sort
chunk

Perhaps starting out with a separate component that uses the same configuration and just re-add one feature at a time and use observables for everything.
I thought I'd try and draw a sketch of the flow using draw.io or something just to make the architecture clearer but that will have to wait a while I'm afraid as I'm going away on a holiday soon and won't be back until mid may. But if you have any suggestions feel free to share them, I'm might not be able to commit any code for a while but I should be able to check my mail from time to time and merge minor pull requests.

OK thanks for the heads-up.


Bifrost 1.17.10

Bifrost 1.17.10
does the new version on the computer download firmware to the Samsung S24? s24+ s24 ultra, does not download on the phone. writing is prohibited. I installed the previous one, everything is fine. I am attaching a screenshot.

This should be fixed in 1.17.11.


SDL3 VisualC project not building: unresolved external symbols

Windows 10, Win32 build produces these errors:
mixer.obj : error LNK2019: unresolved external symbol _Mix_LoadSndFile_RW referenced in function _Mix_LoadWAV_RW
mixer.obj : error LNK2019: unresolved external symbol _SNDFILE_uninit referenced in function _Mix_Quit

Looks like load_sndfile.h and load_sndfile.c didn't get added to the project.

I'll fix that, thanks.

Fixed with a8fd2c1afacf84699f37c1a73f072fcd23560ccb thanks


Adapt to https://github.com/coq/coq/pull/18880

Adapt to https://github.com/coq/coq/pull/18880

Looks good, thanks.


When revisiting projects, I'd like to read my conversation logs

Is your feature request related to a problem? Please describe:
I would like to read (and learn from) previous conversations while building an app.
I think I had a bug in Bolt and wanted to read the conversations and code written to understand if there was a bug learn more about bolt.
Describe the solution you'd like:
I'd like to read my conversation log and see what code was written/changed.

Describe alternatives you've considered:
It could be provided anywhere, just need to go back
Additional context:

This is coming soon! Updates will be posted in #39


[draguboard] Create draguboard app from dragboard

By using a bigger part of the touch area (vs the original dragboard), we can have bigger and more distanced characters to improve accuracy.

Cool!

The UI is drawn like this for me:

That's weird.  What's your firmware version?  Mine is 2v16.7.  Do you see the original dragboard normally?

Seems appRect is not initialized at the same time depending on the params, I'll rework it.  Thanks for testing.


Read/Write from XML elements instead of element attributes

Currently, NAS2D only supports reading from XML attributes. Sometimes this is suboptimal, especially when reading large amounts of text.
For example, in OPHD, we would likely like to remove the planet descriptions from being hardcoded. They would work well stored within an XML element, but would work poorly (it at all?) if stored in an XML attribute.
I haven't checked if TinyXML2 has this support built in. Maybe solvable but upgrading the library dependency.

I believe it would work fine with attributes. Though yes, it does seem a bit messy to store that amount of text in an attribute.
The Configuration class could potentially be extended to read from either attributes or from sub tags/values. I'm not sure how you would specify that for writing with the current API, but it should be easy enough to support reading from either attributes or sub tag values.
Either TinyXML1 or TinyXML2 should have no trouble supporting this feature.

I think I'd like to close this as out of scope for NAS2D.
The TinyXML1 and TinyXML2 libraries both already support reading and writing of XML element values. We don't really need to store values in attributes, this is just an artifact of re-using existing wrapper parsing code that does things that way. In terms of parsing from XML element values, this seems like a higher level parsing concern. In particular, OPHD already makes direct use of TinyXML, so this could be accomplished there.
Granted, we are trying to move away from direct use of TinyXML in OPHD, as loading data through an abstraction layer can make upgrading the library easier. Though that effort is somewhat hampered by some degree of custom parsing needed to handle custom file formats. In particular, reading XML element values would be such custom parsing. It might be that we end up maintaining custom parse routines in OPHD. Hopefully such routines will be a bit more generalized and re-usable, as current parse code is somewhat verbose and spread through the code base. In terms of eliminating it entirely from OPHD, that may not be entirely feasible.
Unless we can come up with a generalized format for parsing all data from a simple generic format, we should probably push this concern to higher level code.

I'm ok using TinyXML directly in OPHD to read from elements and closing this issue. I guess I didn't realize we could do that when writing this up.
-Brett

Ultimately I'd like to move away from XML as a savegame format. I did it initially so I could read the output in a text editor to ensure that things were behaving properly with the ultimate intention of saving it as a binary format somewhere down the line. Or some other human readable form but with a much, much smaller footprint.
If, on the other hand, we opt to stick with XML I want to ensure that whatever library is used has a vcpkg option for Windows and is also available on Linux/macOS.

Hmm, interesting looking library.
I'm fine if we stuck with XML, or another text format.
First issue though is to refactor the code so it is more decoupled from the specific library. Using vcpkg and upgrading, or considering alternate libraries will then be easier at that time.

I'm generally in agreement. I think it would make sense to upgrade to TinyXML 2 via vcpkg, port everything over to that and then strip the XML interface out of NAS2D entirely. I'm curious about thoughts using XML internally for the configuration class but if we go with a strictly dictionary type of approach we could build a basic custom parser that uses a sort of 'INI' type of deal with simple key/value pairs? Would eliminate the dependency on TinyXML in NAS2D itself and then individual projects (like OutpostHD) can decide how to handle it.
It would mean redefining Sprite definitions if XML were dumped. I can still see it being done simply enough using key/value pairs in some way though the XML definition does make it nice and neat. It could be a simple text type of document that's formatted in a particular way using some sort of delimiter, like commas or spaces? Thought now we're getting off the subject at hand and into different issue territory.
Anyway, food for thought.

Yeah, I'd considered that. Because of the current dependence on XML for Configuration and Sprite data, I would keep the dependency for now. I suppose long term you could look at removing the parse code, and instead only offer a constructor with already parsed data.  The client project can then choose which formats it wants to support, and which libraries to use to parse the data. I'm not sure if the classes would be as useful though. Hence that might never happen. Just a thought that we might revisit in the future, when we're in a better position to see where things might go.


Refactor: save lines in extended match and add censor method

Context
After having worked on factorizing make_matches the new goal is to start storing context lines in the Extended Match and have the output entirely depend those line and no longer the result.file.content.
In particular this means a way to censor Extended Match based on the lines and the indices in MatchSpan and no longer relying on censor content.

What has been done
This MR is divided into several commits and test should pass after each commit (best reviewed by commit?).
Several important change have been made:

the Line class has been changed from a NamedTuple to a SimpleNamespace to be able to modify the line.content for censoring.
The display patch hunk indication is no longer present in the patch output as it is not really part of the file and we do not display the patch symbols '+/-/ ' . This means some test secrets are considered shorter and the censoring (proportional to the length of the secret) is shorter as well (so some test in output test text have been changed accordingly).
The '+' have been removed of multi line secret in patches as they where not present in single ligne secret nor in the first and last lines of the multiline secret.
Some test in test output text had falty Match index end creating falty output, it has been cleanned up.

Validation
Test pass and output is more coherent now ? I did my best to make it reviewable but it was quite tricky to work with...

PR check list

[ ] As much as possible, the changes include tests (unit and/or functional)
[ ] If the changes affect the end user (new feature, behavior change, bug fix) then the PR has a changelog entry (see doc/dev/getting-started.md). If the changes do not affect the end user, then the skip-changelog label has been added to the PR.

As discussed, I added back the display of patches hunk and unified the display of patch line "category".
The error on unit test on windows seems to have been due outdated cassettes with faulty indices I'm really surprised the other OS work (and it is a bit worrying).

Codecov Report
Attention: Patch coverage is 98.00000% with 2 lines in your changes missing coverage. Please review.

Please upload report for BASE (agateau/fix-extended-match@8b5d34a). Learn more about missing BASE report.

Files
Patch %
Lines

ggshield/core/lines.py
85.71%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                      Coverage Diff                      @@
##             agateau/fix-extended-match     #899   +/-   ##
=============================================================
  Coverage                              ?   91.96%
=============================================================
  Files                                 ?      172
  Lines                                 ?     7094
  Branches                              ?        0
=============================================================
  Hits                                  ?     6524
  Misses                                ?      570
  Partials                              ?        0

Flag
Coverage Δ

unittests
91.96% <98.00%> (?)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Minor remarks for suggestions, but feel free to merge :smile:


Cannot read properties of undefined (reading 'split')

VS Code Version: 1.86.0
OS Version: win10
Logs:

Steps to Reproduce:

paste any code in the chat or any sentence
I get the error

similar to other issue https://github.com/microsoft/vscode-copilot-release/issues/769


Trackpad scroll-to-zoom while holding Shift or Option/Alt

Closes #5508.
I didn't end up piggybacking on the Control key zooming since that has an extra scale multiplier for the pinch-to-zoom.

I didn't end up piggybacking on the Control key zooming since that has an extra scale multiplier for the pinch-to-zoom.

Thanks @quincylvania looks good... 👍


Replace logging for large blobs with blob length and hash.

Fixes #2394

For huge blobs, what is the performance impact when calculating hashes? Just making sure logging is lightweight...

This is VS-side logging, not broker-side, so I wouldn't anticipate a big impact - it only needs to hash these for one user.
MD5 is very lightweight - around 300-400 megabytes/second on your typical laptop or desktop.

👍


Unable to instantiate the IntuneScepServiceClient (Java)

Attempt to instantiate IntuneScepServiceClient fails without error message.
The best I can tell, the cause seems to be the value of intuneResourceUrl, which is set in the IntuneClient -class (line 83): https://api.management.microsoft.com
That domain no longer exists. I guess it has been replaced by the Microsoft's Graph API?
Will there be an update to the csrValidation -library that fixes this issue?

This URL is not used as an endpoint but instead as a resource to be requested.  It was never a valid URL and isn't supposed to be.


Update c-ares to 1.17.2

Risk Level: Low
Testing: All tests
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
Note: needs backporting to stable branches.
Signed-off-by: Yan Avlasov<EMAIL_ADDRESS>

I think this is the issue with CI - https://github.com/c-ares/c-ares/commit/d5146a6a4d9469ab0ee7252abda92db842ea49c2

Windows break is a real failure.
/wait

We will then have to define HAVE_ARPA_NAMESER_H and push the problem into the BUILD file. Maybe it is better. I will leave it  as is for now.

Wondering if the Windows specific if then statement at https://github.com/envoyproxy/envoy/blob/main/test/common/network/dns_impl_test.cc#L34-L39 can now be removed due to this commit which is part of the release - c-ares/c-ares@c2f3235 ?

/retest

Leaving it as-is seems better from a backporting perspective, the cleanup can live exclusively on main.

We will then have to define HAVE_ARPA_NAMESER_H and push the problem into the BUILD file. Maybe having it in the BUILD file better. I will leave it as is for now though.

Wondering if the Windows specific if then statement at https://github.com/envoyproxy/envoy/blob/main/test/common/network/dns_impl_test.cc#L34-L39 can now be removed due to this commit which is part of the release - c-ares/c-ares@c2f3235 ?

/lgtm deps


Add CodeQL security analysis workflow

Added CodeQL security analysis workflow

Signed-off-by: Austin Vazquez<EMAIL_ADDRESS>

I'm not sure about this. I don't think we have been utilizing CodeQL so far in containerd.

I see, will close for now and can always revisit if things change. Also govulncheck has gotten traction since this one, so that might be worth exploring.


Update documentation to avoid issues with Python 3.12

I've updated the installation instructions to avoid the issues with Python 3.12 and make sure pip is installed from the start.
I've also included myself and Pamela as maintainers in the pyproject file.

Changes to the pyproject file were causing issues with pip installation, so I've removed those and this update now just gives a clearer installation process that makes sure the Python version is 3.11 and pip is installed from the beginning.


Rename InMemoryLoginService.cs

While browsing the source code I noticed that the class InMemoryUserLoginService isn't located in InMemoryUserLoginService.cs as I expected, but in InMemoryLoginService.cs.
If this for any reason is by design, please ignore this PR.

Hi @patriksvensson, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

thanks!


Download Links for Deployment Lab Kit not working

The links to download the Deployment Lab Kit just redirect to https://www.microsoft.com/en-us/download with no option to download the lab kit.
Windows 11 with Office 365 deployment lab kit:
https://www.microsoft.com/evalcenter/evaluate-windows-11-office-365-lab-kit
Windows 10 with Office 365 deployment lab kit:
https://www.microsoft.com/evalcenter/evaluate-lab-kit

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

ID: 21195f1c-0f99-502b-c6e7-a87d35237a81
Version Independent ID: 1848a7e1-4919-8e28-27ea-cfdbef125609
Content: Windows and Office 365 deployment lab kit - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/modern-desktop-deployment-and-management-lab.md
Service: o365-solutions
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@msbemba Thank you very much for the contribution and sharing this explanation. @MichaelBishopMN Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Update Reason
The model is neither used in a standard document or triangle document, which indicates that the model is not used anymore.
If there are still reasons to still have this model in the repository, please provide a short rationale in the comments. If not, please indicate with a "+1" that you are OK with deprecating the model.
Deprecating a model means, that it will be removed after two Tractus-X releases from the repository. For details, please refer to the GOVERNANCE process.
MS1 Criteria

[ ] The model that should be deprecated exists and is not deprecated already
[ ] The respective model does not affect existing use cases (aspect is either used in higher versions or replaced by a different aspect)
[ ] A shared aspect, which will be deprecated, is not used by any released aspect model

@sschul33 please check this request

@jonbckr can you please take over?

@bs-jokri the model can be deprecated. +1


header Parameter throws pydantic error

These docs:
https://starlite-api.github.io/starlite/usage/3-parameters/#header-and-cookie-parameters
have the example for using a header parameter as:
@get(path="/users/{user_id:uuid}/")
async def get_user(
    user_id: UUID4,
    token: Parameter(header="X-API-KEY"),
) -> User:
    ...

when I try the (slightly modified) resource (since I don't have User in my app):
@get(path="/users/{user_id:uuid}/")
async def get_user(
    user_id: UUID4,
    token: Parameter(header="X-API-KEY"),
) -> Request:
    ...

I get the error
RuntimeError: error checking inheritance of FieldInfo(default=PydanticUndefined, alias='', extra={'header': 'X-API-KEY', 'cookie': None, 'query': None, 'required': True, 'examples': None, 'external_docs': None, 'content_encoding': None}) (type: FieldInfo)

This is on python v3.9.7 and pydantic 1.9.0 (which I'm not installing explicitly, and I think got installed as a dependency of starlite)

Peculiar. I'll check this out.

Hmm, this is another error in the documentation
This is how it should be:
@get(path="/users/{user_id:uuid}/")
async def get_user(
    user_id: UUID4,
    token: str = Parameter(header="X-API-KEY"),
) -> User:
    ...

Notice the typing difference in token - Parameter is a default function value, not a type. I will update the docs with fixes.

well, apparently there is also a bug involved - but unrelated to this issue. I am publishing 0.7.2 momentarily, which will also update the docs. Please check that this issue is resolved for you - and if so, close this issue.


chore: go mod tidy

run go mod tidy

Current dependencies on/for this PR:

main

PR #21

PR #23

PR #22

PR #30   👈

PR #31

This comment was auto-generated by Graphite.


Add Support for Compact mode (js-Controller 2.0 Feature)

The upcoming version 2.0.0 of js-controller will support a new mode for low memory devices. For this, a few minor adjustments to the adapter must be made. Please follow these instructions: https://forum.iobroker.net/viewtopic.php?f=24&t=20387#p213466

@ldittmar81 Solved. Can be closed.


merge MONDO_0018399 'rare disorder with female infertility due to a congenital hypogonadotropic hypogonadism'

there is a comment on this class that says consider merge but doesn't say with which class.
Term is: MONDO_0018399 'rare disorder with female infertility due to a congenital hypogonadotropic hypogonadism'

[ ] obsolete MONDO_0018399 'rare disorder with female infertility due to a congenital hypogonadotropic hypogonadism' and merge with MONDO_0018398 'female infertility due to a congenital hypogonadotropic hypogonadism'
[ ] add subClassOf axiom  'female infertility due to a congenital hypogonadotropic hypogonadism'and 'has modifier' some 'rare'


Support for more app type

I don't know if you would consider this out of scope, but are you interested in integrating more app types?
like: https://play.google.com/store/apps/details?id=com.ldvsmartwifi.smart
and https://play.google.com/store/apps/details?id=com.ledvance.smart.wifi but i assume there are (or will be)
a lot more around
i recently managed to add a LEDVANCE wifi bulb to HA that use a custom app but same tuya sdk
if you want to go over i can try to help providing needed infos

Sorry, more app types are not supported now.


Canadian french localization

Category
[x] Enhancement
[ ] Bug
[ ] Question
Add Canadian french localization. (Basically just use the same as fr-fr but in a fr-ca file.)

Hi @vincepunkrock!
fr-ca localization has been added and will be included in the next release.
In a meanwhile you can test the functionality in beta version


Timesheet - 14/04/2021- Jesus Avina

Task 1 4/13/2021
Audit Team Meeting - 2
Proof of Work
We have chat history for joining the call on discord but Manish also created a document for notes that has time recorded.
https://docs.google.com/document/d/1zc-JbOKeTSQwVv_Aukn6V8hlADQtkAECBwKVQxjtK3k/edit
Time Spent
1 hour

Task 2 4/14/2021
Working on breaking down the roles and responsibilities of the auditors and project reviewers. This is not yet completed.
Proof of Work
This document has some time recorded but I have done a lot of work through conversation in discord chat as well as research.
https://docs.google.com/document/d/11qWPCVpcJyH1eeRstjLwBsyBS_kA5m1V40boakWleMk/edit
Time Spent
3 hours

Total Time Spent
4 hours
Account Number
f37aaaa15fe0cec8e7cf1809cc884610122c5d8886486b6c2492555229442a46

Approved

Approved


Library create problem for ios 10

Hi,
I am facing issue with library on ios 10, Earlier It works fine but now when I am trying to record video some time it's works some time not. Their is no log printed that's why I am not able to reach out exact issue please help me to debug the issue.

📎 https://github.com/piemonte/PBJVision/issues/327

@sachin19jan please make sure you're handling permissions properly too, by adding the appropriate camera, microphone, and photos keys to the Info.plist

I'm having a similar error:
https://github.com/piemonte/PBJVision/issues/345
Did you figure out what caused this?


Does it support Vision Transformers?

Hi, I would like to know if ViT supports Eetq and LoRA and if I can have an example of this:
`from transformers import ViTForImageClassification, ViTImageProcessor
from peft import get_peft_model, LoraConfig
original_model = ViTForImageClassification.from_pretrained('google/vit-base-patch16-224-in21k')
peft_config = LoraConfig(
r=8,
lora_alpha=32,
lora_dropout=0.01,
target_modules=["query", "value"],
task_type="FEATURE_EXTRACTOR",
)
peft_model = get_peft_model(original_model, peft_config)
#Apply quantization to all layers except lora
`
Thank you :)

@PaulaDelgado-Santos https://github.com/huggingface/peft/blob/cb0bf077744d11524ec6f68d920f4cfe4ef3e8f3/tests/test_gpu_examples.py#L2698 EETQ currently supports AutoModelForCausalLM in transformers and peft. Use lora only for inference. For more general use, you can refer here: https://github.com/NetEase-FuXi/EETQ/blob/1e89c802ecd4b106d1df2afb3ded51f05f8d3c67/examples/models/llama_transformers_example.py#L165
``

This issue was closed because it has been stalled for 30 days with no activity. Please feel free to reopen it if needed.


Update validation-rules.md

@jibi8086 Please clarify the purpose of your change and the PR here. If not, I will have to close it. Also, please sign the CLA  (just write "Mendix").


T6437-svn-changelist-support

Additions:

the Do you want to add these files to the commit? prompt that is issued by Arcanist on every command if repository has uncommited files should respect --cl attribute

Done in https://github.com/aik099/arcanist/tree/T6437-svn-changelist-support


chore(curriculum): add decimal to binary converter workshop to FSD cert

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes https://github.com/freeCodeCamp/CurriculumExpansion/issues/659

you have merge conflicts


Flutter web not working at all

Launching lib\main.dart on Edge in debug mode...
Warning: In index.html:37: Local variable for "serviceWorkerVersion" is deprecated. Use "{{flutter_service_worker_version}}" template token instead.
Warning: In index.html:46: "FlutterLoader.loadEntrypoint" is deprecated. Use "FlutterLoader.load" instead.
../../../AppData/Local/Pub/Cache/hosted/pub.dev/get-5.0.0-release-candidate-5/lib/get_navigation/src/routes/get_transition_mixin.dart:374:39: Error: Member not found: 'CupertinoRouteTransitionMixin.isPopGestureInProgress'.
CupertinoRouteTransitionMixin.isPopGestureInProgress(route);
^^^^^^^^^^^^^^^^^^^^^^
Failed to compile application.
Exited (1).

flutter 3.22

Happens the same to me with auto_route dependency in flutter 3.22:
https://pub.dev/packages/auto_route

https://github.com/jonataslaw/getx/pull/3095

My web build is busted.  First I tried to address the flutter_service_worker_version error, and now when I build, in my index.html I have double quotes around the version value like this:
  <script>
    // The value below is injected by flutter build, do not touch.
    var serviceWorkerVersion = ""2167537469"";
  </script>

So then I manually delete the double quotes, and I still have the following errors:
flutter.js:3 Uncaught (in promise) FlutterLoader.load requires _flutter.buildConfig to be set in flutter.js:3
  <script>
    window.addEventListener('load', function (ev) {
      // Download main.dart.js
      _flutter.loader.load({
        serviceWorker: {
          serviceWorkerVersion: serviceWorkerVersion,
        },
        onEntrypointLoaded: function (engineInitializer) {
          engineInitializer.initializeEngine().then(function (appRunner) {
            appRunner.runApp();
          });
        }
      });
    });
  </script>

So it's like whack-a-mole at this point.
Any ideas what the root cause of this is?  I used to be able to build web app fine, until upgrades.

Try to changed the index.html from this discussion
https://stackoverflow.com/questions/78152396/how-do-i-replace-serviceworkerversion-with-flutter-service-worker-version

Try to changed the index.html from this discussion https://stackoverflow.com/questions/78152396/how-do-i-replace-serviceworkerversion-with-flutter-service-worker-version

I had it like that basically (except had double quotes, tried changing to single...no difference).
A huge rabbit hole since trying to fix the serviceworker deprecation. I eventually reverted back to an old copy of index.html, and it works.  So I will leave for now.
I wonder if I need to rebuild the web files , perhaps my files in that dir are too old (too many flutter versions ago)? And perhaps they changes several things by now.

serviceWorkerVersion

I tried your solution but I get this error ReferenceError: serviceWorkerVersion is not defined which makes sense given that variable is not defined.

Inside  tag Replace:

@nekomaruh I tried your solution but I get this error ReferenceError: serviceWorkerVersion is not defined which makes sense given that variable is not defined.

For me the solution was to delete the contents of /web dir (backing up any mods to index.html) and then remake using:
flutter create --platforms web .
It was much easier than trying to fix all the changes that have occurred since I last made my project.

I'm having a problem after update my flutter version to 3.22, I modify the index.html to new mode of app initialization. My application have some textfields, and after upgrade the number keyboard doesn't work, because the six and four keys moves the cursor to right and left and not show the number on field, how can i resolve this?

not issue resoolve

Update your getX to v6 candidate

running flutter pub upgrade --major-versions helps to resolve.

Me funcionó colocar lo siguiente en el body
`

`

Deleting the web folder (save off any changes) and then taking a Nuke and Pave approach seems to be the way to do it:

Delete the web folder (save any custom stuff)
run "flutter create --platforms web ."
Test, Test, test your app....

Me funcionó colocar lo siguiente en el body
<script src="flutter_bootstrap.js" async></script>
y
 const serviceWorkerVersion = "{{flutter_service_worker_version}}";

is work form me thanks, in doc have same things


Display names of reacting users when few users have reacted

We're close to merging https://github.com/zulip/zulip/pull/21013 as a Zulip 6.0 release goal, which adds a new (default on) setting for displaying the users who did a particular emoji reaction, when there are few such users. Likely the reasons that motivated this feature apply to mobile as well, though perhaps with a different cutoff given different space considerations. We've intentionally defined the API specification and UI label so that clients can implement this setting in a way that makes sense.
In particular, while the web app rule is currently "up to 3 users", it's possible for mobile the correct number will be 1 or 2; I suppose we may want to tweak the API documentation to make clear the client should decide based on what is best.  (Also, we might change web to have the number vary with screen width down the line).

The back end of this is now merged. The web front end is test-deployed on chat.zulip.org but not yet merged.


Clean up color code to deal with 1-d vs 2-d

Some ColorLists have columns, others do not
I've been pretty good about this but the holes are starting to show...

I didn't make any changes, but I've decided this isn't worth the effort.


Improve validator UX by deprecating/renaming validator --importKeystores* flags

Is your feature request related to a problem? Please describe.
There was a discovery made when wanting to import multiple keystores under different passwords that our validator --importKeystores* CLI doesn't actually import keystores and their password using the --importKeystores and --importKeystoresPassword flags. They act more like "override" or "use only" functionality. It will ignore what was already imported via validator import.
Example:
If you run validator import --importKeystores a, then validator --importKeystores b, only keystore b will be used.
Describe the solution you'd like
The nomenclature of --importKeystores* flags are not accurate and should be renamed. Options include using the word override or use instead of import.
Additional context
For now, documentation has been updated via #4985 for clarity, but this issue should be considered when making a big breaking change to Lodestar.

There's no need to do a breaking change here. Just change the main name of the option so that it shows up in help and docs, while keeping the existing option as an alias.
That's what we do now for this options
https://github.com/ChainSafe/lodestar/blob/5b9a78e63650351f7ae2295facefa2b38be48b29/packages/cli/src/cmds/validator/options.ts#L207-L219

Good point. Would definitely advocate for something along the lines of --overrideKeystores and --overrideKeystoresPassword and temporarily leave the current as an alias for backward compatibility.

https://github.com/ChainSafe/lodestar/pull/4985 merged


radicle-httpd: Add patches listing and patch details endpoint

The patches listing and patch details endpoint from radicle-client-services should be migrated at some point to radicle-dev/heartwood.
https://github.com/radicle-dev/radicle-client-services/blob/ba5fa815ed0fb52b66bbda1c761c5dcdde122888/http-api/src/v1/projects.rs#L495
https://github.com/radicle-dev/radicle-client-services/blob/ba5fa815ed0fb52b66bbda1c761c5dcdde122888/http-api/src/v1/projects.rs#L535

This seems to be implemented here https://github.com/radicle-dev/heartwood/blob/master/radicle-httpd/src/api/v1/projects.rs
Will close this issue


[BUG]Setting content type while uploading streams to azure blob storage is not working (Using 12.27.1 version)

Here is my sample code. Content type is not setting however I try.
BlockBlobClient blockBlobClient = blobContainerClient.getBlobClient("image.png").getBlockBlobClient();
blockBlobClient.setHttpHeaders(new BlobHttpHeaders().setContentType("image/png"));
BlobOutputStream blobOutputStream=null;
try {
blobOutputStream = blockBlobClient.getBlobOutputStream();

ByteArrayInputStream bis = new ByteArrayInputStream(inputStream.readAllBytes());
int next = bis.read();
while (next != -1) {
    blobOutputStream.write(next);
    next = bis.read();
}

}finally {
blobOutputStream.close();
}

Hey @vasavi-gopisetty
You are facing this issue because the content type is being set before the blob upload is complete. This may cause the content type to be overwritten or not applied correctly.
To resolve this issue, try setting the content type after the blob upload is complete.
Here's the revised code:
BlockBlobClient blockBlobClient = blobContainerClient.getBlobClient("image.png").getBlockBlobClient();
BlobOutputStream blobOutputStream = null;
try {
blobOutputStream = blockBlobClient.getBlobOutputStream();
ByteArrayInputStream bis = new ByteArrayInputStream(inputStream.readAllBytes());
int next = bis.read();
while (next != -1) {
blobOutputStream.write(next);
next = bis.read();
}
} finally {
blobOutputStream.close();
}
// Set the content type after the upload
blockBlobClient.setHttpHeaders(new BlobHttpHeaders().setContentType("image/png"));

I have reproduced the issue and found that the revised code below works perfectly when the content type is set after the upload completes:
public static void main(String[] args) {
String connectionString = "DefaultEndpointsProtocol=https;AccountName=gunjanfns;AccountKey=Bw+G0p+NR/Vxo3pQvm9d299Q/VIideDu7b3/H9/OJvooiiXyYQ/ym5l9TVRMo2MfYKhWF96KPvsp+AStZd9CfQ==;EndpointSuffix=core.windows.net";
String containerName = "test";
String blobName = "image.png";
String filePath = "C:\Users\gunjansingh\Downloads\image.png";
    BlobContainerClient blobContainerClient = new BlobServiceClientBuilder()
            .connectionString(connectionString)
            .buildClient()
            .getBlobContainerClient(containerName);

    BlockBlobClient blockBlobClient = blobContainerClient.getBlobClient(blobName).getBlockBlobClient();

    BlobOutputStream blobOutputStream = null;
    BufferedInputStream inputStream = null;
    try {
        blobOutputStream = blockBlobClient.getBlobOutputStream();
        inputStream = new BufferedInputStream(new FileInputStream(new File(filePath)));
        ByteArrayInputStream bis = new ByteArrayInputStream(inputStream.readAllBytes());
        int next = bis.read();
        while (next != -1) {
            blobOutputStream.write(next);
            next = bis.read();
        }
    } catch (IOException e) {
        e.printStackTrace();
    } finally {
        try {
            if (blobOutputStream != null) {
                blobOutputStream.close();
            }
            if (inputStream != null) {
                inputStream.close();
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    // Set the content type after the upload
    blockBlobClient.setHttpHeaders(new BlobHttpHeaders().setContentType("image/png"));
}

Hi,
Thank you so much for reaching out.
My issue is resolved.
I was setting conent type using options.
Here is the sample code
BlockBlobSimpleUploadOptions options =
new BlockBlobSimpleUploadOptions(dataStream, dataStream.available())
.setTier(AccessTier.HOT)
.setHeaders(new BlobHttpHeaders().setContentType("image/png");
Thanks,
Vasavi
From: gunjansingh-msft @.>
Sent: Wednesday, December 11, 2024 11:20 PM
To: Azure/azure-sdk-for-java @.>
Cc: Vasavi Gopisetty @.>; Mention @.>
Subject: Re: [Azure/azure-sdk-for-java] [BUG]Setting content type while uploading streams to azure blob storage is not working (Using 12.27.1 version) (Issue #41744)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
I have reproduced the issue and found that the revised code below works perfectly when the content type is set after the upload completes:
public static void main(String[] args) {
String connectionString = "DefaultEndpointsProtocol=https;AccountName=gunjanfns;AccountKey=Bw+G0p+NR/Vxo3pQvm9d299Q/VIideDu7b3/H9/OJvooiiXyYQ/ym5l9TVRMo2MfYKhWF96KPvsp+AStZd9CfQ==;EndpointSuffix=core.windows.net";
String containerName = "test";
String blobName = "image.png";
String filePath = "C:\Users\gunjansingh\Downloads\image.png";
BlobContainerClient blobContainerClient = new BlobServiceClientBuilder()

        .connectionString(connectionString)

        .buildClient()

        .getBlobContainerClient(containerName);

BlockBlobClient blockBlobClient = blobContainerClient.getBlobClient(blobName).getBlockBlobClient();

BlobOutputStream blobOutputStream = null;

BufferedInputStream inputStream = null;

try {

    blobOutputStream = blockBlobClient.getBlobOutputStream();

    inputStream = new BufferedInputStream(new FileInputStream(new File(filePath)));

    ByteArrayInputStream bis = new ByteArrayInputStream(inputStream.readAllBytes());

    int next = bis.read();

    while (next != -1) {

        blobOutputStream.write(next);

        next = bis.read();

    }

} catch (IOException e) {

    e.printStackTrace();

} finally {

    try {

        if (blobOutputStream != null) {

            blobOutputStream.close();

        }

        if (inputStream != null) {

            inputStream.close();

        }

    } catch (IOException e) {

        e.printStackTrace();

    }

}

// Set the content type after the upload

blockBlobClient.setHttpHeaders(new BlobHttpHeaders().setContentType("image/png"));

}
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-java/issues/41744#issuecomment-2535838604, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BLCAIVGKYJ2I3G2HJEK5CP32FAUVRAVCNFSM6AAAAABNXZAFYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZVHAZTQNRQGQ.
You are receiving this because you were mentioned.Message ID: @.***>
This message is intended for the addressee named and may contain confidential information. If you are not the intended recipient, please delete it and notify the sender. Views expressed in the message are those of the individual sender, except where the message states otherwise and the sender is authorised to state them to be the views of Aware Super Pty Ltd ABN 11 118 202 672, AFSL 293340, trustee for Aware Super ABN 53 226 460 365 or Aware Financial Services Australia Limited ABN 86 003 742 756, AFSL 238430. Issued by Aware Super Pty Ltd, with the exception of financial planning services and Aware Investment Funds A and B which are issued by Aware Financial Services Australia Limited, which is wholly owned by Aware Super. Seek professional financial advice, consider your own circumstances and read our product disclosure statement before making a decision about Aware Super or Aware Financial Services Australia Limited. Call us or visit our website aware.com.auhttp://aware.com.au/ for a copy


Error: got an unexpected keyword argument 'version'

After the latest fix yesterday (it also didn't work before the fix), I'm getting this error
2023-12-11 08:41:40.031 | INFO | main:get_kids:70 - Loading kids list
2023-12-11 08:41:40.033 | DEBUG | main:init_notifires:53 - Setting Apprise notification channels
2023-12-11 08:41:40.034 | DEBUG | main:init_notifires:56 - Adding: tgram://
2023-12-11 08:41:40.037 | DEBUG | main::156 - Setting schedule to everyday at 07:45
2023-12-11 08:41:40.038 | DEBUG | main::156 - Setting schedule to everyday at 08:45
2023-12-11 08:45:00.205 | INFO | main:main:127 - Getting tasks for: kid
2023-12-11 08:45:00.205 | INFO | main:init_browser:40 - Initializing browser
2023-12-11 08:45:00.206 | ERROR | main:main:139 - ChromeDriverManager.init() got an unexpected keyword argument 'version'

Please pull the new version (3.1.0) and try.

Working now.
Thanks for your help (and a great job in general :) )


chore: import repository https://github.com/hazem-internship/golang-demo.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Could not run python files -command 'python.execinterminal-icon' not found

Installed new version of vscode for Mac .
Version: 1.57.1
Commit: 507ce72a4466fbb27b715c3722558bb15afa9f48
Date: 2021-06-17T13:28:32.912Z
Electron: 12.0.7
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 15.6.0
Installed  Extensions :
Python for VScode
Python
Pylance
Jupyter
When I click run button I get this Error "command 'python.execinterminal-icon' not found " - No idea what this mean ?  Can you let me know how ti fix it ?

/extPython

Fix: Make sure you have selected the correct version of Python as the interpreter. You will get this error if you have more than one version of Python installed. The current Python interpreter version is mentioned at the bottom left corner in VS Code

[Already Solved] command ‘python.execInTerminal-icon’ not found in VS Code


Fix build issues related to the latest inputmask

inputmask.phone.extensions.js was removed from inputmask in 4.0.2, ember-inputmask 0.8.1 should fix this

Updating today


Add option to keep existing compat entries

Sometimes you might want to keep existing compat entries, e.g. if you tested that two different sets of package versions work well with your project.

This is exactly what I wanted to try. Why is the TOML not just written using TOML library functionality? That seems to make everything a lot more complicated.
    toml_data = TOML.parsefile(toml_file)
     ...
    open(toml_file, "w") do toml_io
        TOML.print(toml_io, toml_data)
    end

As a side note, I would also suggest using the caret ^ explicitly. I don't find it that intuitive that the default Dep = "1.2" is implicitly Dep = "^1.2", especially for novice users (coming from Python?) it looks like it is Dep = "=1.2".

As a side note, I would also suggest using the caret ^ explicitly. I don't find it that intuitive that the default Dep = "1.2" is implicitly Dep = "^1.2", especially for novice users (coming from Python?) it looks like it is Dep = "=1.2".

I do not understand this comment. To which parameter of which function do you refer to?

This comment is about the form of the compat entry. In Julia, if only a version is specified, this is interpreted the same as if it were given with a caret (^). To me, it seems like it would be nicer to people starting with the language (or looking at compats for the first time) to be explicit that a version specifier is not a strict bound, but a semantic versioning compatibility. I believe adding the caret symbol makes this more explicit.

This comment is about the form of the compat entry. In Julia, if only a version is specified, this is interpreted the same as if it were given with a caret (^). To me, it seems like it would be nicer to people starting with the language (or looking at compats for the first time) to be explicit that a version specifier is not a strict bound, but a semantic versioning compatibility. I believe adding the caret symbol makes this more explicit.

If this shall be an actionable item, please create a new issue for it.

Closed by https://github.com/ufechner7/PkgHelpers.jl/pull/9


⚠ DesiTorrents has degraded performance

In 447f6df, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 1309 ms

Resolved: DesiTorrents performance has improved in eabffdb after 19 minutes.


Appsync implementation plans

Hi, I'm wondering about the plans for implementing appsync support. Looking at the IMPLEMENTATION COVERAGE, it seems that only "admin" endpoints are planned. But what about implementing a mock API for queries, mutations and subscriptions? I think this should be doable with plain HTTP for queries/mutations and websockets for subscriptions.
So what's the reason for not including Query/Mutation/Subscription support in that plan?

Any work on Appsync would be great.
It isn't a priority of mine right now, but I'd be happy to accept a PR from anyone that has some time. I'm going to mark this as a feature request.


[SPARK-28017][SPARK-28656][SQL][FOLLOW-UP] Restore comments in date.sql

What changes were proposed in this pull request?
Restored comments in date.sql removed by https://github.com/apache/spark/commit/924d794a6f5abb972fa07bf63adbb4ad544ef246 and https://github.com/apache/spark/commit/997d153e54e3034f469191f69118897cad3e9d46 . The comments was introduced by 51379b731db4523d5e10ad9f2c063e445ae10e72 .
How was this patch tested?
By re-running date.sql via:
$ build/sbt "sql/test-only *SQLQueryTestSuite -- -z date.sql"

@dongjoon-hyun Please, take a look at the PR.

Test build #108983 has started for PR 25422 at commit 72c5800.

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14052/
Test PASSed.

Test build #108983 has finished for PR 25422 at commit 72c5800.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108983/
Test FAILed.

The PySpark failure is irrelevant to this one.

pyspark.ml.tests.test_image.ImageFileFormatTest.test_read_images


Unable to connect to https://rye-up.com/get ?

I'm sorry if this isn't quite right place to report; currently Rye's recommended installation method is not possible as the website is down.

Strange, nothing here has changed recently...

Was about to alert on twitter, but glad I checked github first. @charliermarsh holler if you need any help!

Thanks, I'm looking into it now. Sorry about that.

I was able to download the script when I accessed it directly in Chrome, but in curl I get curl: (7) Failed to connect.
Reference.

In the interim, you can run curl https://raw.githubusercontent.com/astral-sh/rye/main/scripts/install.sh | sh.

The website is completely down.

If you need to access the documentation, you can serve mkdocs locally
$ git clone https://github.com/astral-sh/rye
$ cd rye/
$ rye sync

In the interim, you can run curl -sSf https://raw.githubusercontent.com/astral-sh/rye/main/scripts/install.sh | sh.

I get:
sh: 2: set: Illegal option -o pipefail
This is on Ubuntu 22.04.3 LTS WSL. From google this happens because that option not working on dash shell, /bin/sh links to  /bin/dash.
Workaround:
curl -sSf https://raw.githubusercontent.com/astral-sh/rye/main/scripts/install.sh chmod +x install.sh ./install.sh

Can confirm, website is down

I'm working on getting the domain back up -- there was kind of a perfect storm of events and now it's stuck waiting on some third parties. I can share more details once it's resolved, but ultimately it's my fault.
In the meantime, I'm going to move the docs etc. to a subdomain on astral.sh. We can redirect back to rye-up.com (or vice versa) once it's restored.

Re-opening since rye-up.com is still down. But all the docs, references, etc. now point to https://rye.astral.sh.
For example:
curl -sSf https://rye.astral.sh/get | bash

Hugsops! Thank you @charliermarsh for quickly resolving this!

Thank you for the kind words @edran, I appreciate it!

I'm sorry we screwed something up there when I handed the domain over to Astral. Looks like we're now in for some waiting as neither one of our registrars is entirely sure what's going on :(

Hugops to you too @mitsuhiko--happens to the best of us!
Modulo the unfortunate CI breakages, I do think there's silver lining to be found with the new setup, at least wrt. user confidence :) rye and astral.sh together IMO make each other's brands stronger.

Thanks for all the great work!!

Okay, the domain is finally back up. I have it redirecting to rye.astral.sh, so either domain will work. Apologies everyone and thanks for your patience.


Atomicals ElectrumX database snapshot@824012 decompression failed

Atomicals ElectrumX database snapshot@824012 decompression failed

same
plyvel._plyvel.IOError: b'IO error: utxo/1799751.ldb: Invalid argument'

same @atomicals-electrumx

same

I also encountered this error:
tar: Skipping to next header
tar: Skipping to next header
tar: Skipping to next header
tar: Skipping to next header
tar: Exiting with failure status due to previous errors
See also:
https://stackoverflow.com/questions/46116940/corrupt-tar-archive-after-replacing-a-single-file-skipping-to-next-header
https://copyprogramming.com/howto/corrupt-tar-archive-after-replacing-a-single-file-skipping-to-next-header


Comment color coding breaks when comment follows String and contains quote(s)

And some other oddity.

Heh, that's a bug :)
/cc @onnlucky

Yeah, but it is a known issue, it shows the limits of our current highlighting strategy. If it helps, it will not break if one side uses single quotes, and the other side double quotes, e.g. this does show up correctly: "123" # '123'

If it matters, this came up when I was making instructional materials and needed to distinguish between variables, key words, and strings. Simple example:
word = "cat"    #  'word' is a variable that contains the string "cat".


Square and multiply for fast exponentiation

How about square and multiply technique for fast exponentiation

Can you elaborate a bit on how you'd like to go about this? Which algorithm would you use?

It's probably the one called Al-Khwarizmi Multiplication Algorithm

Hi, it's the implementation of fast modular exponentiation. https://en.wikipedia.org/wiki/Exponentiation_by_squaring. But I believe it has already been implemented.

@apoorva-mk Yep, it's already implemented. Feel free to suggest any other algorithm or data structure to add.


rbac roles

hello,
i cannot create roles and clusterroles , is rbac mode active ?
if not how can i turn it on
thanks
canonical kubernetes on lxd

They are not at the moment. You can enable them with the following script, though some services like the dashboard, may not work as expected:
juju ssh kubernetes-master/0 -- 'sudo snap set kube-apiserver
authorization-mode=RBAC'
juju ssh kubernetes-master/0 -- 'sudo service snap.kube-apiserver.daemon restart'
sleep 120
juju ssh kubernetes-master/0 -- '/snap/bin/kubectl create clusterrolebinding root-cluster-admin-binding --clusterrole=cluster-admin --user=admin && /snap/bin/kubectl create clusterrolebinding kubelet-node-binding --clusterrole=system:node --user=kubelet'

The team will soon have a configuration option to enable/disable RBAC in a Canonical Kubernetes cluster

ok, thank you

Latest charms/bundles contain RBAC support, which you can enable via charm config, like so:
juju config kubernetes-master authorization-mode="RBAC,Node"

More info here: https://github.com/juju-solutions/bundle-canonical-kubernetes/wiki/Authorization-Mode-and-RBAC


Footer additions

What is this PR trying to achieve?
Adding licensing info to the footer. Also adding an RSS icon to Cardigan, but not the app – after discussion with @jamesgorrie it seemed wise to delay linking to RSS until we're off WordPress (because we won't be able to redirect from a user's RSS reader). Closes #851.
What does it look like?

Closes #846


delete vertex

i'm trying some routines to delete vertex but nothing works
comandoo='delete vertex where @class = "repos"'
client.command(comandoo)
client.query("delete vertex where @class = repos;")

there is some option to do this?
thanks!

sorry there is no issue on your sw, there is a problem in my code!


Cloudstrap-teardown doesn't delete aws keypair

Doing this automatically could have surprising results if pointed at a VPC that wasn't cloudstrapped, or that has instances using a non-cloudstrap-generated SSH key.
This can be done with the AWS CLI. Here's a (quick and dirty) way to do it.
aws iam delete-ssh-public-key --user-name $(cloudstrap-cache | awk '$1 == "username" { print $2 }') --ssh-public-key-id $(printf "lkg@%s/%s\n" $(cloudstrap-cache | awk '$1 ~ /^(uuid|username)$/ { print $2 }'))

Drop the $ in the $()s if you're using fish...


0.3.1: failed to load plugin

After update to 0.3.1 I cannot activate the plugin anymore (iOS).

Roll back to version 0.2.21
We would like to inform you that we have identified some issues with the mobile version of our plugin, which may affect its performance and usability https://github.com/nhaouari/obsidian-textgenerator-plugin/pull/144 . In order to ensure the best possible experience for our users, we have decided to roll back to version 0.2.21 until these issues are fixed.
We apologize for any inconvenience this may cause, and we want to assure you that our development team is working diligently to resolve these issues as soon as possible. In the meantime, you can install the latest version of the plugin with the BRAT plugin, which will provide access to the latest features.
Thank you for your understanding and continued support. If you have any questions or concerns, please do not hesitate to contact us in our discord server.
Best regards,

@niksite
Thank you for reaching out! I'm sorry to hear that you're experiencing issues with the plugin after updating to version 0.3.1 on iOS. The good news is that there is a newer version available now, version 0.3.3.
I suggest trying to update to version 0.3.3 and see if that resolves the activation issue you're facing. If you're still having trouble, please let me know and we can explore other possible solutions together.

I tried 0.3.3, and it is still failing.


Update Low Fantasy Gaming/LFGSheet.html

Changes
Revved version and maintained unix line feeds.
Updated initiative roller to correctly omit dexterity modifier.
Added spell tracking for each spell level.
Additional comments
None

Oh, and updated image to point to the one in the roll20 repo.

Thanks for updating the image.


provide required cache invalidation info to broccoli-babel-transpiler

for https://github.com/ember-animation/liquid-fire/issues/485

:+1: - This is needed to prevent warnings with the latest version of ember-cli-htmlbars.


Missing source file should result in error (not warning)

Describe the bug
A missing source file is detected, but currently only a warning is issued resulting in:
C:\w\csolution-examples\Hello>cbuild hello.csolution.yml --context .Debug
C:/w/csolution-examples/Hello/Hello.cproject.yml:17:17 - warning csolution: path './hello.c' was not found
C:/w/csolution-examples/Hello/Hello.cproject.yml:17:17 - warning csolution: path './hello.c' was not found
+----------------------------------------
(1/1) Cleaning context: "Hello.Debug+AVH"
+----------------------------------------
(1/1) Building context: "Hello.Debug+AVH"
FAILED: Hello.Debug+AVH/src/Hello.Debug+AVH-stamp/Hello.Debug+AVH-configure C:/w/csolution-examples/Hello/tmp/Hello.Debug+AVH/src/Hello.Debug+AVH-stamp/Hello.Debug+AVH-configure
C:\WINDOWS\system32\cmd.exe /C "cd /D C:\w\csolution-examples\Hello\tmp\1 && C:\Users\reikei01\.vcpkg\artifacts\2139c4c6\tools.kitware.cmake\3.28.4\bin\cmake.exe -P C:/w/csolution-examples/Hello/tmp/Hello.Debug+AVH/src/Hello.Debug+AVH-stamp/Hello.Debug+AVH-configure-.cmake && C:\Users\reikei01\.vcpkg\artifacts\2139c4c6\tools.kitware.cmake\3.28.4\bin\cmake.exe -E touch C:/w/csolution-examples/Hello/tmp/Hello.Debug+AVH/src/Hello.Debug+AVH-stamp/Hello.Debug+AVH-configure"
CMake Error at C:/w/csolution-examples/Hello/tmp/Hello.Debug+AVH/src/Hello.Debug+AVH-stamp/Hello.Debug+AVH-configure-.cmake:49 (message):
  Command failed: 1

   'C:/Users/reikei01/.vcpkg/artifacts/2139c4c6/tools.kitware.cmake/3.28.4/bin/cmake.exe' '-G' 'Ninja' '-S' 'C:/w/csolution-examples/Hello/tmp/Hello.Debug+AVH' '-B' 'C:/w/csolution-examples/Hello/tmp/1' '-DSOLUTION_ROOT=C:/w/csolution-examples/Hello' '-DCMSIS_PACK_ROOT=C:/Users/reikei01/AppData/Local/Arm/Packs' '-DCMSIS_COMPILER_ROOT=C:/Users/reikei01/.vcpkg/artifacts/2139c4c6/tools.open.cmsis.pack.cmsis.toolbox/2.5.0/etc'

  See also

    C:/w/csolution-examples/Hello/tmp/Hello.Debug+AVH/src/Hello.Debug+AVH-stamp/Hello.Debug+AVH-configure-*.log

ninja: build stopped: subcommand failed.
error cbuild: exit status 1
+------------------------------------------------------------
Build summary: 0 succeeded, 1 failed - Time Elapsed: 00:00:04
+============================================================

To Reproduce
Use csolution-examples Hello and rename Hello.c.
IMPORTANT Files can be also generated by other projects or execute scripts. It is therefore important to check if a file is created during build. Such missing files should be not flagged.
Expected behavior
Output error and stop build process.
C:/w/csolution-examples/Hello/Hello.cproject.yml:17:17 - error csolution: file './hello.c' not found
Environment:

CMSIS-Toolbox 2.5.0
OS: Windows

Please note that generated source files (those that are created as outputs from executes: directives) currently also produce the path "foo.c" was not found warning when they are missing in the source tree and should not produce an error, since otherwise the project will not be buildable without creating those files first.


Add ResourceEvent To EVENT_FETCH_PRE

This pull request adds \ZF\Rest\ResourceEvent to EVENT_FETCH_PRE event.

Coverage increased (+0.03%) to 53.951% when pulling 9df97b72bdc90b195dd058f6ee68eece12509622 on datasage:datasage-fetch-pre into 6d45064514260162e382a56555873a5a748d1611 on zfcampus:master.

@datasage We need test case for that change, thanks!

@datasage HMU if you need help with that.

Finding where to test this was tricky, so I've added a unit test for this and pushed to the branch created by @datasage


Redone laser gun and added laser pistol

Better sprite for the laser gun and added the laser pistol.
This PR does not contain any radical coding so it shouldn't affect Sloth's upcoming gun refactoring.
Some screenshots:

*the old laser gun model

*the new laser gun and the laser pistol
Update: got rid of the pistol.
🆑
-fix: laser gun resprite

Ok, I put more thought into this so here's what I'm going to do:
I'm gonna close this PR myself since I saw two great Cev sprites for a laser pistol and a rifle. I'm gonna have those scaled to 32x32, work on their mags and inhands and have those introduced in a new PR.
Plus, I can't seem to figure on why this still has conflicts.


CRITICAL: Dead link.

Page URL:
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-6.html
Issue:
This link is dead, and I'm not sure what to link to instead.
https://github.com/microsoft/TypeScript-Website/blob/62f87a80f571762ff252eb38b53b890f9f864acb/packages/documentation/copy/en/release-notes/TypeScript 2.6.md?plain=1#L15

Hmm, I just checked the GitHub repository and this still appears to be a broken link.  Is the fix perhaps in a branch or something?

In the template you filled out:

For errors, typos, etc in documentation or other website copy, please send a PR

A link to the web archive would be fine. But this isn't a critical issue as defined by the issue template.


feat(migration-page): create migration page

Describe pull-request
Created a page for the migration where we can log breaking changes.
Solving issue
Fixes: AB#2494
How to test

Go to storybook link below.
Check in Intro -> Migration

Screenshots

Additional context

Personally, I might prefer using markdown over HTML as it is a bit easier to read while you are editing, but this can be the first version. The only thing that might be useful is to have this one  as a comment somewhere so everyone can pick it up.

I agree. I have updated the PR and added the link you posted in the migration.md aswell.


Make Home a non special destination

Completing refactoring the app navigation.
Note: there are still TODOs where we were already having blocking calls in the main path.
Next, probably profile and optimise queries to make sure we don't over fetch.

@martinbonnin Am I misunderstanding cache normalisation? If I call GetSessionsQuery, shouldn't that cache all instances so I'll get a cache results for GetSessionQuery(sessionId)
https://www.apollographql.com/blog/apollo-client/caching/demystifying-cache-normalization/

@yschimke you got it right, it "just" requires some configuration because the GraphQL spec doesn't say anything about caching or "ids" so we have to tell the runtime what fields are ids. I opened https://github.com/joreilly/Confetti/pull/264 for that. If you're curious, there's also some doc there: https://www.apollographql.com/docs/kotlin/caching/declarative-ids/


Error: Could not load UVM kernel module. Is nvidia-modprobe installed?

In trying to use nvidia-docker, I get:
Error: Could not load UVM kernel module. Is nvidia-modprobe installed?

Looking at sudo service nvidia-docker status:
$ sudo service nvidia-docker status
● nvidia-docker.service - NVIDIA Docker plugin
   Loaded: loaded (/lib/systemd/system/nvidia-docker.service; enabled; vendor preset: enabled)
   Active: inactive (dead) (Result: exit-code) since Tue 2017-04-25 21:21:00 UTC; 53min ago
     Docs: https://github.com/NVIDIA/nvidia-docker/wiki
 Main PID: 1596 (code=exited, status=1/FAILURE)

Apr 25 21:20:58 hostname systemd[1]: nvidia-docker.service: Unit entered failed state.
Apr 25 21:20:58 hostname systemd[1]: nvidia-docker.service: Failed with result 'exit-code'.
Apr 25 21:21:00 hostname systemd[1]: nvidia-docker.service: Service hold-off time over, scheduling restart.
Apr 25 21:21:00 hostname systemd[1]: Stopped NVIDIA Docker plugin.
Apr 25 21:21:00 hostname systemd[1]: nvidia-docker.service: Start request repeated too quickly.
Apr 25 21:21:00 hostname systemd[1]: Failed to start NVIDIA Docker plugin.

I think nvidia-modprobe is needed: https://github.com/NVIDIA/nvidia-docker/wiki/Installation: sudo apt install nvidia-modprobe.

I create a PR https://github.com/uchida/packer-nvidia-docker/pull/2

closing because #2 has merged.


Fix typo in language name

The country is Slovenia, the Language is Slovenian - this commit fixes the typo.
Question: Slovenian isn't the main language in any other country, so only sl.js was expected. Spanish has es.js (Spanish, no location given) and not es-ES.js (Spanish-Spain), while other variants would also exist (Spanish Mexican, Costa rican, Colombian, Panama,... about 20 in total).
Can we rename sl-SI to sl? The ICU Locale sl is valid: https://www.localeplanet.com/java/sl/index.html

Coverage remained the same at 100.0% when pulling 9f822db71fbfd3928758e980ff7d35ea759919b7 on titpetric:patch-1 into de0fc75fe6f6572eebf47f94da20716ab7690cff on charliekassel:master.


Update Partner Page

Just got some feedback from a diaper bank -- On the partner edit page the Send Invitation button is confusing when they update a partner. Diaper banks, when updating partners think that since it says send invitation they think if they enter a new partner email in there it will be updated to that.
Let's change the Send Invitation to only show on new partners and when they are updating/editing an existing partner it shows Update Partner

Hi! Can I take in this one?


How to open untitled file and add text content in it

There should be a small example to do this.
I also want to open the untitled file and add text content on it.
I have opened the JSON file by using this code
var setting: vscode.Uri = vscode.Uri.file(en.APP_SETTINGS); vscode.workspace.openTextDocument(setting).then((a: vscode.TextDocument) => { vscode.window.showTextDocument(a, 1, false); });
I get error

cannot open
untitled:c:/Users/shana/AppData/Roaming/Code/User/summary.txt. Detail: expected URI
untitled:c:/Users/shana/AppData/Roaming/Code/User/summary.txt BUT GOT
untitled:c:\Users\shana\AppData\Roaming\Code\User\summary.txt

How to open the untitled file and add text content in it ?
Please give the simplest code as example of provider is difficult to understand at first.

You can try to analyze this code, looks pretty simple.
Also you can look inside some code beautifiers that already exist :-)


Adding random command selection function to Endpoint

An increasing number of our tests rely on the following pattern to execute a randomly selected command on each test execution.
func check() {
    commands := [][]string{
        {"cmd.exe", "/C", "foo.exe bar"},
        {"cmd.exe", "/C", "c:\Windows\System32\bar.exe 123"},
        { "powershell.exe", "-c", "Get-Foobar" },
    }
    index := rand.Intn(len(commands))
    command := commands[randomIndex]
    Endpoint.Shell(command)
}

This is a valuable feature of our tests as it lets us extend a give test to exercise multiple variations of the same operation. To reduce code reuse and standardize our implementation (no more math/rand vs crypto/rand), this new Endpoint function abstracts command selection away from the developer and leverages the exists, tested Shell function to execute the randomly chosen command.

This is PR'd over in our test repo already and approved there: https://github.com/preludeorg/tests/pull/49

merging.


[Feature Request] Is there a way to handle binary response?

I find it possible to pass request body in binary, but is it also possible to handle binary response, e.g. converting them to blob?
Thanks in advance.

Anyway, there must be a way to scramble with plugins. This is my way to get around with the limitation.
const blobPlugin = (): ZodiosPlugin => {
  const exampleBlob = new Blob();

  return {
    name: 'blob',
    request: async (api, config) => {
      const endpoint = api.find((e) => e.method === config.method && e.path === config.url);

      if (!endpoint) return config;

      // We have to guess whether the response schema is `z.instanceof(Blob)` because the info is not reflected
      let isLikelyInstanceOfBlob = false;

      try {
        if (endpoint.response.safeParse(exampleBlob).success) {
          isLikelyInstanceOfBlob = true;
        }
      } catch (e) {}

      if (!isLikelyInstanceOfBlob) return config;

      return {
        ...config,
        responseType: 'blob',
      };
    },
  };
};

Then you simply define your endpoint with response: z.instanceof(Blob) to achieve both blob handling and type-safety

if you need a blob result, you don't need a plugin, just do :
client.download({ responseType: 'blob', });

if you pefer using your plugin, don't attach it globally but only on the route you need it:
client.use('download', pluginBlob);

if you need a blob result, you don't need a plugin, just do :
client.download({ responseType: 'blob', });

if you pefer using your plugin, don't attach it globally but only on the route you need it, so no need to safeParse a fake blob:
client.use('download', blobPlugin());

Thank you for your advice. My concern is this should probably done in the API Definition level instead of the client. Because the binary response is actually determined by the API contract instead of the client or server.
And I wonder where is
client.download({ responseType: 'blob', });

in the documentatation. Am I missing anything?

There is not yet exemples for binary downloads in the docs. it can definitely be added.
Blob download is just described in client options:
http://www.zodios.org/docs/client#request-options
i agree, that we could leverage the api binary option to force a blob or stream.
will think about a way to declare both for zodios v11

Is there an update on this example? passing { responseType: 'blob' } as a second parameter of a method generated from an API is not working for me.

Is there an update on this example? passing { responseType: 'blob' } as a second parameter of a method generated from an API is not working for me.

guess you have to write a middleware yourself


->autoIncrement() not desirable behaviour

In mysql, the database modifier ->autoIncrement() seems to try and set the field to an auto-increment primary key (as stated in documentation).
This is undesirable behaviour.
$table->uuid('id');
$table->primary('id');
$table->integer('incr_id')->unsigned()->index()->autoIncrement(); //should not also try and make this a primary key

This errors out but should be permitted.
SQLSTATE[42000]: Syntax error or access violation: 1068 Multiple primary key defined (SQL: alter table `users` add primary key `users_id_primary`(`id`))

For suggestions and feature requests please use the https://github.com/laravel/internals repo.


Added default implementation with warning for symbolic links

Hi
I changed the default implementation for writing symbolic links to emit a warning instead of throwing an exception for this issue.
Not sure how this runs for performance.
Another thing we could try is to create platform specific commands that call out to a shell command for the various platforms.
Something like:
if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux)) {
   ShellCommands.Exec($"mklink /H {source} {target}")
} else if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX)) {
   ShellCommands.Exec($"ln -s {target} {source}")
} else if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows)) {
   ShellCommands.Exec($"ln -s {target} {source}")
}

After using this source code for launching OS specific shell commands, I did manage to unpack a tar with symbolic links using this code:
https://github.com/RealOrko/dotnet-cli-ghinstaller/blob/232d13a57121cb421b6b10d0f3695dddfa840c02/src/ghinstaller/Modules/Archiving/Archive.cs#L29-L54

Thanks for this.  Should probably put some kind of warning in.