Issues with SSL Cert on Nginx Reverse Proxy

Carrying on from this tweet, I believe I have AutoTLS enabled in the config file, but I control the certificates on the host machine/hypervsior (for some reason, Certbot won't work on guest machines properly).

I'm looking at trying out Offen on a virtual server, using Nginx reverse proxy.  The install steps were fine, however, I am having issue with loading the site (just redirects to the IP) and the SSL certificates ("SSL_ERROR_INTERNAL_ERROR_ALERT").

Hi, thanks for getting back.

I run Certbot on the host machine.  Certbot picks up the site-enabled config file, and it validates against offen.classicniall.co.uk
Offen is ran on a guest machine/virtual server.  Essentially bare metal Ubuntu 20.04 install, using the Systemd install instructions given.
I had set OFFEN_SERVER_AUTOTLS to true, and that's it.

OFFEN_SERVER_AUTOTLS is not a Boolean flag, instead it expects you to pass the domains: https://docs.offen.dev/running-offen/configuring-the-application/#offen_server_autotls
As you acquire the certificates using Certbot I would think you have two options now:

pass the certificate and key files to Offen using OFFEN_SERVER_SSLCERTIFICATE and OFFEN_SERVER_SSLCERTIFICATE
terminate SSL at nginx level using the certs acquired by certbot (you are running nginx right?) and just reverse_proxy to whichever port you are running Offen on

Option 2 is probably the easiest as otherwise you'll need to configure nginx to act as a TCP proxy which can be hairy.

Aye, I think option 2 is what I'm trying to achieve.
On the host machine, I have various virtual machines.  I use Nginx Reverse Proxy on the host machine, and run Certbot on that.
That does generate the certificates (i.e. there is a certificate for Offen and one for my Apache guest machine, as well as many others).
As I mentioned earlier, I removed the OFFEN_SERVER_AUTOTLS from the config, and now it loads 502.

It's hard for me to tell like this and I feel this might not necessarily be related to Offen, but this is what I'd do to debug this:

check if the Offen process is actually running
check if the Offen process logs anything suspicious
check if the Offen process is running on the expected port and hostname (i.e. matches your nginx config)
check if you can reach Offen from within your local network (however that might look like in detail), e.g. if you are bound to the default port of 3000, run curl http://localhost:3000/healthz and see if you get a proper response

`● offen.service - Offen Web Analytics
Loaded: loaded (/lib/systemd/system/offen.service; enabled; vendor preset: enabled)
Active: active (running) since Tue 2022-06-07 11:43:37 UTC; 2min 21s ago
Main PID: 650 (offen)
Tasks: 6 (limit: 1066)
Memory: 23.6M
CGroup: /system.slice/offen.service
└─650 /usr/local/bin/offen
Jun 07 11:43:37 offen systemd[1]: Started Offen Web Analytics.
 Jun 07 09:00:53 offen systemd[1]: offen.service: Succeeded. Jun 07 09:00:53 offen systemd[1]: Stopped Offen Web Analytics. Jun 07 09:00:53 offen systemd[1]: Started Offen Web Analytics.  Jun 07 11:42:54 offen systemd[1]: offen.service: Succeeded. Jun 07 11:42:54 offen systemd[1]: Stopped Offen Web Analytics. -- Reboot -- Jun 07 11:43:37 offen systemd[1]: Started Offen Web Analytics. Jun 07 11:43:37 offen offen[650]: time="2022-06-07T11:43:37Z" level=info msg="Successfully applied database migrations" Jun 07 11:43:38 offen offen[650]: time="2022-06-07T11:43:38Z" level=info msg="Server now listening on port 3000" Jun 07 11:43:38 offen offen[650]: time="2022-06-07T11:43:38Z" level=info msg="Cron successfully pruned expired events" removed=0
I've just left the reverse proxy as default 80/443 (going off the guide), and I can access it just fine via SSH, but not with the IP (URL/subdomain is 502).

Am I to assume that Offen installs all the needed packages using the Systemd method?  Or am I to install packages (LAMP, Nginx, etc.)?

I've just left the reverse proxy as default 80/443 (going off the guide),

I don't understand what that means? Which proxy in which guide are you referring to?
When it comes to configuring Offen, you can do one of two things:

run Offen using AutoTLS, which means it will listen on port 80 and 443
run Offen without AutoTLS which means it will listen on the configured port (3000 as the default)

You want to do the latter, so your reverse_proxy directive in the nginx config should be pointing to <hostname>:3000. I don't know which additional layers are in between nginx and Offen in your case, but in case all of them allow traffic on port 3000, nginx should be able to route traffic to your running service.

I was talking about this guide.
Thanks for the explanation.  Without AutoTLS, and with the certs handled by the Host machine, all I needed to do is add port 3000 to the Reverse Proxy IP, and the site loads :-)
There was no clear instructions as to what to do with reverse proxy from the Docs, but it's been cleared up now :-)


FEATURE: add partial (args/kwargs) field to Action?

Should we add the ability store args/kwargs (like a partial) in an Action?
    action1 = Action(id="cmd_id", title="title1", callback=cb)
    action2 = Action(id="cmd_id2", title="title2", callback=cb, kwargs={"a": 1})

or should the callback just be the partial.  Need to think about pros cons.
first thought: the "pro" of just using callback=partial(cb, a=1) is that it's familiar regular python.  The "con" is that it (currently) would require us to create a new command id for every partial variant.  Maybe that's not a big deal

Yes, we should add this. It will allow writing much cleaner code and avoid leaking memory.

We discussed this recently and @Czaki was in favour of this and NOT partial (was there concern on the use of partials in loop/nested functions?).
@tlambert03 are you happy to take a PR to add kwarg field to Action ?

I suspect the concert is strong refs inside the partial, is that true @Czaki?
If so, it implies that the "proper" solution here would need to consider or reimplement all of the weak referencing stuff in psygnal.
@lucyleeow i would definitely appreciate and consider a PR.  Id say let's start with the simple/direct approach, not worrying about weakrefs, and see how it generally looks and feels

I suspect the concert is strong refs inside the partial, is that true @Czaki?

Strong ref, harder debugging, less control. Promoting of bad patterns (not all callbacks are as good, prepared for partial like psygnal).

If so, it implies that the "proper" solution here would need to consider or reimplement all of the weak referencing stuff in psygnal.

I think that we may start with a simple dict, but limit usage to basic types like str, int, float
And I'm not sure if such advanced things like in psygnal are required. As actions are global, how we expect to hardcode short living objects bind to Action.
Or is there an obvious use case for short living actions?

Or is there an obvious use case for short living actions?

Nope I'm happy to consider these as long lived objects and just use a simple dict

Revisiting things, could someone clarify, if we are limiting kwargs to basic types, how would this fix the strong refs inside partial problem?
i.e., would passing an int via partial(cb, a=1) be better than Action(id="cmd_id2", title="title2", callback=cb, kwargs={"a": 1}) ?
Or is the improvement here in regards to action kwargs being neater and the code thus easier to debug? Also I am not clear what you mean by partials providing "less control" @Czaki?
Also if we were to implement action kwargs, would we use a partial e.g., partial(self.resolved_callback, **kwargs) here:
https://github.com/pyapp-kit/app-model/blob/e8fb729c450ee7a9bcda4c21997912d5f3aeb240/src/app_model/registries/_commands_reg.py#L76
or would pass the kwargs when we execute the callback in Store.inject in in-n-out ?
Thanks
cc @tlambert03

Revisiting things, could someone clarify, if we are limiting kwargs to basic types, how would this fix the strong refs inside partial problem?

Because we do not worry about keeping reference to a string or float object. We worry about keeping reference to big object's live viewer or big numpy array.

i.e., would passing an int via partial(cb, a=1) be better than Action(id="cmd_id2", title="title2", callback=cb, kwargs={"a": 1}) ?

I, personally, prefer kwargs as an action parameter, as it makes debugging easier. There will be no need to deal with inspecting partial.
Also, I expect that people are looking to napari codebase for inspiration. And they may not notice subtle difference and use partial/lambdas as signal callback, that may lead to storing hard references to problematic objects and even segfaults.

or would you pass the kwargs when we execute the callback in Store.inject in in-n-out ?

I think that we should update in-n-out and pass kwargs directly.

To clarify, if we were to expand kwargs to take complex objects (e.g., window/widgets) we would pass them safely: use a weak ref when passing to Action and before passing to the action callback we would unpack the weak ref?
cc @Czaki

To clarify, if we were to expand kwargs to take complex objects (e.g., window/widgets) we would pass them safely: use a weak ref when passing to Action and before passing to the action callback we would unpack the weak ref?

We expect to not pass complex objects. We expect to pass int, str, Enum, float etc.
We expect that actions are created on import time, and in such situation, the complex object should not be created.

Ah okay there is never an intention of passing more complex objects.

... and if there is never an intention of passing more complex arguments, can you remind me again why
Action(id="cmd_id2", title="title2", callback=cb, kwargs={"a": 1})
is so much better than
Action(id="cmd_id2", title="title2", callback=partial(cb, a=1))
?  It kinda seems like added code and complexity here for something that already has an acceptable solution

is so much better than
Action(id="cmd_id2", title="title2", callback=partial(cb, a=1))
? It kinda seems like added code and complexity here for something that already has an acceptable solution

Napari codebase is a place where people look for inspiration of how to write a code. Connecting partial to signal is a bad pattern that may lead to multiple problems (especially when writing tests).
It is also more readable when going through stacktrace when using debugger.

Connecting partial to signal is a bad pattern that may lead to multiple problems (especially when writing tests).

not when all of your arguments are simple.    We are far removed here from a simple connection to a signal.  In fact, this isn't a direct connection to a signal.  it's a connection to an in-n-out function, (whether you connect it to a signal or not is independent of app-model).

Napari codebase is a place where people look for inspiration of how to write a code.

Leaving aside for a moment that this is not the napari repo, that argument is so vague and relatively self-important without much justification that it adds nearly nothing to the motivation for this feature on way or the other.  A) it presumes that Action(id="cmd_id2", title="title2", callback=cb, kwargs={"a": 1}) is prima-facie better code (which, it is not, without knowing much more about the situation B) it assumes that someone looking through the codebase and seeing Action(id="cmd_id2", title="title2", callback=cb, kwargs={"a": 1}) has any clue whatsoever that this is somehow avoiding using a partial because somewhere else in the codebase that action might end up being hooked up to a signal somewhere.
I'm looking for substantive discussion here, with concrete issues and proposals.  Something more than a one or two line comment that just states as fact that your desired outcome is better.

not when all of your arguments are simple. We are far removed here from a simple connection to a signal. In fact, this isn't a direct connection to a signal. it's a connection to an in-n-out function, (whether you connect it to a signal or not is independent of app-model).

You know this, I know this,and for example jni makes PR with lambdas connected to signals in examples (link.

B) it assumes that someone looking through the codebase and seeing Action(id="cmd_id2", title="title2", callback=cb, kwargs={"a": 1}) has any clue whatsoever that this is somehow

No. This is to not inspire people to use partial in their code. People who learn how to code to implement own napari plugin will not see the difference between Action callback and Signal connected callback.
Main difference between:

Action(id="cmd_id2", title="title2", callback=cb, kwargs={"a": 1})
Action(id="cmd_id2", title="title2", callback=partial(cb, a=1))

Is that in the first case the reference is stored in the Action class, that is a better pattern in general.

is it? that kinda entirely depends on how the kwargs feature is added here and in in-n-out. This statement needs much justification

When we use kwargs, there is explicit injection of parameters to function, visible as steep in the debugger. When using partial, the injection happens on creation of Action time, and no one could easily use conditional breakpoint on injection steep without need to check value from partial.
Conditional breakpoint on partial need to base on string representation, that is not optimal. But maybe this is too personal feeling.

You know this, I know this,and for example jni makes PR with lambdas connected to signals in examples (https://github.com/napari/napari/pull/6881.

But that's only an issue because it's wrapping a complex object right?  In a previous post you said "We expect to not pass complex objects. We expect to pass int, str, Enum, float etc." ... and so I asked "in that case, what's so bad about a partial", to which you responded by showing me code where Juan used a partial to bind a very complex object.   Can you see why I'm confused with your position here?

The general problem is that people without experience will not know that there is a difference between wrapping complex and non-complex object. But, as kwargs approach is not easy to apply to signals, it cannot be reproduced, by non-experienced user, as mistake that lead to problematic situation.

perhaps one way we could go about this is for you to make a subclass if Action in napari as an exercise in implementing whatever changes you'd like to see.  I would be happy to make any changes necessary on this side for you to be able to experiment with new features with thin subclasses, without having to vendor large parts of app-model in order to accommodate those new features.  I think that would be mutually beneficial.  It would increase the customizability of app-model, it would let you do whatever you want more easily, and by nature of implementing and testing the changes you want to see to app-model, it would help make more concrete the (currently rather abstract) issues being discussed here.
It's not I don't think there's something potentially interesting here in this issue, but I'm not yet seeing a unambiguous and undeniable benefit or motivation behind it.

I will be happy with such a solution. But As I check, it will also require changes in Command registry? As it will be easier than changing QAction related class?

ok, see #194 which passes control of Action registration to the CommandRegistry.  Note that the command registry has always been extendable: you need to pass your own custom subclass when you create your Application instance

#194 is released in v0.2.7.  So if you'd like to experiment more on the napari side, it should now be easier


Application stops collecting data

Hi,
I'm facing the issue as follows:
I have Influx-capacitor (checked both <IP_ADDRESS> and <IP_ADDRESS> version) installed on one of my hosts and try to collect perfmon counters from couple of remote machines. I use console version just to watch it working, but it behaves the same way with the service version. After some time from the counter start I can see  a sequence of messages like:
Engine A: Dropping X steps.
Engine A: Dropping X steps.
Engine A: Dropping X steps.
Engine A: Dropping X steps.
Engine B: Dropping X steps.
Engine A: Dropping X steps.
Engine A: Dropping X steps.
Engine A: Missing 6 cycles. Resetting and starting over.
...and the application stops logging data into database. The only way to make it functional again is restart (in case of service - kill the process - as stop action on the service is timing out).
I use InfluxDB ver. 0.12.2 and Grafana ver. 3.0.0-beta7 (also tried 3.0.0-beta5)
Best Regards
Tomasz

It looks like you are using the "Exact"-Collector. Try using the "Safe"-Collector instead and see what happens.
The Exact-Collector is unstable and I am actually thinking of discontinuing that one.

You're right - I was using "Exact" mode.
I've switched to "Safe" and as for now - it's working and collecting data longer than ever before.
Thanks.


fix: get all commits from pr

Resolves #653
Just changed to fetch all the commits from pr.
By default it listCommits only fetches 30 commits from pr so couldn't get last commit.

I just took a quick look and if you look at the PR #649 30th commit date is August 19 4AM UTC. But if you look at the follow up comment the last activity date is Aug 18 2023 05:57:07 which is exactly the same time as the last comment on the issue. While this does solve the issue with commits pagination, it looks like the bot is looking only at the comments but not the PR. Can you research further?

I guess this issue is related to drafted pull request. @pavlovcik was right. There were 2 problems related to this issue. 1 is that not all commits are fetched and another one is drafted pull requests are filtered to not be considered according to https://github.com/ubiquity/ubiquibot/blob/e9cfeccac40928132b32bebbce564dd9b93e9091/src/helpers/issue.ts#L636C17-L636C17
I am not sure how to QA my idea...

Could you have a look on this please, @whilefoo ?

Because you changed the function to return draft PRs too, you should be careful about other functions using this function. I think getAvailableOpenedPullRequests should still filter draft PRs so can you fix that?

Thanks @whilefoo
I added a state parameter to the function so we can get "ready for review" or "draft" or all.

Can you post a QA example please.

Can you post a QA example please. Just set the follow up time to something very short and open two pull requests to two issues. One as a draft and one as a ready pull request.

I am not sure how to QA this pr. May we need to wait for few days?

Just set the follow up time to something very short  in your config.

can you post QA?

Yes, I will
I am contacting with Netlify support team because I have some issues with my Netlify account. I will post QA asap.

@byteballet

@byteballet please post QA

@byteballet please post QA

Could you please check the issue I mentioned? I am sick recently so didn't have chance to work on it. :( So please allow me some time if this issue is not  urgent...

Codebase looks good. We need a QA to merge the PR @byteballet.

Codebase looks good. We need a QA to merge the PR @byteballet.

Yes, I know it but I have issues while running the bot. The bot says "the command is disabled in this repo." although I set all the configs to true. I am not so proficient in blockchain and dapp. Is it something related to it?

Codebase looks good. We need a QA to merge the PR @byteballet.

Yes, I know it but I have issues while running the bot. The bot says "the command is disabled in this repo." although I set all the configs to true. I am not so proficient in blockchain and dapp. Is it something related to it?

If you are facing that kinda issue you can just slash the command disable config check feature out so that it will accept the commands even the config is set false.
But before that Plaes make sure if all of them are set correctly for your repo like Readme.md.

If you @byteballet still have an issue with your netlify account, you can also do a QA by following up the readme.


Make separator between dataview components configurable

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.
[X] I am only requesting a single feature. Multiple changes should be split into individual requests, with links between them.
[X] I believe my requested feature will be generally applicable to other users, not just me: it is not uniquely tied to my personal workflow.

🔖 Feature description
You should add a configuration option to automatically add ,  instead of   in front of dataview components.
✔ Solution
The added configuration option should affect the following part of the code.
https://github.com/obsidian-tasks-group/obsidian-tasks/blob/48e82505bc72511d595186d86bdc78c9a812a274/src/TaskSerializer/DataviewTaskSerializer.ts#L129-L134
The default/current behavior could, e.g., be named "Separate by two spaces".
If set to "Separate by comma", above lines should behave like:
return shouldMakeInlineField
    ? `, [${stringComponent.trim()}]`
    : stringComponent;

❓ Alternatives
You could separate by ,  regardless. Though, I did not consider all the situations where adding a leading comma may be undesirable. Given that #1505 is still open (as of 2024-04-26), I suppose that all dataview fields are at the end of a line. If that is the case, I personally would not mind the initial comma.
📝 Additional Context
Such a configuration option would allow the tasks plugin to play more nicely with the "Remove multiple spaces" option of the linter. In particular, when marking a task done from within a tasks view, the linter is not run over that file to remove the leading spaces of the added   [completion:: today]. When the user is later editing that file directly, the linter is removing the duplicate space. This introduces unnecessary changes when synchronizing the vault using Git.
Furthermore, suppose a user manually added ,  in between two dataview components. After editing that task using the "Tasks: Create or edit task" command/GUI, all the ,  will be replaced by  . For example,
- [ ] foo, [priority:: high], [due:: 2024-04-26]

becomes
- [ ] foo,  [priority:: high]  [due:: 2024-04-26]

when immediately confirming the GUI dialog. Note the two spaces after the comma. Upon saving, the linter replaces the task by
- [ ] foo, [priority:: high] [due:: 2024-04-26]

Thanks for the suggestion.
I am concerned about the complications that this would introduce for users who already have space-separated dataview fields in their tasks, and then change the setting.
Please have a read of What do I need to know about the order of items in a task?. A comma between fields would break parsing, currently, as it is not a space.
I can't see a clean (maintainable) way of implementing this suggestion in the current code.
Once https://github.com/obsidian-tasks-group/obsidian-tasks/issues/1505 is implemented, this request would be easier to implement.
But I do feel that the right solution is for somebody to take the time to report the underlying Issue on the Obsidian Forum  - that Obsidian displays reference-style links incorrectly in Live Preview mode.

But I do feel that the right solution is for somebody to take the time to report the underlying Issue on the Obsidian Forum - that Obsidian displays reference-style links incorrectly in Live Preview mode.

I've reflected on this, since writing the above. I am sure that it is the correct solution to the problem. @jonas-schulze, if you want to pair together to work on a Forum bug report, please email me at the address in my GitHub profile.
In view of this size of the backlog for this project, and the demotivating effect of a very long backlog on volunteer energy levels, I'm going through a continual process of reviewing tasks for things that are one or more of the following:

unlikely to be realistically achievable, from the design of the current code, and the planned future refactorings
not sufficiently high priority to a broad community of users, based on the feedback I receive from a variety of sources
do not fit the core purpose of the Tasks plugin
are better covered by a different plugin - to share the development and maintenance effort

So I'm marking this one as 'not planned'. This doesn't mean 'won't fix' - it means not likely to be done in the foreseeable future.


Spinlock try lock failed,  should yield current thread.

see https://probablydance.com/2019/12/30/measuring-mutexes-spinlocks-and-how-bad-the-linux-scheduler-really-is/
terrible_spinlock: like spinlock.h

Type
Average test duration
Four longest waits

std::mutex
62 ms
2.9 ms, 2.8 ms, 1.5 ms, 1.4 ms

terrible_spinlock
825 ms
103.5 ms, 90.6 ms, 77.1 ms, 75.7 ms

spinlock_amd
68 ms
62.3 ms, 61.5 ms, 60.9 ms, 59.8 ms

spinlock
69 ms
11.4 ms, 10.8 ms, 10.4 ms, 9.9 ms

ticket_spinlock
93 ms
1.5 ms, 1.5 ms, 1.49 ms, 1.48 ms

@sniper00 The link you posted may have measure something wrong. Refer to this for what linus have replied to that post. And using pthread_mutex would be a better choice when having confusion.

@sniper00 The link you posted may have measure something wrong. Refer to this for what linus have replied to that post. And using pthread_mutex would be a better choice when having confusion.


[Bug] mmcv now has version 1.7.1, do we need to modify mmdet3d/init.py with mmcv_maximum_version = '1.7.1'?

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (dev) or latest version (1.x).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
master branch https://github.com/open-mmlab/mmdetection3d
Environment
mmcls                   0.25.0
mmcv-full               1.7.1
mmdet                   2.27.0
mmdet3d                 1.0.0rc6
mmsegmentation          0.30.0
pytorch-lightning       1.6.0
torch                   1.9.0
torchmetrics            0.11.0
torchvision             0.10.0a0+300a8a4
Reproduces the problem - code sample
Based on these versions (mmcv-full==1.7.1, mmdet3d==1.0.0rc6), running any script is reproducible
Reproduces the problem - command or script
Based on these versions (mmcv-full==1.7.1, mmdet3d==1.0.0rc6), running any script is reproducible
Reproduces the problem - error message
AssertionError: MMCV==1.7.1 is used but incompatible. Please install mmcv>=1.5.2, <=1.7.0.
Additional information
modify mmdet3d/init.py with mmcv_maximum_version = '1.7.1'?

We should upgrade, we can set maximun version to 1.8.0.

Would you like to create a PR to fix it?

cool！ that's great. i can do it

PR has been submitted, please check and review. @ZwwWayne


networkx 3.3

Destination channel: main
Links

PKG-3167
release-notes: https://github.com/networkx/networkx/releases
release-notes-tagged: https://github.com/networkx/networkx/releases/tag/v3.3
release-diff: https://github.com/networkx/networkx/compare/v3.2.1...v3.3
license: https://github.com/networkx/networkx/blob/main/LICENSE.txt
requirements-tagged:

https://github.com/networkx/networkx/blob/networkx-3.3/pyproject.toml

Explanation of changes:

Skip py<310
Update run_constrained pinnings
Remove networkx.tests from imports  as redundant

pyproject.toml says Homepage: https://networkx.org/ which is where https://networkx.github.io/ redirects to.

@ifitchet I updated it


V0.7 downloads

👍 👍 👍

Windows 7 Home Premium Service Pack 1
Beaker Browser 0.7.11
You should be able to change the location of downloads in the settings or downloads tab.
One minor thing when opening the downloads tab in windows it shouldn't say "Show in Finder" since finder is for Mac. It should probably say "Show in folder" or "Show in explorer" which is better understood by Windows users.

@ideocrat your screenshot is out of date, we have a new release out


Diverging evaluation loss using finetuning scripts Guanaco 7b

Is anyone else having this issue when using the finetune_guanaco_7b.sh script? I keep seeing the evaluation loss diverge rather than converge. I noticed this originally trying to finetune on my own datasets, and after troubleshooting, noticed that this happens for me as well with the original training scripts provided.

The training loss below:

Same issue here, wonder if it is led by the token_id?

wonder if it is led by the token_id?

Glad to hear I'm not the only one. But what do you mean exactly by this comment?

wonder if it is led by the token_id?

Glad to hear I'm not the only one. But what do you mean exactly by this comment?

Basically, I tried 4 ways to get rid of it, but none of them work:

tried different dataset formats;
lr tuning;
change the source_max_len and target_max_len;
A larger model (30b)

Now I suspect the reason for this issue is that:

The length and format of the input, as some of my inputs are long (around 500 tokens), but some of them are just 200 tokens, and I do have lots of \n and - in my dataset, so I wonder if these are the causes.
Maybe the size of the dataset is not large enough? But it is still weird that my train_loss is convergent but eval_loss increased.

wonder if it is led by the token_id?

Glad to hear I'm not the only one. But what do you mean exactly by this comment?

Basically, I tried 4 ways to get rid of it, but none of them work:

tried different dataset formats;
lr tuning;
change the source_max_len and target_max_len;
A larger model (30b)

Now I suspect the reason for this issue is that:

The length and format of the input, as some of my inputs are long (around 500 tokens), but some of them are just 200 tokens, and I do have lots of \n and - in my dataset, so I wonder if these are the causes.
Maybe the size of the dataset is not large enough? But it is still weird that my train_loss is convergent but eval_loss increased.

Interesting. I tried several things too, including some lr tuning, larger models as well as different models like falcon-7b and falcon-40b. Also played a bit with the source_max_len parameter, and used different datasets. Then I decided to train the default script without changing anything about the model or dataset, but then we still see training loss convergence but eval loss divergence.

Same issue, I think maybe the model is too large and dataset is too small, so model is overfitted into training dataset ?

I have the same issue, with flan-t5-xxl, ul2 an xglm. But I ran the same code without 4bit and just LORA, and the model converged normally.  So it is the 4bit part.
And as far as I see the performance on the training set is decreasing as well.

@artidoro

Relieved to see I'm not the only one. 😅 I get practically the same results as you @KJ-Waller when using the Guanaco fine-tuning script for LLaMA 2, and performance on MMLU goes down. Did anybody here figure out what's going on?

Did anybody here figure out what's going on?

I guess that's because the evaluation is done on answer+question. While the model learns how to answer a question in the train set, it will not learn which questions will appear in the eval set.
This is how I always thought of it.

Hello! 4-bit quantization is not responsible for what you are describing here. You are observing diverging loss and oscillating MMLU for the following reasons.

In NLP eval loss is not always directly related to downstream performance (like task accuracy measured by GPT-4 eval).
Dev set of MMLU is small explaining swings in MMLU accuracy while finetuning. These values are indicative and you have to compute test set performance to have a more stable result. We use the last checkpoint for this.
As shown in our paper, finetuning on the OpenAssistant dataset significantly improves chatbot performance, as measured by Vicuna GPT-4 eval. However, it does not help much on MMLU (performance degrades or stays the same compared to no OA finetuning).

Ultimately, we showed that you should be evaluating on your downstream task when finetuning on a dataset. And you should think very carefully about what target benchmark you are optimizing as this is not always indicative of the desired performance. MMLU dev results were used to tune hyperparameters in our paper but Vicuna eval was much more relevant for chatbot evaluation.

Thank you @artidoro! I ran the finetune_llama2_guanaco_7b.sh script to try to orient myself to what I should expect with qlora before running on my own dataset and thought I had screwed something up. Very much appreciate the clarification!
I get the reasoning around not depending on MMLU metrics (is there a reason why you have it on by default in the script?), but I thought eval loss should still give some indication of overfitting/underfitting issues. Is that a misconception on my part?

@KJ-Waller @FHL1998 i get the same loss trend when full finetune on llama 2, have you solved this problem finally?

@KJ-Waller @FHL1998 i get the same loss trend when full finetune on llama 2, have you solved this problem finally?

Sorry, I haven't worked on this in a while and didn't pursue it any further. Good luck


update transformer.py

No Class in tensor named 'Device,' changed to tinygrad.ops

It just looks like a dead import

It just looks like a dead import

👍, will clean dead imports out and rm this PR.
Thanks,
-David


Update Catch to 1.8.2

I also changed the licence link to respect release version.

@horenmar,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@horenmar, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Thanks!


Page view ID doesn't regenerate in rare cases

Affects about 0.01% of page views on our website. The visitor will browse to another page and the page view ID remains the same. The events appear to be from real users and different browsers and platforms.

It's a little stranger than I first thought.
Ten different domain_userid on different browsers (Chrome and Firefox) and platforms, and from different locations. All look like legitimate users.
In each case, the following happens:

a page view and page pings are sent from a page on our website
a single page ping with the same event ID is sent with page_url  = http://snowplowanalytics.com/privacy/ (which all pages on our site link to)

Not an important issue, but I'm leaving it open nonetheless in case other people find similar edge cases.

Pusing back.

@BenFradet - @chuwy can probably help on this one...

@bogaert
@chuwy mentioned that he changed how page view ids are generated in 2.7.0, have you seen the bug recently (if you've switched to 2.7.x)?

@dilyan can you take a look?

I had a look and was not able to find any events that match what @bogaert describes in version 2.7.x. My query returned results matching the odd behaviour for version 2.6.x, but nothing after that.

@dilyand thanks a lot for investigating! :+1:

Thanks @dilyand. How big was the dataset for 2.7.X? It affected about 0.01% of page views before.

The data set for 2.7.x had 541,492 events, not counting robot/spider users.

Thanks!

I assume this was implicitly resolved by https://github.com/snowplow/snowplow-javascript-tracker/issues/436

Thanks everybody!


Feature request: multiple user monitoring

I use MMM-Spotify for monitoring purposes only (not for controlling it).
I'd like to be able to see all monitored accounts of my family account at a glance, e.g. as a minilist (small cover symbols only) or a table (who is playing what on which device).

As written on the MM forum:

I am unsure how that could be solved - since UI is configured / optimized to display exactly one song. That would mean some massive design changes. In addition, it would need some looking into how to support querying multiple accounts at the same time / in different instances.
For me this is out of scope at the current point in time. You can, of course, create a new issue with an enhancement request and will see how things change in the future, or maybe somebody else wants to contribute code to that idea.


ENH: Support to Session Activities

it closes #15 .
The solution was to create a pandas.MultiIndex with (start, time) indexes combining several runpandas.Activity DataFrames. We also developed a accessor to the DataFrame so we can compute several metrics over the combined session and a summary function to compute the condensed statistics over all the session activities.

Codecov Report

Merging #47 (00552da) into master (52de344) will increase coverage by 0.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
+ Coverage   96.89%   97.37%   +0.48%
==========================================
  Files          19       21       +2
  Lines         677      801     +124
  Branches       83       97      +14
==========================================
+ Hits          656      780     +124
  Misses         10       10
  Partials       11       11

Impacted Files
Coverage Δ

runpandas/_utils.py
97.01% <ø> (+0.18%)
:arrow_up:

runpandas/io/gpx/_parser.py
95.00% <ø> (+0.71%)
:arrow_up:

runpandas/reader.py
94.87% <ø> (+2.01%)
:arrow_up:

runpandas/types/__init__.py
100.00% <ø> (ø)

runpandas/types/acessors/__init__.py
100.00% <ø> (ø)

runpandas/types/acessors/session.py
100.00% <ø> (ø)

runpandas/types/frame.py
93.87% <ø> (-0.19%)
:arrow_down:

runpandas/types/summary.py
100.00% <ø> (ø)

runpandas/types/acessors/metrics.py
95.06% <0.00%> (-0.18%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


Announcement: Breaking changes

This issue will contain breaking changes to Neogit and subsequent actions that may need to be taken by users

https://github.com/NeogitOrg/neogit/pull/636 - The default key to bring up the RevertPopup has been changed from _ to v to more closely align with Magit.

The doc/tags has been removed to avoid causing conflicts when updating Neogit if we've updated our docs.
This may cause you to have to run git restore doc/tags inside ~/.local/share/nvim/lazy/neogit or the equivalent plugin location if updating fails.

https://github.com/NeogitOrg/neogit/pull/667 - The config settings for unmerged and unpulled are now split into unpulled_upstream/unpulled_pushRemote, and unmerged_upstream/unmerged_pushRemote

https://github.com/NeogitOrg/neogit/pull/674 - Removes the use_magit_keybindings config. If you want to change the bindings to better align with Magit, it can be done manually.

https://github.com/NeogitOrg/neogit/pull/725 - Branch Popup Keybinding change - the r keybinding has been changed from the branch.<name>.rebase config to checkout recent, with the config getting R instead.

https://github.com/NeogitOrg/neogit/pull/758 - Log Popup Keybinding Change - Trigger changed from L to l (lower case)

https://github.com/NeogitOrg/neogit/pull/1038 - Removes the option disable_commit_confirmation, and changes the behaviour of the BufUnload autocmd when closing the commit editor. Instead of writing the buffer, or asking for confirmation, in the autocmd, we can just ask in the Close action, and expose that to users to configure. Additionally, a Submit and Abort action are exposed.

https://github.com/NeogitOrg/neogit/pull/1060 - Removes DiffAtFile command from status buffer. This is now available via diff popup, dd.
Remapped diff popup from D to d

As of https://github.com/NeogitOrg/neogit/commit/16ee9ae957db2142fb189f5f2556123e24c5b7fb master requires neovim 0.10.


ZIP Download Issue

When downloading this library using Github's ZIP download feature instead of manually cloning, I'm getting a really old version of the library (commit 75c511de).
Is this a Github bug or something is wrong with this repository?

Very good point. This is more bug on Github I would say as I cannot control this.

This has been reported to Github support and they promised to give me feedback.
Please try to use clone feature for now.


Implement LRU Cache

As the title says, we need to implement LRU Cache.

@manrajgrover Hi! I'd like to work on this if that's okay!

@strobelo Go ahead! Ping me if you face any issue!

Hi!, I'm looking for my first contribution. Can I take up this issue?

Tuhin,
I haven't worked on it since my PR last Hacktoberfest. Have at it :)
On Tue, Oct 8, 2019 at 9:29 AM Tuhin Roy<EMAIL_ADDRESS>wrote:

Hi!, I'm looking for my first contribution. Can I take up this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/manrajgrover/algorithms-js/issues/17?email_source=notifications&email_token=ACQETRFBL6RCKUHT5DJAK4DQNSYPFA5CNFSM4DOYJHW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAUZENQ#issuecomment-539595318,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACQETRHH5APWIYOEYD2SNHTQNSYPFANCNFSM4DOYJHWQ
.

--

Derek

Cool, thanks. Will let you know in case of any issues.
On Tue, Oct 8, 2019 at 10:41 PM Derek Strobel<EMAIL_ADDRESS>wrote:

Tuhin,
I haven't worked on it since my PR last Hacktoberfest. Have at it :)
On Tue, Oct 8, 2019 at 9:29 AM Tuhin Roy<EMAIL_ADDRESS>wrote:

Hi!, I'm looking for my first contribution. Can I take up this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/manrajgrover/algorithms-js/issues/17?email_source=notifications&email_token=ACQETRFBL6RCKUHT5DJAK4DQNSYPFA5CNFSM4DOYJHW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAUZENQ#issuecomment-539595318
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ACQETRHH5APWIYOEYD2SNHTQNSYPFANCNFSM4DOYJHWQ
.

--

Derek

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/manrajgrover/algorithms-js/issues/17?email_source=notifications&email_token=AF5VZALHUSNIMK55E62ATIDQNS5NDA5CNFSM4DOYJHW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAU5KTA#issuecomment-539612492,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF5VZALOUGN65IJXSIUYUZ3QNS5NDANCNFSM4DOYJHWQ
.

Hi, I would like to work on it and my programming language will be java. Thanks!

Hey, I am looking for my first contribution this seems like a good first issue can i take it ?

Hi @manrajgrover ,
Can I give it a go?

Hi @manrajgrover , Is it still open ? Can I contribute ?


mrpt_bridge: 1.0.1-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository mrpt_bridge to 1.0.1-1:

upstream repository: https://github.com/mrpt-ros-pkg/mrpt_bridge.git
release repository: https://github.com/mrpt-ros-pkg-release/mrpt_bridge-release.git
distro file: melodic/distribution.yaml
bloom version: 0.11.1
previous version for package: 1.0.0-1

mrpt_bridge
* Add License file
* Formatting fixes
* Update README.md
* Contributors: Jose Luis Blanco Claraco

@jlblancoc What's the status of the mrpt suite of packages on Melodic?  I'd like to do a Melodic sync, but there are regressions on a number of packages: http://repositories.ros.org/status_page/ros_melodic_default.html?q=REGRESSION .
If you are going to work on fixing it in the next few days, I'll hold off.  If not, I'll go ahead and do a sync without them and they'll get reintroduced when they are fixed.

Hi @clalancette
Thanks for the heads up.

mvsim: I was waiting for https://github.com/ros/rosdistro/pull/33622 to see if the current develop branch builds ok for ros2 too, but I'll make a release straight away for ros1 and that one will be fixed.
mrpt_bridge: This package has actually become obsolete. I would probably prefer to remove it from the binary distributions. What's the preferred procedure for it? Manually propose a PR against rosdistro? Should I leave at least the "doc" entries?
the rest mrpt_*: I'll try to release within 1-3 days. If it happens that you have no news, feel free to do the synch w/o them.

Thanks. I'll hold off for a few days, then.

I think I'll be done within ~1 more days. Thanks for the patience!

@clalancette Most of the new packages have been released and PR opened to rosdistro now. I still have to wait until https://github.com/ros/rosdistro/pull/33685 is merged to release one of the pending packages, which now depends on a new package so bloom-release complains about it...
Cheers

@clalancette Most of the new packages have been released and PR opened to rosdistro now. I still have to wait until #33685 is merged to release one of the pending packages, which now depends on a new package so bloom-release complains about it...

Great, thanks!  I've now just merged in #33685 , depending on the size of the package it will take an hour or so to build.

Thanks! Bloom release was already happy for the one pending release just
with the merge, done now.
Note that they're dependencies between them so until all are merged I guess
they won't be green in Travis....
Best
El vie., 24 jun. 2022 14:37, Chris Lalancette @.***>
escribió:

@clalancette https://github.com/clalancette Most of the new packages
have been released and PR opened to rosdistro now. I still have to wait
until #33685 https://github.com/ros/rosdistro/pull/33685 is merged to
release one of the pending packages, which now depends on a new package so
bloom-release complains about it...
Great, thanks! I've now just merged in #33685
https://github.com/ros/rosdistro/pull/33685 , depending on the size of
the package it will take an hour or so to build.
—
Reply to this email directly, view it on GitHub
https://github.com/ros/rosdistro/pull/33016#issuecomment-1165532563, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJ6HWSSQZ56V7NL3BBH2YTVQWTYJANCNFSM5U6EJYBQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

These two additional PRs should,I think, fix the remaining issues:

https://github.com/ros/rosdistro/pull/33705
https://github.com/ros/rosdistro/pull/33704

If they don't, feel free to go on without further waiting for the
failing packages... thanks!

(sigh) sorry... there was a cmake typo in the last release.
This one fixes it: https://github.com/ros/rosdistro/pull/33706

All regressions fixed! Thanks for the heads up and for merging during the
weekend! 👍👍
El mar., 21 jun. 2022 16:41, Chris Lalancette @.***>
escribió:

@jlblancoc https://github.com/jlblancoc What's the status of the mrpt
suite of packages on Melodic? I'd like to do a Melodic sync, but there are
regressions on a number of packages:
http://repositories.ros.org/status_page/ros_melodic_default.html?q=REGRESSION
.
If you are going to work on fixing it in the next few days, I'll hold off.
If not, I'll go ahead and do a sync without them and they'll get
reintroduced when they are fixed.
—
Reply to this email directly, view it on GitHub
https://github.com/ros/rosdistro/pull/33016#issuecomment-1161840997, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJ6HWXYL565P7YSWX63FJDVQHIDBANCNFSM5U6EJYBQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix authentication and faster startup when data is recent

Fixes #235
Speeds up the startup, when there is already recent data available, use momentjs for date formatting

If you remove all store/* files to minimize the diff, that'd be dope. Maybe add those to the .gitignore and commit that.

Auth fix looks good to me, and the data fetching optimization, too. Should close #235.
Note: I have not tested this.
@shobrook ?

@alichtman Sorry, was not on the computer. I verified that it could pick up my recent transactions. But did not verify trading yet

If you could test that out and let me know, that'd be great. I don't have a lot of free time to work on this for a bit.

Great improvements! I reverted your changes to the colorize function, since you were pulling the wrong item from row. I also reverted the change to the transaction window size, just to keep the UI looking organized. Other than that, everything looked solid!


Drake scans down

Source information
Drake scans 1.4.42
Source language
English
Steps to reproduce

Open the extension via latest tab
You will observe nothing loading unable to reach host

Expected behavior
It should load manga previews
Actual behavior
Nothing appearing
Mihon/Tachiyomi version
Version 0.16.5-6768
Android version
Android 14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Source changed domain URLs
Don't close this, but if you go into WebView, and let it load, then back out of WebView, you should be able to load stuff up (until the extension gets updated with new URL)

Still same, nothing working. Yes, they change it to drake comic

Thank you.. aah sorry for that thumbs down.. as it was not working for me. Now, i got it properly explained.

Still same, nothing working. Yes, they change it to drake comic

Don't know why you'd thumbs down when I'm trying to provide a possible workaround. Fresh app, works immediately, try going to Mihon Settings > Advanced, and "Clear Cookies" and "Clear WebView Data". Maybe also Mihon Settings > Data and storage, and "Clear Chapter Cache" as well.

Worked for few minutes then again


Renaming fileClass links

I like that I can have field options in md files that are stored in a folder. The only issue I have is that if I need to rename a file that contains field options, I need to manually update it in all fileClasses. Is there a way to make this automatic?

Hi, unfortunately that is not yet possible because obsidian links aren't managed in frontmatter (they don't update automatically when the note's name has been changed).
Maybe it will be possible in a near future because obsidian team is reported working on improving frontmatter management.


ARROW-12636: [JS] ESM Tree-Shaking produces broken code

This pull request marks arrow as having side effects since we do have side effects in the code.

We might want to add this to the 5.0 release unless we can find a better fix for tree shaking bundlers.


*: replace defunct coreos.com/operators/ with kubernetes.io link

Description of the change: remove old link
Motivation for the change: use upstream docs
/kind documentation
Closes #3801
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[x] Add or update relevant sections of the docs website in website/content/en/docs

/assign @asmacdo


V3

remove Mongoskin dependency
new documentation using gitbook
add opts arg to Collection.count and collection.distinct
deprecate Collection.removeById, Collection.findById, Collection.updateById in favor of using remove, findOne and update directly
deprecate collection.id and manager.id in favor of monk.id
monk('localhost') can be used as a promise which resolves when the connection opens and rejects when it throws an error

TODO

[ ] finish documentation

[Current coverage][cc-pull] is 94.91%

Merging [#142][cc-pull] into [master][cc-base-branch] will decrease coverage by 4.16%

@@             master       #142   diff @@
==========================================
  Files             4          5     +1
  Lines           326        393    +67
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            323        373    +50
- Misses            3         20    +17
  Partials          0          0

Powered by Codecov. Last updated by [e97b614...6be556b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Automattic/monk/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/Automattic/monk/compare/e97b6144444f57f0fd4b918c4de300f7bf6f6232...6be556bb58d560c01be9aabf1ac41c5b9f5419fc
[cc-pull]: https://codecov.io/gh/Automattic/monk/pull/142?src=pr


Build dynamic routes

As commented here this issue is to keep track of the state of dynamic routes build.
IMO this should be supported because is a basic feature of almost any app, and because previous versions of Bankai supported this. The problem is that the server side rendering for the build command is hard (for start, @Flet push some commits that support it in #310). So here is my first proposal/question, can we just disable SSR on dynamic routes, and leave that to choo?
Also my guess is that dynamic SSR requires a smart server, so could be an option that, if we detect dynamic routes, then we add a simple server.js file that would handle SSR on dynamic routes.

So here is my first proposal/question, can we just disable SSR on dynamic routes, and leave that to choo?

Do you mean eg only building a catchall non-SSR index.html if there are dynamic routes? Or maybe build SSR-able routes like now, and for dynamic routes build a separate 200.html or so catchall? I think both of those can make sense

@goto-bus-stop I though on keep building everything for static routes, and a catchall for dynamic, so the 200.html would be fine. Maybe also a 404.html if there is a /* route defined


Laravel 9 Support

Greate package, Can you please update it to Laravel 9. So I could use it in my project.
Big Thanks.

Sure!
Feel free to use the new version 1.5.0 where Laravel 9 support has been added.
v1.5.0


Compiletime assertion and DBG_VAR macro

makes for more expressive code than putting "volatile" everywhere.
You should only use volatile when you need it.

Sorry.
a) your commitlints failed
b) the build check didn't fail because the code is in a header, however if it was included and NDEBUG was defined we would have multiple compiler errors

Ah alright. I didnt check compiling


Prevent store data leaking across requests

The previous store implementation created stores that existed at global scope and were shared across requests. This creates a risk that store data could be be shown to the incorrect user. This PR fixes this issue by updating query, fragment, and mutation stores to hold onto their data in a session-specific way, preventing any data from leaking to the wrong user.
Note: there are no new tests since this is an entirely internal refactor. To verify these changes work, I'm going to increase the number of workers on the CI to try to create many concurrent requests.

I had to bring the worker count down to 5 on github actions but i can consistently run the test suite with 20 workers on my machine.

What's the idea behind Playwright "projects"?

They allow us to run the same test suite with different configurations. I was using them here to give us the option to run the test suite against a headless browser or replay but i think it'd be better to just have 2 different config files so we don't have to keep juggling flags

Hey @AlecAivazis! Happy to answer any questions you might have. Also as a heads up Replay Chromium does not currently have sourcemaps. It will next week though. In the interim, we recommend Replay Firefox, but totally up to you. Also, if you'd like to create a new Houdini Playwright team, we can enable it for Test Suites so you can get the new Test Suite dashboard and PR comments.
We'll be announcing the dashboard in tomorrow's changelog, but here's a sneak peek https://www.notion.so/replayio/Changelog-37-Test-Suite-Dashboard-3ad6be27f04b4cb4982ea28811cc2322

Really appreciate all of this support @jasonLaster! I was genuinely shocked to see that Jaril had forked houdini to get things working ❤


Brexit.cy incorrect/missing lang attributes

The Welsh language version of the Brexit page has large chunks of English language content without appropriate lang attributes which is an accessibility problem. The bits of content which are in English should be identified with lang="en" so that screenreader software reads them out properly.
Additionally, the links in the bottom section (a part of the page which actually is in Welsh) has lang="en" which means it is not read out correctly by AT (for instance on that portion of the page VoiceOver on Mac reads the character â as "a-circumflex" as can be heard in the recording below)
https://user-images.githubusercontent.com/7116819/146046797-9c561035-97cf-4615-9a9f-6c015074f6b3.mov

https://www.w3.org/TR/UNDERSTANDING-WCAG20/meaning-other-lang-id.html

Thanks for this @danacotoran. The page no longer exists, so I'm going to close the issue.


Autocomplete [tab] winget commands in PowerShell

I'd like to be able to use autocomplete for winget commands in PowerShell so I don't have to type them in their entirety.

Proposed technical implementation details (optional)

You can do this yourself with PowerShell's Register-ArgumentCompleter command like so:
$wingetCompleter = {
    param($wordToComplete, $commandAst, $cursorPosition)

    $tokens = $commandAst.Extent.Text.Trim() -split '\s+'
    $completions = switch ($tokens[1]) {
        'install' { "-m","--manifest","--name","--moniker","--version","--soruce","--exact","--interactive","--silent",
                    "--log","--override","--location","-v","-s","-e","-i","-h","-o","-"; break }
        'show'    { "--query","--manifest","--id","--name","--moniker","--version","--source","--exact","--versions",
                    "-q","-m","-v","-s","-e"; break }

        default { "install","show","source","search","hash","validation","--help","--info","-v","--version" }
    }

    $completions | Where {$_ -like "${wordToComplete}*"} | Sort-Object | ForEach-Object {
        [System.Management.Automation.CompletionResult]::new($_, $_, 'ParameterValue', $_)
    }
}

Register-ArgumentCompleter -Native -CommandName winget -ScriptBlock $wingetCompleter

This isn't fully fleshed out but could be done so fairly easily.  That said, rather than folks external to the is project having to provide the completions and then update those completions when new commands are added, it would be better to have winter return completions via a hidden complete command. This is what dotnet does which makes the arg completed very simple:
Register-ArgumentCompleter -Native -CommandName dotnet -ScriptBlock {
    param($wordToComplete, $commandAst, $cursorPosition)

    dotnet complete --position $cursorPosition $commandAst | ForEach-Object {
        [System.Management.Automation.CompletionResult]::new($_, $_, 'ParameterValue', $_)
    }
}

Closing issue as this has been released, see the doc for more details.


Fastnlight: Watch stabilizertech.com

Fastnlight requests the watch of the watch_keyword stabilizertech\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

stabilizertech\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by cocomac in Charcoal HQ


Add New CODEOWNERS

Adds CODEOWNERS for the following:

Container Registry: @pallavit, @alzimmermsft, @jianghaolu
Remote Rendering: @MalcolmTyrrell

@MalcolmTyrrell, feel free to suggest or add additional owners in the PR or in the future.

/check-enforcer override


Update paths to use new inputs area

This PR is a first step to creating a designated area to hold all inputs for the 'make boundary conditions' package (projects/gmao/bcs_shared).
This PR was tested for ICA/C90/O2; NL3/C180/O2; NL3/C720/O2, NL4/M36; NL5/M09; V09/C180/O2 all are zero diff when compared to what we have on develop.
@gmao-rreichle you can take a look and let me know if we need more changes.

@biljanaorescanin:
Inside the make_bcs_inputs dir, there is a link back to the dir, which seems odd and should be removed. Does the PR rely on having this link?
discover31:bcs_shared/make_bcs_inputs pwd
/gpfsm/dnb06/projects/p185/make_bcs_inputs
discover31:bcs_shared/make_bcs_inputs ll make_bcs_inputs
lrwxrwxrwx 1 borescan bcsgen 60 Dec 23 14:15 make_bcs_inputs -> /discover/nobackup/projects/gmao/bcs_shared/make_bcs_inputs//

@gmao-rreichle you can take a look at the PR.
I've tested all land options and different oceans and we have all that develop has.

@biljanaorescanin:

I added a few inline comments, see above.
I added one commit (https://github.com/GEOS-ESM/GEOSgcm_GridComp/pull/694/commits/3e52a535cabcf2ae57510b57423d06354465b9a2) that contains some minor cleanup and tentatively removes create_vegdyn_ndvi.F90, which seems obsolete.  I'm waiting for the CircleCI build test to see if the build works.  If it does, we need to re-run make_bcs to see if that also still works.
I wonder if mkLISTilesPara.F90 is also obsolete.  Please double-check.
Has this PR been tested with the new python scripts (bcs_utils.py)?  I forget if they are sufficiently ready for testing.

successfully re-tested using "make_bcs" by @biljanaorescanin for Icarus-NLv3 (tripolar & cube-shere), NLv4 M36, NLv5 M09, (old land) Icarus w/ ocean O2 (Reynolds)

@biljanaorescanin, @weiyuan-jiang:

I added a few more trivial fixes and cleanups.  For easy reference, the cumulative changes from @biljanaorescanin's most recent commit are here:  https://github.com/GEOS-ESM/GEOSgcm_GridComp/compare/3f1615d..5e9cf75

I have one more question (mostly for @weiyuan-jiang) re. the following two files:  mod_process_hres_data.F90 and rmTinyCatchParaMod.F90.
Rather than adding the declaration
character*400 :: MAKE_BCS_INPUT_DIR
into each and every one of the many subroutines, why not add it as a global variable for the entire module at the top?  I think this should work.
Now if we do that, can we also assign the string value just once at the top?  That is, would it work to just have the following call once (at the top)?
call get_environment_variable ("MAKE_BCS_INPUT_DIR",MAKE_BCS_INPUT_DIR)
It would simplify the code greatly.  Sorry, my Fortran is a bit rusty.

@biljanaorescanin, @weiyuan-jiang:

I added a few more trivial fixes and cleanups.  For easy reference, the cumulative changes from @biljanaorescanin's most recent commit are here:  https://github.com/GEOS-ESM/GEOSgcm_GridComp/compare/3f1615d..5e9cf75
I have one more question (mostly for @weiyuan-jiang) re. the following two files:  mod_process_hres_data.F90 and rmTinyCatchParaMod.F90.
Rather than adding the declaration
character*400 :: MAKE_BCS_INPUT_DIR
into each and every one of the many subroutines, why not add it as a global variable for the entire module at the top?  I think this should work.
Now if we do that, can we also assign the string value just once at the top?  That is, would it work to just have the following call once (at the top)?
call get_environment_variable ("MAKE_BCS_INPUT_DIR",MAKE_BCS_INPUT_DIR)
It would simplify the code greatly.  Sorry, my Fortran is a bit rusty.
I think it makes sense to put the declaration in the module and make it global. Let me modify it.

If Land Team happy with this than so am I


PROBLEMS WITH tutorial SDN_WISE and ONOS

I am trying to run the tutorial of SDN-WISE and ONOS but I have problems.
When I try to build ONOS with BUCK I obtain this message:
curl (52): empty reply from server.
And I don't know how to solve it. Please I would thank if anyone could  help me.

@damuocle Hi, you seem to have the same issue as I mentioned in my last comment. The answer can be found in the following issue:
https://github.com/sdnwiselab/sdn-wise-contiki/issues/17#issuecomment-720515894
I hope this is helpfull.
Regards


Newsletters: add 90 (2020-03-25)

[ ] StackExchange section @bitschmidty
[ ] [Eclair #1339][] @adamjonas
[ ] [BOLTs #751][] @dongcarl

Note, we discussed maybe bringing back the Network Activity section because of last week's moderate backlog; however, some of that was caused by the significant drop in hashrate and thus the slower block production---which will mostly change around the time we go to publish because of a retarget, so I decided not to write the section.

ACK @harding's write-up of LND 4051.

An attempt:
BOLTs #751 updates BOLT7 to allow nodes to announce multiple addresses of a given type. This ensures that multi-homed nodes can better inform the network of their network connectivity, though it seems that the previous restriction of one-address-per-type was not enforced by implementations.

Pushed a commit for the stack exchange section

Pushed several commits:

A few edits to @bitschmidty SE section (good section, thanks!)
@dongcarl's writeup with some small edits (good description, thanks!)
Revised @adamjonas's Eclair description with the improved understanding of the situation from @jnewbery (thanks!)
A correction and editor's note to Newsletter 77 based on John's research (thanks again!).  I assigned the authorship of this commit to John---please let me know if you object to that and I'll change it.

ACK af39d41c377983ee9524aa2a27e006ab76a32986
Thanks all!

That was fast :smile:

Squashed and merged
Thanks to @harding @dongcarl and @adamjonas for contributing this week! 💪 and ty to @jonatack and @jnewbery for the reviews


Bit-reversal in BatchEncoder

I wonder why there must be performed bit-reversal operation before inverse_ntt_negacyclic_harvey in BatchEncoder::encode()? Could you please explain the rationale? Thanks.

NTT outputs in bit-reversed order; so inverse NTT takes inputs in bit-reversed order. Before calling inverse_ntt_negacyclic_harvey, inputs are in normal order; so we have to perform bit-reversal on coefficients' indices first.

NTT outputs in bit-reversed order; so inverse NTT takes inputs in bit-reversed order. Before calling inverse_ntt_negacyclic_harvey, inputs are in normal order; so we have to perform bit-reversal on coefficients' indices first.

Well, so I understand it  makes message inputs and coefficients all in normal order;
But if I remove this part of  bit-reversal operation, I find the seal examples of 1_bfv_basics and 2_encoders run right, and curiously the examples of 5_rotation runs incorrecly, it could not get the correct matrix after rotation; I guess there may be something in performing bit-reversal operation to make the rotation correctly?

The bit-reverse permutation is an involution, hence if you apply it twice (for example before NTT and after iNTT) it cancels itself. When you are not moving coefficients around (that is all operations except automorphisms), you can omit the bit-reverse permutation because the bit-reverse permutation has no impact on these operations (this is why the first examples work fine). However, as soon as you start moving encoded coefficients around using automorphisms, then you need to take into account that the output of the iNTT during the encoding is bit-reversed, hence you need to apply the bit-reverse permutation during the encoding.

The bit-reverse permutation is a linear involution, hence if you apply it twice (for example before NTT and after iNTT) it cancels itself. When you are not moving coefficients around (that is all operations except automorphisms), you can omit the bit-reverse permutation during the encoding because it has no impact on these operations (this is why the first examples work fine). However, as soon as you start moving encoded coefficients around using automorphisms, then you need to take into account that the output of the iNTT during the encoding is bit-reversed, hence you need to apply the bit-reverse permutation during the encoding.

Much appreciated！


User facing documentation and guidance for Bedrock Cross-Region Inference

User Story
As a Bedrock User considering or using cross region inference
I expect to have access to documentation that explains the implications of using it, and relevant guidance
So that I can make an informed decision as to whether it is suitable for my use case and how to use it
Value / Purpose
By providing documentation and guidance on this new feature potential users will have:

information describing what this feature is
an understanding of it's limitations and impacts

By providing documentation and guidance to users of the feature they will have:

a first line of information, before having to contact the AP team

Useful Contacts
AWS own docs on the feature
User Types
Users with data that does not have encumberances on which region it can be processed within.
Hypothesis
If we make sure users of Bedrock Cross-Region Inference have the documentation they need
Then uptake of the new feature will increase and need for support from the AP team will be reduced
And the risk of the feature being used in non-compliant scenarios is reduced
Proposal
Ensure that our docs make clear the tradeoff - latency, availability, and throughput are being traded for not being able to guarantee where in EU-West your requests are going to be processed. For some users that will sound great, but many will be restricted from processing data in this manner and they should be aware of such.
Additional Information
Dependent on the solution delivered in https://github.com/ministryofjustice/analytical-platform/issues/5796
Suggestion is that a different developer completes this story to validate implementation of above
Definition of Done
Example - [ ] Documentation has been written / updated

[ ] README has been updated
[ ] User docs have been updated
[ ] Another team member has reviewed
[ ] Tests are green

https://aws.amazon.com/blogs/machine-learning/getting-started-with-cross-region-inference-in-amazon-bedrock/


blockchain/indexers: Fix reorg related bugs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

155 of 164 (94.51%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.3%) to 55.485%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

blockchain/validate.go
10
11
90.91%

blockchain/common.go
141
149
94.63%

Totals

Change from base Build<PHONE_NUMBER>:
1.3%

Covered Lines:
26437

Relevant Lines:
47647

💛  - Coveralls


The game crashes when starting in version 1.20.5 & 1.20.6

The game crashed whilst bootstrap
Error: java.lang.RuntimeException: Mixin transformation of net.minecraft.class_7714 failed

farbic-loader-0.15.10-1.20.5 using the mods:

(Fabric)Chiseled-Bookshelf-Visualizer-1.20-2.0
fabric-api-0.97.6+1.20.5

It will not be available for version 1.20.6, but it will be ported to 1.21 and 1.21.1.


Introduce torch.compile to all PyTorch examples

As the previous discussion https://github.com/kubeflow/training-operator/pull/2024#discussion_r1521943323, I would suggest to introduce torch.compile in PyTorch examples, following the migration of PyTorch to v2 (#2024).
I think it should be guaranteed to work torch.compile, which is the main feature of PyTorch v2 and able to speed up computation and memory access, on PyTorch operator.
However, as @tenzen-y mentioned at https://github.com/kubeflow/training-operator/pull/2024#discussion_r1521683099, there is a concern about supported GPUs (or I think also CUDA versions).
Please let me know your opinions!

/remove-lifecycle stale

/remove-lifecycle stale

/reopen


Usage with an error will block other usages from being passed to meter

When we're submitting 5 usages at once, and one of the usage has an error, it will not let the other 4 passes. I'm suspecting that this is caused by the recent changes that I made to the collector. I'm currently working on the fix.

The fundamental issue here is that the current API doesn't have a proper way to report individual usage doc processing success or error. I'm going to change the API as follows:

submit a single usage doc per API call
support API call batching like we do for all the other API calls we support in Abacus

I'm actually not convinced that resource providers will get significant performance gains from batching the calls to the service collector API, so I'm not going to add a description of the batching support to the doc right away. We can always add that description later if needed.

Will commit the change to the collector API shortly.


How can I get the userId if the message type is ActivityTypes.ConversationUpdate.

I want to show the user name in the beginning of the welcome message like Hi . I have developed a Bot application in .Net and configured with webchat.
I am passing the userid and username from the MVC application. I got the userid if the message type is ActivityTypes.Message
I could not find the userid if the message type is ActivityTypes.ConversationUpdate.
Once the message type changed to ActivityTypes.Message then the actual userId is available or the user input the value to chat.
How can I get the userId if the message type is ActivityTypes.ConversationUpdate.

Activity.MembersAdded might be what you need:
https://github.com/Microsoft/BotBuilder/blob/master/CSharp/Library/Microsoft.Bot.Connector.Shared/ConnectorAPI/Models/Activity.cs#L128

Thanks for your quick information.
This is not the correct answer.
I am passing the userId and user name to the below query string.
https://webchat.botframework.com/embed/?s=&username=&userid=
if the message type is ActivityTypes.ConversationUpdate, this case I am getting the userid from From.Id or Activity.MembersAdded[0].id but that is not the correct userId and expecting the actual userid from wechat url.
if the message type is ActivityTypes.Message, this case I am getting the correct userid which I have passed from wechat url.
The requirement is once the user logged into the Webchat application  I have to display the user name with welcome message in the Chatwindow . storing the userid and token information to BotState from the web application. If I get the userId from Activity I can pulling the token information from botstate. I have a problem only with ActivityTypes.ConversationUpdate that have some id but not the actual userid.

Same issue here. I even raised an issue few days earlier but didn't get a satisfying answer. https://github.com/Microsoft/BotBuilder/issues/1994

please help me to get the right answer.
I am working on proactive Bot application and send the notification to webchat. The user signed into webpage that page has webchat, How the user will get the notification without input any message to webchat.
Here I am looking the solution to get the ResumptionCookie data from ActivityTypes.ConversationUpdate.
Is there any solution to send the notification message to webchat without user input to webchat?

@danmarshall @eanders-MS do you know?

I don't know, Can I contact this person over Email?

@JeevaJose I am closing this issue for inactivity, please continue the conversation here if needed.  We can reopen the issue if it is still not resolved.

Still active in case of webchat client.

Hello all, remember that ActivityTypes.ConversationUpdate is going to be called twice - once for the bot user and once for the user. See https://github.com/Microsoft/BotBuilder/issues/1930
Also, the userid URL parameter is not officially supported, it will not be passed to DirectLine and it will not be the same id in the activities.

In the case of WebChat it is called only once. And only for Bot side! And it is the issue! Please take a look.

Hello @andrejs-mamontovs can you please let us know your SDK and provide a code sample?

Hi
Now it is Microsoft.Bot.Bulder <IP_ADDRESS> (C#) (valid for older versions as well) on the backend. Just run WebClient from https://dev.botframework.com/bots portal, test section.
Demo code is as simple as possible, just handle ConversationUpdate (MVC controller code) and  prepare response.
In my case, only one greeting was returned (Works fine in the official emulator).

@andrejs-mamontovs can you try this sample: https://github.com/Microsoft/BotBuilder-Samples/tree/master/CSharp/core-GetConversationMembers

I deployed the C# sample from the link above and verified that I get 2 updates, one for the bot, and one for the end user:

So, so far no luck. I tried your code as part of my project with no luck. will try to find the reason later.

Were you able to implement this?

Same issue with me. Are you guys able to solve this ?

Hi @aqanonas
This is a known issue with a workaround described here: https://blog.botframework.com/2018/07/12/how-to-properly-send-a-greeting-message-and-common-issues-from-customers/
There is also a related issue on the WebChat repository:
https://github.com/Microsoft/BotFramework-WebChat/issues/1371


Maven Configuration Plugin - Collections without specifying the argument type and collections with argument types which do not have @Configuration annotation will provide improper configuration

Specifying
private List listenerConfigurations =
            Arrays.asList(new HttpListenerConfiguration(), new HttpsListenerConfiguration());

will throw an exception. However
private List<Object> listenerConfigurations =
            Arrays.asList(new HttpListenerConfiguration(), new HttpsListenerConfiguration());

will work and produce an output as stated below:
listenerConfigurations:
    - !!org.wso2.msf4j.configuration.transport.listener.HttpListenerConfiguration {}
    - !!org.wso2.msf4j.configuration.transport.listener.HttpsListenerConfiguration {}

In case if we specify without specifying the argument type, the maven configuration plugin should ideally throw a warning and ignore the element assuming that the argument type is of Object type.
Furthermore specifying an argument type that do not have the @Configuration annotation also should ideally throw a warning and ignore the element

Closing this issue since obsolete


Unhandled Exception on Application Start - Ubuntu 20.04 LTS

Pulled latest release (ULogViewer-<IP_ADDRESS>9-linux-x64.zip)
Changed ULogViewer to executable (a+rwx)
Attempted to run the application (./ULogViewer)

Following exception occurred
2022-04-10 12:24:59.7276 27965 1    DEBUG App: Created
2022-04-10 12:24:59.7860 27965 1    DEBUG App: Linux distribution: Ubuntu
2022-04-10 12:24:59.9562 27965 1    WARN  App: Try connect to multi-instances server
2022-04-10 12:25:00.4567 27965 1    WARN  App: Unable to connect to multi-instances server
2022-04-10 12:25:00.4707 27965 1    WARN  App: Multi-instances server stream created
2022-04-10 12:25:00.4751 27965 1    WARN  App: Start waiting for multi-instances client
2022-04-10 12:25:00.5467 27965 1    DEBUG App: Start loading persistent state
2022-04-10 12:25:00.5670 27965 1    DEBUG App: Complete loading persistent state
2022-04-10 12:25:00.5683 27965 1    WARN  App: Privacy Policy is not agreed yet
2022-04-10 12:25:00.5683 27965 1    WARN  App: User Agreement is not agreed yet
2022-04-10 12:25:00.5683 27965 1    DEBUG App: Start loading settings
2022-04-10 12:25:00.5715 27965 1    DEBUG App: Complete loading settings
2022-04-10 12:25:00.5715 27965 1    DEBUG App: No need to output log to localhost
2022-04-10 12:25:01.7918 27965 1    INFO  App: Start checking update
2022-04-10 12:25:01.7918 27965 1    DEBUG JsonPackageResolver: Change state from Initializing to Started
2022-04-10 12:25:01.8007 27965 1    ERROR App: ***** Unhandled application exception *****  System.NullReferenceException: Object reference not set to an instance of an object.
at CarinaStudio.AppSuite.Controls.SplashWindowImpl.OnOpened(EventArgs e)
at Avalonia.Controls.Window.ShowCore(Window parent) in //src/Avalonia.Controls/Window.cs:line 702
at Avalonia.Controls.Window.Show() in //src/Avalonia.Controls/Window.cs:line 626
at CarinaStudio.AppSuite.AppSuiteApplication.OnPrepareStartingAsync()
at CarinaStudio.ULogViewer.App.OnPrepareStartingAsync() in C:\Studio\ULogViewer\ULogViewer\App.axaml.cs:line 317
at CarinaStudio.AppSuite.AppSuiteApplication.<>c__DisplayClass165_0.<b__4>d.MoveNext()
--- End of stack trace from previous location ---
at System.Threading.Tasks.Task.<>c.b__128_0(Object )
at Avalonia.Threading.AvaloniaSynchronizationContext.<>c__DisplayClass5_0.b__0() in //src/Avalonia.Base/Threading/AvaloniaSynchronizationContext.cs:line 33
at Avalonia.Threading.JobRunner.Job.Avalonia.Threading.JobRunner.IJob.Run() in //src/Avalonia.Base/Threading/JobRunner.cs:line 166
at Avalonia.Threading.JobRunner.RunJobs(Nullable`1 priority) in //src/Avalonia.Base/Threading/JobRunner.cs:line 37
at Avalonia.X11.X11PlatformThreading.CheckSignaled() in //src/Avalonia.X11/X11PlatformThreading.cs:line 164
at Avalonia.X11.X11PlatformThreading.RunLoop(CancellationToken cancellationToken) in //src/Avalonia.X11/X11PlatformThreading.cs:line 244
at Avalonia.Threading.Dispatcher.MainLoop(CancellationToken cancellationToken) in //src/Avalonia.Base/Threading/Dispatcher.cs:line 65
at Avalonia.Controls.ApplicationLifetimes.ClassicDesktopStyleApplicationLifetime.Start(String[] args) in //src/Avalonia.Controls/ApplicationLifetimes/ClassicDesktopStyleApplicationLifetime.cs:line 120
at Avalonia.ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime[T](T builder, String[] args, ShutdownMode shutdownMode) in //src/Avalonia.Controls/ApplicationLifetimes/ClassicDesktopStyleApplicationLifetime.cs:line 209
at CarinaStudio.ULogViewer.App.Main(String[] args) in C:\Studio\ULogViewer\ULogViewer\App.axaml.cs:line 95

@ZeroAviation Is it 100% reproducible? I have tried the following ways to run ULogViewer on my Ubuntu 20.04.3 LTS but cannot reproduce the error:

By pre-built package
1-1. Download ULogViewer-<IP_ADDRESS>9-linux-x64.zip.
1-2. Make ULogViewer executable (by GUI).
1-3. Run ./ULogViewer.
By building from source code
2-1. Checkout 1.0 branch and pull the latest code.
2-2. Change to ULogViewer sub-folder.
2-3. Run dotnet restore.
2-4. Run dotnet run.

@carina-studio Yes, I am able to reproduce it on this machine without issue. This is a fresh Linux install witth dev tools (Rider, visual studio, docker, etc)
I attempted to build/run from source and got the following
(see screen grab here for better formatting: https://www.dropbox.com/s/06jgam9clcuk72h/Screenshot from 2022-04-11 12-16-54.png?dl=0)
zeroaviation@mattXPS:~/playground$ mkdir ulv zeroaviation@mattXPS:~/playground$ cd ulv zeroaviation@mattXPS:~/playground/ulv$ git clone https://github.com/carina-studio/ULogViewer.git Cloning into 'ULogViewer'... remote: Enumerating objects: 7928, done. remote: Counting objects: 100% (906/906), done. remote: Compressing objects: 100% (240/240), done. remote: Total 7928 (delta 789), reused 749 (delta 665), pack-reused 7022 Receiving objects: 100% (7928/7928), 16.66 MiB | 5.60 MiB/s, done. Resolving deltas: 100% (6307/6307), done. zeroaviation@mattXPS:~/playground/ulv$ cd ULogViewer zeroaviation@mattXPS:~/playground/ulv/ULogViewer$ dotnet restore   Determining projects to restore... /usr/share/dotnet/sdk/6.0.201/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.Sdk.FrameworkReferenceResolution.targets(90,5): error NETSDK1100: Windows is required to build Windows desktop applications. [/home/zeroaviation/playground/ulv/ULogViewer/ULogViewer.Tests/ULogViewer.Tests.csproj] zeroaviation@mattXPS:~/playground/ulv/ULogViewer$ ls BuildMacOSPackages.sh  docs     NotarizeMacOSPackages.sh  PackageManifest.json          README.md   ULogViewer.sln BuildPackages.bat      LICENSE  PackageInfo.json          PackageManifest-Preview.json  ULogViewer  ULogViewer.Tests

Changing to the correct folder, the restore is successful however, running the application "dotnet run" renders the same exception as originally.
Any suggestions on how to debug this issue?

@ZeroAviation The error occurred belongs to class in another project:
https://github.com/carina-studio/AppSuiteBase/blob/1.0/Core/Controls/SplashWindowImpl.axaml.cs
You can clone the source code and run Tests project to check whether the same error happens or not.
I think the NRE is caused by Null value returned by Screens.ScreenFromVisual():

Fixed by https://github.com/carina-studio/AppSuiteBase/commit/01110042b570c647556cdf7d8817a3504cfbcdf7 and released with AppSuite <IP_ADDRESS>7.

Released with <IP_ADDRESS>2.


maintainers: add derjohn

Description of changes
new maintainer: derjohn

This doesn't need to be in a separate PR, since this is already part of #226289, I will go ahead an close this


Use award.project as mechanism for showing '4DN' or 'External' as data provider.

On the encode site they appear to use a combination of award.project and award.rfa (or maybe only one or the other) to display the little images in experiment that show a data attribution eg. Encode : Encode phase 3 icons in the attribution field of https://www.encodeproject.org/experiments/ENCSR209YCR/
We want to base this attribution simply on the project field in the award schema.

Done in the banner; will be implemented in experiment view when that is made.


doc: add jbarz as maintainer

Fixed it to be in alphabetical order. :-)

@jBarz feel free to land this and whatever other PRs of yours are ready to go.


Revert "Add an overload of append(contentsOf:) on Array that takes a Collection instead of a Sequence, and try using it to accelerate wCSIA-compatible Sequences"

Reverts swiftlang/swift#77487
rdar://141617031

@swift-ci please smoke test


Devise doesn't produce localized link in confirmation email

devise-i18n (1.0.1)
Devise doesn't produce localized link in confirmation email.
It has to be:
http://localhost:3000/sk/users/confirmation?confirmation_token=...

and is:
http://localhost:3000/users/confirmation?confirmation_token=...

routes.rb
Rails.application.routes.draw do
  scope "(:locale)", locale: /#{I18n.available_locales.join('|')}/ do
    root to: 'visitors#index'
    devise_for :users
    resources :users
    get "/pages/:id" => "high_voltage/pages#show", :as => :page, :format => false
  end
end

im having the same issue i tried to customize the email views and its not working the emails keep loading from
gems/devise-i18n-1.6.5/app/views/devise/mailer/reset_password_instructions.html.erb

Is the email otherwise localized properly?


Aria labeling functionallity not working

I'm developing an accessible page but tests with NVDA show that Aria tags are not working properly, might I ask what screen reader software was this tested on?

It was tested using the kinda crappy ChromeVox extension (Developing on Ubuntu so no NVDA :disappointed:). Can you be a bit more specific about what isn't working properly?

the virtual cursor is standing on the list and points out the number of options available, but it doesn't read the aria-label tag in the input element, also ariaLabeledBy and ariaDescribedby don't have any effect while using NVDA for the same reason

looks like ariaLablel is being ignored and each option is using 'value' key....


Watch wallet crashes

After adding and viewing the QR code once on the Watch, the Wallet app (I.e. Apple Pay) crashes as soon as I launch it via double-tap on the side button. Removing the QR pass + killing the Wallet app on the Watch resolves the problem.

Hi, on what Apple Watch generation and what version of watchOS does this happen?

All Latest Gens S6 / Mini 12 / 14.6

Ok, that is interesting. I'm on an iPhone 11 with an Apple Watch S6 on the latest stable IOS / watchOS release and I cannot reproduce the problem.
Maybe anyone else can?

Does your vaccination certificate contain any special characters, which may lead to a crash due to wrong encoding? However characters like "ö" like in my name, which aren't ASCII but ISO-8851-1 works absolutely fine.

I just looked a bit through the Apple Communities and this seems to be a known Problem on some Phones / Watches.
Discussion: https://discussions.apple.com/thread/251932786
The only solution seems to be resetting the phone / watch and re-pairing it. However, I don't think that this Problem is related to this repository, but to Apple's Wallet App on the Watch.

Closing this issue as it does not seem to be related to CovidPass.


Export FZF_DEFAULT_OPTS for fish shell

Without exporting the variable, the environment variable is not propagated to fzf

https://github.com/fnune/base16-fzf/pull/19


The map error with gmapping

Hi, when I build map with gmapping and rplidar, there is something wrong. If I move or turn it little by little, it can build the right map as picture 1. However, when I move it fast or turn a large angle in a short time, it doesn't work in a right way. As you can see in picture 2-4, the previous map doesn't move or rotate, so it builds new map incorrectly. I haven't changed the original launch file and don't know why it happens. And I also wanna know, why it says I need to change the static_transform_publisher if I use a real robot in "gmapping.launch" and what I should do. Thanks for your help!

check your tf_tree and  frame transform .    http://www.robotics.sei.ecnu.edu.cn/ROS2017/docs/0728/RPLidar For ROS Based SLAM_slamtec_KintZhao_2017.pdf


Fix the invalid argument issue for LightboxSpatieMediaLibraryImageEntry

Laravel Version: 10.35.0
Filament Version: 3.1.15
When using the LightboxSpatieMediaLibraryImageEntry in Infolist, it throws an error because thelabel method from Filament\Infolists\Components\Component only accepts Illuminate\Contracts\Support\Htmlable, Closure, null, or string.
This pull request simply changes false to null.

Thanks!


Added cudaGetLastError() calls to reset benchmarking kernel errors (issue 88).

Created the NVBENCH_CUDA_CALL_RESET_ERROR() macro and use it to wrap cudaStreamSynchronize() calls.

/ok to test

LGTM, thanks for the PR!


FormBuilderDateTimePicker validator vs validators

This may just be a documentation issue, but...
I have a FormBuilderDateTimePicker and needed to add ONE custom validation, so specified validator except that it didn't work.  When I moved it to a single element array in validators (plural), everything worked as expected.
Is validator deprecated?  Or is it doing something else?  I'm a little confused.

Kindly use validators, that's what we provide for usage in this package since it allows us to chain more than one validator methods.
I may have forgotten to remove the original provided method - I should remove it. Thanks for bringing this to my attention.
Cheers


POC: Click on Template file -> Create Document (Web, Backend, mobile, desktop)

Description
User Stories

As a user I want to create a document by clicking on a template file so that I can use the templates from a shared folder.

Acceptance Criteria

click on a template file eg. letter.ott creates a new writable document letter.odt, new document gets opened
Supported file formats:

Must:

DOTX
OTT

Nice to have:

XLTX
POTX
OTS
OTP

the document gets implicity saved to the users personal space: letter.odt; [Name of template].[corresponding ext]
filename collisions are implicitly solved: letter (1).odt
provide the location of letter (1).odt so that clients can show a notification where the file got stored (in personal)

also see https://github.com/ONLYOFFICE/DocumentServer/issues/2839
Definition of ready

[ ] Everybody needs to understand the value written in the user story
[ ] Acceptance criteria have to be defined
[ ] All dependencies of the user story need to be identified
[ ] Feature should be seen from an end user perspective
[ ] Story has to be estimated
[ ] Story points need to be less than 20

Definition of done

Functional requirements

[ ] Functionality described in the user story works
[ ] Acceptance criteria are fulfilled

Quality

[ ] Code review happened
[ ] CI is green (that includes new and existing automated tests)
[ ] Critical code received unit tests by the developer

Non-functional requirements

[ ] No sonar cloud issues

Configuration changes

[ ] The next branch of the ocis charts is compatible

hm, how do other solutions like collabora and ms solve this?

Needed Changes in Web and oCIS
app/list/

We will add a flag template=true similar to the secureView property

We will add a field targetext=<extension> to indicate which extension should be used by the new file

app/open

We will add the template_id to provide a way to tell the OfficeApp which template to process. The template_id will only be used if the file is empty.

Background: It is important that the template is processed by the Web App to execute the field functions and other stuff.
Web Client

Web Client reads the template and targetext fields from the app/list response.
Files with that flag will trigger a new default action, which is "open a new file from this template" (new web extension inside of external-app)
The existing file action "Open this File in ..."  will stay and can be used via right click.

New default action "open a new file from this template"

It asks the user about the new file name in the personal space root (collision handling included, no personal space means "feature not available")
It will create a new file via app/new using the suggested file extension
It uses the new file to switch the current context
It will open the new file via app/open and send the template_id

@kulmann backend has been merged.

API tests: https://github.com/owncloud/ocis/issues/10367
WebUI tests: https://github.com/owncloud/web/issues/11811


Add color swatch to color options

When you add options to a input of type color, it shows the hex values but no preview of the color.
It would be nice to show a swatch of the color alongside the color number.
With the current setup (system's select / option ) it would only be possible to do for the selected option (not while selecting). To overcome this we could move away from the system select and instead use a react-based one like react-select to be able to show a color swatch alongside every color option and do fancier styles for the option list.

issue 💯


added Base2Tone colorschemes

Base2Tone colorschemes are based on DuoTone themes by Simurai for Atom.
I converted these themes from the ones I made for iTerm2.

base2tone-evening-dark should come close to the default duotone-dark theme by Simurai for Atom
base2tone-morning-light should come close to the default duotone-light theme by Simurai for Atom

like:

base2tone-sea-dark should come close to Sea duotone theme by Simurai for Atom
base2tone-space-dark should come close to Space duotone theme by Simurai for Atom
base2tone-earth-dark should come close to Earth duotone theme by Simurai for Atom
base2tone-forest-dark should come close to Forest duotone theme by Simurai for Atom

The others are colorschemes I created myself based on the duotone concept.

Thanks for your contribution :beer:


fix: mention enabling ESM support

Adds a short paragraph with example package.json about the ESM modules.
https://docs.apify.com/academy/web-scraping-for-beginners/crawling/pro-scraping#crawlee-installation

Closes #1063

Thanks! This is also going to be useful once I'm updating the JS beginner course.


Simplify Kotlin Examples with main.kts scripts

Thought experiment about using kotlin scripting, which works with nice completion in basically any Intellij or Android Studio project.  So these samples are trivial to cut and paste and have running without a surrounding project including okhttp.
Also run straight from the command line with chmod u+x script.main.kts
https://github.com/Kotlin/kotlin-script-examples/blob/master/jvm/main-kts/MainKts.md

@swankjesse thoughts on this? Runs as a bash shell (if you have kotlin installed), runs in Intellij, completely self contained (doesn't use libraries from containing project).

Went with a stack overflow Q&A https://stackoverflow.com/questions/66364993/how-to-make-a-reproduction-of-a-bug-for-okhttp/66364994#66364994


[Draft] Workers overview

Summary
Add Workers overview article, and sample code.
Fixes #24308
Internal preview
✔ Worker Services in .NET
✔ Create a Queue Service in .NET
✔ Use scoped services within a BackgroundService in .NET
✔ Create a Windows Service using BackgroundService in .NET
✔ Implement the IHostedService interface in .NET

Links verifier failure is being fixed in https://github.com/dotnet/docs/pull/24372. Sorry for that!

Links verifier failure is being fixed in #24372. Sorry for that!

No worries, this is currently a draft. :)

this is exciting. can't wait to see these out there supporting our cloud native back-end service developers.


Looking up user by address or name returns error

Describe the bug
The lookup functions in UserRegistry always return error, even though it's obviously not.
await userRegistry.lookupByUsername("dwr");

Even that errors out. Contracts are currently being upgraded to v2. Could be the cause?
To Reproduce
Steps to reproduce the behavior:

Install a fresh Next.js app
Copy-paste code in "Look up a user" in the README

Expected behavior
User that exist in the network should not return error, likewise with custody address that has fid.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Fedora 34
Node version: 16.16
Code Version: 1.1.0

Additional context
Add any other context about the problem here.

My bad. I use InfuraProvider without importing it. All good when I import the right provider.


Support Field Presence (proto3)

Field presence is a new experimental feature that adds support for optional fields.
TLDR: it works in a somewhat hacky way by representing an optional field with a oneOf with only one variant.
https://github.com/protocolbuffers/protobuf/blob/master/docs/implementing_proto3_presence.md#satisfying-the-experimental-check

Implemented in versionv3.0.0-alpha.9.

Awesome, thanks!


refactor: remove gateway

Description

References

Review Checklist

[ ] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

What is the plan to replace this? This will drop internal visibility if there isn’t an alternative.

I'll close this for now


Sync product owners with security-as-code

This writes to a yaml file in eng-pipes containing information relevant for automations regarding product owners.
Must go in before https://github.com/getsentry/security-as-code/pull/311
This is similar to the setup for sentry and sentry-docs. See here for more details:
https://github.com/getsentry/sentry/pull/48600

tested locally and it looks good :)
product_areas:
  Alerts:
  - team-issues
  Crons:
  - team-crons
  Dashboards:
  - team-discover-and-dashboards
...
repos:
  arroyo: team-sns
  cdc: team-sns
  craft: team-ospo
  relay: team-ingest
  responses: team-ospo
  self-hosted: team-ospo
  sentry-android-gradle-plugin: team-mobile-sdks
  sentry-cocoa: team-mobile-sdks
  sentry-dart: team-mobile-sdks
  sentry-dart-plugin: team-mobile-sdks
...
teams:
  team-crons: null
  team-discover-and-dashboards: null
  team-docs: null
  team-enterprise: null
  team-ingest: null
  team-issues: null
  team-mobile-sdks: null
  team-ospo: C04B27EECUB