Storing the definition's model file name in RefRulePosition

Details
Feature implementation for Issue #277

Code review checklist

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Commit messages are meaningful (see this for details)
[x] Tests have been included and/or updated
[x] Docstrings have been included and/or updated, as appropriate
[x] Standalone docs have been updated accordingly
[x] Changelog(s) has/have been updated, as needed (see CHANGELOG.md, no need
to update for typo fixes and such).

Thank you for your contribution.
I assume you need this PR for model editing purposes?

Thank you for your contribution.
I assume you need this PR for model editing purposes?

To me the PR looks good, so far. @igordejanovic any comments?
@hlouzada : I am looking forward to see what you put together... Thank you for your interest and your work+time!

To me the PR looks good, so far. @igordejanovic any comments?
@hlouzada : I am looking forward to see what you put together... Thank you for your interest and your work+time!

Technically, your changes look OK, after a first quick check... Do you think you can sketch the use case related to your change. It would ease the review.

A use case would be using an IDE's go-to definition that uses the model's attribute _pos_crossref_list, enabled by textx-tools, on a language that its grammar allows cross models imports. Thus, the goto-definition needs to know the position of the definition and which model it's from.
Take this grammar as an example:
Model:
        imports*=Import
        packages*=Package
;
Package:
        'package' name=ID '{'
            objects*=Object
        '}'
;
Object:
    'object' name=ID ('ref' ref=[Object|FQN])?
;
FQN: ID+['.'];
FQNI: ID+['.']('.*')?;
Import: 'import' importURI=FQNI;

Now a model named ./folder/A.model:
package packageA1 {
    object A
}
package packageA2 {
    object A ref packageA1.A
    object B ref A
}

And another model named ./B.model:
import folder.A

package packageB {
    object A1 ref packageA1.A
    object A2 ref packageA2.A
}

I would like to know where is the packageA1.A and packageA2.A definition.

Technically, your changes look OK, after a first quick check... Do you think you can sketch the use case related to your change. It would ease the review.

A use case would be using an IDE's go-to definition that uses the model's attribute _pos_crossref_list, enabled by textx-tools, on a language that its grammar allows cross models imports. Thus, the goto-definition needs to know the position of the definition and which model it's from.
Take this grammar as an example:
Model:
        imports*=Import
        packages*=Package
;
Package:
        'package' name=ID '{'
            objects*=Object
        '}'
;
Object:
    'object' name=ID ('ref' ref=[Object|FQN])?
;
FQN: ID+['.'];
FQNI: ID+['.']('.*')?;
Import: 'import' importURI=FQNI;

Now a model named ./folder/A.model:
package packageA1 {
    object A
}
package packageA2 {
    object A ref packageA1.A
    object B ref A
}

And another model named ./B.model:
import folder.A

package packageB {
    object A1 ref packageA1.A
    object A2 ref packageA2.A
}

I would like to know where is the packageA1.A and packageA2.A definition.

@hlouzada Looks good! Thanks for the contribution.

@hlouzada Looks good! Thanks for the contribution.


Fix makefile to add a npm build to build commant

Add a npm build on "make build" command to prevent an exception on local. Is it a good idea ?

Welcome @hmeneuvrier :-)


Enhance the coverage report to show the missing part, branches, etc.

use --cov-report term-missing


Update netsurf.browser and netsurf.lib*, fix on darwin

Description of changes
Updates netsurf.browser from 3.10 to 3.11, and similar updates for all corresponding libraries(see commit messages).

changelog for NetSurf/browser and various libs
changelog for libcss

These include a few minor but potentially breaking changes, like deprecating TLS 1.0 / 1.1.
Adds compile flags in browser.nix to allow building on darwin and with clang in general.
The build system is already updated in #277664.
Supersedes #277662 and #277669.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Remaining / TODO:

[x] Test with uilib=framebuffer

Seems to work fine on NixOS, build fails on darwin

[x] Test with uilib=gtk2

Builds fine, shows websites fine, but seems to be missing some UI icons (e.g. back button)
[ ] Fix missing resources

[x] Test with uilib=sixel

Works fine on NixOS, but SDL_sixel is broken on darwin

[ ] Enable tests in testPhase
[ ] Enable/fix "Export to PDF" with libharu
[ ] Look into broken gimpPlugins, imv and toppler

Result of nixpkgs-review run on aarch64-darwin 1

2 packages marked as broken and skipped:

gimpPlugins.exposureBlend
gimpPlugins.resynthesizer

2 packages failed to build:

gimpPlugins.bimp
gimpPlugins.gap

21 packages built:

gegl
gegl.dev
gegl.devdoc
gimp
gimp.dev
gimpPlugins.farbfeld
gimpPlugins.fourier
gimpPlugins.gmic
gimpPlugins.lightning
gimpPlugins.lqrPlugin
gimpPlugins.texturize
gimpPlugins.waveletSharpen
netsurf.browser
netsurf.libcss
netsurf.libdom
netsurf.libhubbub
netsurf.libnsbmp
netsurf.libnsgif
netsurf.libnsutils
netsurf.libparserutils
netsurf.libsvgtiny

Result of nixpkgs-review run on x86_64-linux 1

2 packages marked as broken and skipped:

gimpPlugins.exposureBlend
gimpPlugins.resynthesizer

3 packages failed to build:

imv
imv.man
toppler

27 packages built:

gegl
gegl.dev
gegl.devdoc
gimp
gimp-with-plugins
gimp.dev
gimpPlugins.bimp
gimpPlugins.farbfeld
gimpPlugins.fourier
gimpPlugins.gap
gimpPlugins.gimplensfun
gimpPlugins.gmic
gimpPlugins.lightning
gimpPlugins.lqrPlugin
gimpPlugins.texturize
gimpPlugins.waveletSharpen
gnome-photos
gnome-photos.installedTests
netsurf.browser
netsurf.libcss
netsurf.libdom
netsurf.libhubbub
netsurf.libnsbmp
netsurf.libnsgif
netsurf.libnsutils
netsurf.libparserutils
netsurf.libsvgtiny

Add a :+1: reaction to pull requests you find important.

I have not undrafted it yet as it still breaks imv and toppler, should that be ignored? I haven't quite figured out why they break yet. Also, the results from @wegank are a bit surprising as it builds fine on my aarch64-darwin machine.

According to ofborg logs, netsurf.libparserutils might need libiconv as a dependency on x86_64-darwin.

Result of nixpkgs-review pr 285118 run on x86_64-darwin 1

2 packages marked as broken and skipped:

gimp-with-plugins
toppler

15 packages built:

gegl
gegl.dev
gegl.devdoc
gimp (gimpPlugins.gimp)
gimp.dev (gimpPlugins.gimp.dev)
gimpPlugins.gmic
netsurf-browser (netsurf.browser)
netsurf.libcss
netsurf.libdom
netsurf.libhubbub
netsurf.libnsbmp
netsurf.libnsgif
netsurf.libnsutils
netsurf.libparserutils
netsurf.libsvgtiny

Probably drop libnsgif from withBackends in imv? I don't see a patch for https://bugs.gentoo.org/922496 in other distros, most of them just pass -Dlibnsgif=disabled.
Also, toppler isn't a regression of this PR. It already fails on master.

I have not undrafted it yet as it still breaks imv and toppler, should that be ignored?

This is not the first time we throw Darwin under the bus to the backlog.

Result of nixpkgs-review pr 285118 run on x86_64-linux 1

2 packages marked as broken and skipped:

gimpPlugins.exposureBlend
gimpPlugins.resynthesizer

1 package failed to build:

toppler

29 packages built:

gegl
gegl.dev
gegl.devdoc
gimp
gimp-with-plugins
gimp.dev
gimpPlugins.bimp
gimpPlugins.farbfeld
gimpPlugins.fourier
gimpPlugins.gap
gimpPlugins.gimplensfun
gimpPlugins.gmic
gimpPlugins.lightning
gimpPlugins.lqrPlugin
gimpPlugins.texturize
gimpPlugins.waveletSharpen
gnome-photos
gnome-photos.installedTests
imv
imv.man
netsurf.browser
netsurf.libcss
netsurf.libdom
netsurf.libhubbub
netsurf.libnsbmp
netsurf.libnsgif
netsurf.libnsutils
netsurf.libparserutils
netsurf.libsvgtiny


docs: website margin improvements

Hi I found some bugs on the site so I fixed that.
Changes

Fixed the floating div that didn't have enough space when scrolling.
Added side spacing on mobile view.
Added animaitons for logo and text in navbar.

@malezjaa Hey thanks so much for submitting these changes- I know we didn't do much mobile testing for layout, and it's good to know about some of these issues.
When I tested the changes out on my 1920x1080 monitor, things looked a little off. I'd prefer to maintain the original alignments/sizes of some of these items that appear to have changed.
Would you be able to test out your changes on a 1080p screen and make sure the overall appearance of the website doesn't change in these areas compared to the live https://arktype.io site?
###Original (changes circled):

###Updated:

I also noticed the background of some of the code demos changed and looks a little off to me now. Not sure if again it just has something to do with the screen:
###Original:

###Updated:

In general, if there are a few margins you can adjust that still look good on a desktop and fix the display on mobile, that's great and it would be awesome if you could submit those!
There are a lot of other tweaks here though, the purpose of which isn't totally clear,  so if you want to make other changes maybe you can individually explain them ideally with images of where it looks problematic, that'd be really helpful so I can better understand!

Sorry about that 😄 I think I have fixed it.

Awesome, this looks much better!
The only thing left is just some of the proportions around the installation block, which I notice is quite a bit larger, filling up the entire space of the header.
Is there any way to solve the issue you're having on mobile while still preserving the margins that were originally defined or something close to them?
Original:

Updated:

I definitely prefer the condensed look for this widget.

Thank you for your time. 🙏


Provide a way to sleep until rate limit resets

Because this is tough to get right and doesn't work in all cases, I am not going to add this until someone shows a need for this and an approach is found to be reasonable in the common case to warrant adding the code.

One approach as a context manager (to help delineate the code waiting for quota):
class _RateLimitManager:

    """A context manager which waits until there is some request quota.

    This context manager is **not** thread-safe.
    """

    def __init__(self, gh: "GitHubAPI", need: int) -> None:
        self._gh = gh
        self._need = need

    def update_rate_limit(self) -> None:
        """Update the rate limit in a blocking fashion."""
        pass

    async def __aenter__(self) -> None:
        """Wait until there is at least enough requested quota."""
        if self._gh.rate_limit is None:
            # First time using the GitHubAPI instance, so get accurate rate
            # limit details.
            self.update_rate_limit()
        # A worst-case scenario is when the rate limit is reached, the limit is
        # N requests, there are N+1 sleeping coroutines, and all N+1 coroutines
        # wake up in FIFO order and being waiting for responses before the first
        # request completes with the new rate limit details. To prevent this
        # worst-case from occurring, the last step is short-circuited by doing
        # a **blocking** check for the current rate limit. That makes sure
        # there's no potential race conditions in getting the new rate limit
        # when the reset epoch has passed.
        while self._gh.rate_limit.remaining < self._need:
            now = datetime.datetime.now(datetime.timezone.utc)
            if self._gh.rate_limit.reset_datetime > now:
                wait = self.rate_limit.reset_datetime - now
                await self._sleep(wait.total_seconds())
            else:
                self.update_rate_limit()

    async def __aexit__(self, exc_type, exc, tb):
        pass

Could add a method like await gh.wait_for_quota(method, *args, **kwargs) which simply postpones calling the method provided until quota for a single call is available.

With the automerge feature, miss-islington now encounters ratelimit issues 😞
So this feature will be appreciated.

A slightly less elegant approach would be add one more piece of state to the GithubAPI class of a user-injectable rate limit manager co-routine + keeping track of the number of requests that are "out".
I propose the following:

adding a requests_in_fight counter
add a user injected co-routine to the GithubAPI instance to decide how to throttle requests
add another method to GithubAPI called _managed_request that updates the "in-flight" counter and calls the above co-routine.

The requests_in_flight is a bit redundent the rate limit, but I think is needed.  In _make_request the rate limit is pre-emptively decremented, but I think that leaves open the issue of make N of N requests before any come back, the first one comes back with N-1 requests remaining, try to make N requests -> a whole bunch of those are over the rate limit and get bounced.
Making the logic inject-able via a co-routine makes it easy for users to experiment with different strategies for deciding when / how long to sleep based on the rate limit.  For example I can imagine that in some cases you would prefer to throttle yourself but maintain a constant rate of requests rather than exhausting the limit in bursts and have long silent periods.

About 10 minutes after posting the previous message I re-read the code and realized that _make_request is the right place to put the additional accounting so the last point should be "put the extra logic in _make_request :man_facepalming:

I do like your requests_in_fight typo. 😉

Punting on the solution out to users to decide how they want to manage it is definitely interesting. Maybe if a check_quota() method is defined then it is used at the very start of _make_request()? Or provide a default method that immediately returns and allow for overloading in subclasses (once again called at the start of _make_request()? Since the current rate limit is stored on the GitHubAPI instance already it shouldn't be necessary to do much else.
Or am I missing something from this idea? Seems too straight-forward ...

I am definitely looking for this feature right now, and not finding it anywhere.
A general comment.
https://github.com/brettcannon/gidgethub/blob/3b8941a0ccdad007db09dd601963ce989d64074f/gidgethub/abc.py#L114
As I understand the code right now, it seems that the class stores the rate limit of whatever method you happened to be calling last (there are separate rate limits). Running out of one quota, then calling another kind of method to "reset" the quota, will avoid detection.

It seems to me that a robust solution would have to react to the event that GitHub responds with "out of quota", rather than doing all the magic to avoid doing it. And, again, looking at the code, there's absolutely no way for a user of this library to achieve this.
In particular, consider getiter.
https://github.com/brettcannon/gidgethub/blob/3b8941a0ccdad007db09dd601963ce989d64074f/gidgethub/abc.py#L138
Let's say I'm doing a search which has 100 pages. The search quota is 30 per minute. How am I supposed to ever consume the whole output, if my only recourse is to catch the exception and try again from the beginning?
And the context manager shown above is also completely deaf to getiter usages, it only ensures that we have 1 quota remaining as I see, but during iteration nothing is checked.

I've been encountering rate-limit issues myself, and also related to the search API using the getiter coroutine, as described by @oprypin above.
At the moment I tried overcoming this by not using getiter, and just use getitem and do my own iteration. But it is a bit tedious.
Currently the getitem just ignored the more value that was returned by _make_request. Perhaps if we can start returning it to the caller, that might help?
https://github.com/brettcannon/gidgethub/blob/4774a896f87023bcd3954cdf072656e69ca3e9ea/gidgethub/abc.py#L133-L136
https://github.com/brettcannon/gidgethub/blob/4774a896f87023bcd3954cdf072656e69ca3e9ea/gidgethub/abc.py#L120

How would changing the return value be backwards-compatible? I suspect most users don't hit limits so I would prefer to not break the API for this if there's another way to provide details.
Another option is to do something as @oprypin suggested in https://github.com/brettcannon/gidgethub/discussions/153 and have the iterator have rate limit details that are updated after each iteration. It would require iterating manually or using a custom iterator underneath that implemented whatever backpressure strategy you wanted to use.

im hitting this when converting some code from pygithub that uses the search/issues endpoint
probably relevant - the code is not making concurrent requests - so simple serial requests
i can probably think of a workaround - but i thought gidgethub handled rate limiting transparently


Link to Documentation is broken

Link: https://yweather.readthedocs.org/
Results in page not found: "You've found something that doesn't exist."

@matt-the-ogre Thanks for pointing that out! I fixed the settings on Read the Docs, so the link is working now.


home-manager extraCss option for more targets

Currently, it's quite difficult to add CSS on top of the CSS generated by Stylix. The main issue is when it comes to overwriting CSS rules, because the user-defined CSS (for example programs.waybar.style in home.nix) is applied before the CSS generated by Stylix.
Perhaps we could add an extraCss option to more (hopefully eventually all) targets, as it is already a feature for GTK, and would be as simple (I think?) as appending the text in the user-defined extraCss to the CSS that is already being applied to the target. For example:
programs.waybar.style =
  # ...
  + (if config.stylix.targets.waybar.extraCss then stylix.targets.waybar.extraCss else ""); # I don't think this code works, but I hope It's somewhat illustrative

This could also act as a band-aid solution to #429 and maybe some other feature requests concerning CSS, as users could simply overwrite existing CSS rules.
The other option may be to somehow get the user-defined programs.waybar.style to apply after Stylix's version, but I'm not an expert at nix, and I don't know how feasible this is. It's also not as universal a solution.

There are mkBefore and mkAfter functions in lib. These change the order in which option definitions are merged; for example the two definitions "a" and mkBefore "b" would be guaranteed to merge as "ba".
If we add mkBefore to any CSS defined by Stylix, then we can guarantee that CSS added by the user will be applied later without needing to create another option.

So,
config = lib.mkIf (config.stylix.enable && config.stylix.targets.waybar.enable) {
    programs.waybar.style = # ...
};

would become
config = lib.mkIf (config.stylix.enable && config.stylix.targets.waybar.enable) lib.mkBefore {
    programs.waybar.style = # ...
};

?
I'm relatively new to the nix language, so sorry if I'm missing the obvious.

I believe the mkBefore needs to go on the programs.waybar.style option specifically, so it should be programs.waybar.style = lib.mkBefore ...

Ah, okay. I'm still getting my setup in order, so it may take a while for me to submit a PR.


Pure JSON layout and autoload

I would recommand to use pure json layout and autoload layout

I have thought of it, but I didn't want to cause path or network issues.
Also, it's nice to only load the layouts you need.
I'm open to suggestions though :)

Well, I'm not talking about path error, it's simple loading based on path declaration like a translation or something.
You can add a default layout inside the min version of the lib and keep things separate before build them.

Good idea to minify default layout with the lib !

Just added a build task wich minify scripts and styles inside ./dist/ folder

:+1:


Free the managers

When deploying on a Swarm cluster (made of more than 1 manager node), the best practice is to not deploy any task on the manager nodes.
Services should be constrained out of the manager nodes:

haproxy
amplifier
It has an impact on how these services have access to the Swarm API (they used to mount the docker socket). Keep in mind that the Engine API may be not exposed, or may require mutual auth.

The worker nodes have the knowledge about the manager nodes, it can be easily discovered:
$ docker info --format '{{range .Swarm.RemoteManagers}}{{.Addr}} {{end}}'


Remove version: IrfanSkiljan.IrfanView.PlugIns version 4.70

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


FAT-6493: actualize match rules JSON according to SRM response

Purpose
To fix the Scenario: FAT-4701 check that default rules in karate tests matches what is in the SRM under test.
Approach
Use the actual JSON response from SRM. We see that test Scenario: FAT-4701... is not failing anymore:
https://jenkins-aws.indexdata.com/job/Testing/job/Karate tests/674

Learning
JIRA

Is only the formatting different, or some rules were not the same as well?

Looks like yes, after the conditions up to date in https://github.com/folio-org/folio-integration-tests/pull/1024


ios must have secure endpoint

string queryString = "http://api.openweathermap.org/data/2.5/weather?zip="
+ zipCode + ",us&appid=" + key + "&units=imperial";
should use https://api.openweathermap.org

This is the topic: https://docs.microsoft.com/en-us/visualstudio/cross-platform/build-apps-with-native-ui-using-xamarin-in-visual-studio#write-shared-data-service-code
@Oostwegel Thanks for reporting this.


[BUG] Problem with /improve and /impove --extended

Full log:
WARNING:root:Failed to generate prediction with gpt-4: Traceback (most recent call last):
  File "/app/pr_agent/algo/git_patch_processing.py", line 211, in convert_to_hunks_with_lines_numbers
    start1, size1, start2, size2 = map(int, match.groups()[:4])
TypeError: int() argument must be a string, a bytes-like object or a real number, not 'NoneType'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/pr_agent/algo/pr_processing.py", line 215, in retry_with_fallback_models
    return await f(model)
  File "/app/pr_agent/tools/pr_code_suggestions.py", line 75, in _prepare_prediction
    self.patches_diff = get_pr_diff(self.git_provider,
  File "/app/pr_agent/algo/pr_processing.py", line 60, in get_pr_diff
    patches_extended, total_tokens, patches_extended_tokens = pr_generate_extended_diff(pr_languages, token_handler,
  File "/app/pr_agent/algo/pr_processing.py", line 109, in pr_generate_extended_diff
    full_extended_patch = convert_to_hunks_with_lines_numbers(extended_patch, file)
  File "/app/pr_agent/algo/git_patch_processing.py", line 213, in convert_to_hunks_with_lines_numbers
    start1, size1, size2 = map(int, match.groups()[:3])
TypeError: int() argument must be a string, a bytes-like object or a real number, not 'NoneType'

INFO:root:Getting PR diff...
WARNING:root:Failed to generate prediction with gpt-3.5-turbo-16k: Traceback (most recent call last):
  File "/app/pr_agent/algo/git_patch_processing.py", line 211, in convert_to_hunks_with_lines_numbers
    start1, size1, start2, size2 = map(int, match.groups()[:4])
TypeError: int() argument must be a string, a bytes-like object or a real number, not 'NoneType'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/pr_agent/algo/pr_processing.py", line 215, in retry_with_fallback_models
    return await f(model)
  File "/app/pr_agent/tools/pr_code_suggestions.py", line 75, in _prepare_prediction
    self.patches_diff = get_pr_diff(self.git_provider,
  File "/app/pr_agent/algo/pr_processing.py", line 60, in get_pr_diff
    patches_extended, total_tokens, patches_extended_tokens = pr_generate_extended_diff(pr_languages, token_handler,
  File "/app/pr_agent/algo/pr_processing.py", line 109, in pr_generate_extended_diff
    full_extended_patch = convert_to_hunks_with_lines_numbers(extended_patch, file)
  File "/app/pr_agent/algo/git_patch_processing.py", line 213, in convert_to_hunks_with_lines_numbers
    start1, size1, size2 = map(int, match.groups()[:3])
TypeError: int() argument must be a string, a bytes-like object or a real number, not 'NoneType'

Traceback (most recent call last):
  File "/app/pr_agent/algo/git_patch_processing.py", line 211, in convert_to_hunks_with_lines_numbers
    start1, size1, start2, size2 = map(int, match.groups()[:4])
TypeError: int() argument must be a string, a bytes-like object or a real number, not 'NoneType'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/pr_agent/servers/github_action_runner.py", line 120, in <module>
    asyncio.run(run_action())
  File "/usr/local/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/local/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
    return future.result()
  File "/app/pr_agent/servers/github_action_runner.py", line 94, in run_action
    await PRAgent().handle_request(pr_url, new_command)
  File "/app/pr_agent/agent/pr_agent.py", line 82, in handle_request
    await command2class[action](pr_url, args=args).run()
  File "/app/pr_agent/tools/pr_code_suggestions.py", line 57, in run
    await retry_with_fallback_models(self._prepare_prediction)
  File "/app/pr_agent/algo/pr_processing.py", line 215, in retry_with_fallback_models
    return await f(model)
  File "/app/pr_agent/tools/pr_code_suggestions.py", line 75, in _prepare_prediction
    self.patches_diff = get_pr_diff(self.git_provider,
  File "/app/pr_agent/algo/pr_processing.py", line 60, in get_pr_diff
    patches_extended, total_tokens, patches_extended_tokens = pr_generate_extended_diff(pr_languages, token_handler,
  File "/app/pr_agent/algo/pr_processing.py", line 109, in pr_generate_extended_diff
    full_extended_patch = convert_to_hunks_with_lines_numbers(extended_patch, file)
  File "/app/pr_agent/algo/git_patch_processing.py", line 213, in convert_to_hunks_with_lines_numbers
    start1, size1, size2 = map(int, match.groups()[:3])
TypeError: int() argument must be a string, a bytes-like object or a real number, not 'NoneType'

can you share a reproducable example ?

Yep, here is demo repository with github action. It use pr-agent form github package. PR-agent fork with pr_commands and local config reader applied to github_action_runner and published to github package. I've added you to both repositories.

i think this should fix this, some patch formatting issues
https://github.com/Codium-ai/pr-agent/pull/315


Key mismatch while loading the model?

I am having issue loading the trained checkpoint to FPNSSD512 model.
How can I fix that?
RuntimeError: Error(s) in loading state_dict for FPNSSD512:
	Missing key(s) in state_dict: "fpn.conv1.weight", "fpn.bn1.running_var", "fpn.bn1.bias", "fpn.bn1.running_mean", "fpn.bn1.weight", "fpn.layer1.0.conv1.weight", "fpn.layer1.0.bn1.running_var", "fpn.layer1.0.bn1.bias", "fpn.layer1.0.bn1.running_mean", "fpn.layer1.0.bn1.weight", "fpn.layer1.0.conv2.weight", "fpn.layer1.0.bn2.running_var", "fpn.layer1.0.bn2.bias",

        Unexpected key(s) in state_dict: "module.fpn.conv1.weight", "module.fpn.bn1.weight", "module.fpn.bn1.bias", "module.fpn.bn1.running_mean", "module.fpn.bn1.running_var", "module.fpn.layer1.0.conv1.weight"

following code before loading the checkpoint solved the issue
if device == 'cuda':
    net = torch.nn.DataParallel(net)
    cudnn.benchmark = True

Dear @Jumabek,
I have also your reported issue.
my script is something like this:
import torch
import torch.backends.cudnn as cudnn
from models.fpnssd.net import FPNSSD512

# Print the PyTorch Version:
print(torch.__version__)  # 0.4.0

# *************** Parameters **************** #
# Check use GPU or not
use_gpu = torch.cuda.is_available()  # use GPU
if use_gpu:
    device = torch.device("cuda:0")
else:
    device = torch.device("cpu")

# ** Loading Pre-Trained Weights:
net = FPNSSD512(num_classes=20).to(device)
net = torch.nn.DataParallel(net)
cudnn.benchmark = True
# download pre-trained weights from:
# https://drive.google.com/open?id=1yy_kUnm_hZR3uk9yLcaQSMwxVn7wApTU
net.load_state_dict(torch.load('./fpnssd512_20_trained.pth'))
net.eval()

However, I got your reported error. Would you please help me to address this issue?

Dear @kuangliu,
Would you please answer my above question?

@ahkarami sorry for late reply.
While I do not fully understand the issue.
Can you run the code below:
import torch
import torch.backends.cudnn as cudnn
from models.fpnssd.net import FPNSSD512

# Print the PyTorch Version:
print(torch.__version__)  # 0.4.0

# *************** Parameters **************** #
# Check use GPU or not
use_gpu = torch.cuda.is_available()  # use GPU
if use_gpu:
    device = torch.device("cuda:0")
else:
    device = torch.device("cpu")

# ** Loading Pre-Trained Weights:
net = FPNSSD512(num_classes=20).to(device)
net = torch.nn.DataParallel(net)
cudnn.benchmark = True
# download pre-trained weights from:
# https://drive.google.com/open?id=1yy_kUnm_hZR3uk9yLcaQSMwxVn7wApTU
net.load_state_dict(torch.load('./fpnssd512_20_trained.pth'))
net.eval()

Dear @Jumabek,
Thank you for your reply. Sorry for my inconvenience. I have tested your recommended script, but unfortunately the error is remain. The error is:
Traceback (most recent call last):
  File "/home/user/TorchCV/Attempt1.py", line 54, in <module>
    net.load_state_dict(torch.load('./fpnssd512_20_trained.pth'))
  File "/opt/pytorch4/torch/nn/modules/module.py", line 721, in load_state_dict
    self.__class__.__name__, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for FPNSSD512:
	Missing key(s) in state_dict: "fpn.conv1.weight", "fpn.bn1.running_mean", "fpn.bn1.running_var", ...
	Unexpected key(s) in state_dict: "extractor.conv1.weight", "extractor.bn1.weight", "extractor.bn1.bias", ....

Process finished with exit code 1

It is worth nothing that I have tested the above code on a system which has just one GTX 1080ti GPU (with CUDA 9.0 & cuDNN 7).

Dear @Jumabek,
Thank you for your reply. Sorry for my inconvenience. I have tested your recommended script, but unfortunately the error is remain. The error is:
Traceback (most recent call last):
  File "/home/user/TorchCV/Attempt1.py", line 54, in <module>
    net.load_state_dict(torch.load('./fpnssd512_20_trained.pth'))
  File "/opt/pytorch4/torch/nn/modules/module.py", line 721, in load_state_dict
    self.__class__.__name__, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for FPNSSD512:
	Missing key(s) in state_dict: "fpn.conv1.weight", "fpn.bn1.running_mean", "fpn.bn1.running_var", ...
	Unexpected key(s) in state_dict: "extractor.conv1.weight", "extractor.bn1.weight", "extractor.bn1.bias", ....

Process finished with exit code 1

It is worth nothing that I have tested the above code on a system which has just one GTX 1080ti GPU (with CUDA 9.0 & cuDNN 7).

Hi I followed your code and seems helped me solve the issue of unexpected key,
But I'm wondering what's the reason for it to occur?
Why is DataParallel help to solve it?

Dear @zacario-li,
Unfortunately I couldn't address the issue. I can train & test model by my own GPU (i.e., my trained models are correct) but the released pre-trained model has the above issue. I think the problem related to this fact that the pre-trained model has been trained on a machine with multi GPU but now we want to use it in a machine with just one GPU. However, In this case using the torch.nn.DataParallel(net) command must address the problem, but we saw that this command can't solve the problem!!!

Dear @ahkarami ,
I think the pretrained fpnssd model provided by @kuangliu is not the same as /models/fpnssd/net.py. Actually, he said that he just replaced vgg16 by fpn50 in ssd512 which is /models/ssd/net.py. So you could not use the model created by /models/fpnssd/net.py to load the wights in /models/ssd/net.py as the keys are not matched.
The solution to use his provided pretrained model is to train his ssd512 model with fpn50 not fpnssd512 model in /models/fpnssd/net.py.
Also, it seems that he did not put all of his examples on this github or he delete something before pushing.

Dear @silkylove,
Thank you very much for your useful information. Could you load & use his pre-trained network?
If yes, would you please release its loading code?

Dear @ahkarami ,
Ok, I will release the code after I get similar performence compared to his pretrained fpnssd512 model.

Thank you very much @silkylove.

@ahkarami
Please check my code.
https://github.com/silkylove/ObjectDetection/tree/master/example/fpnssd
I also uploaded the training log with adam with 100 epochs which could get 73.95mAP until now. I am now training SGD with 200 epochs on that which I think would get higher mAP, I will release the training log later.
Also, you can uncommen this line in eval.py https://github.com/silkylove/ObjectDetection/blob/master/example/fpnssd/eval.py#L25 to got his pertrained model's performence (about 56mAP)

@ahkarami
I uploaded the sgd training and eval log. And with sgd, I can only got aound 76% mAP now. The pretrained model was in here.

@dearleiii and others, why using torch.nn.DataParallel, the issue gets resolved?
I faced the same issue, and the same solution as @Jumabek worked for me, but not able to understand the reasoning.
Thanks.


CAMEL-16064: added batching classes

TODO: add documentation.

CI tests are looking good. I'll provide the documentation updates on a separate ticket.


utf8 problem on Windows

In version 7.4.1., pylama cant open files containing utf-8 characters. When executed in --verbose mode, it points to file core.py:
Traceback (most recent call last):
  File "***\site-packages\pylama\core.py", line 42, in run
    with CodeContext(code, path) as ctx:
  File "***\site-packages\pylama\core.py", line 196, in __enter__
    self.code = self._file.read()
  File "***\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x8d in position 835: character maps to <undefined>

This is caused by line 195 in core.py, because default encoding in Windows is not utf-8.
self._file = open(self.path, mode)

Changing the line to this fixes the issue.
self._file = open(self.path, mode, encoding='utf-8')

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@polakluk Hello, fixed in version > 7.4.2


If you scroll down then exit, this doesn't work

Basically, if I don't scroll and exit from the top, it works great. But on a longer page, if I scroll down a bit then it no longer works. Might have to do with it being bound to the document instead of the viewport? Unsure.

Yep, noticed the same thing.
This is easily fixed by using clientY instead of pageY in line 5.


pymongo.errors.OperationFailure

When I install girder follow the HismoicsUI respository，execute “girder serve”,
I get pymongo.errors as follows:
pymongo.errors.OperationFailure: command createIndexes requires authentication, full error: {'ok': 0.0, 'errmsg': 'command createIndexes requires authentication', 'code': 13, 'codeName': 'Unauthorized'}

Did you install Mongo so that it requires authentication?  If so, you'll need to modify the database uri for Girder to include the appropriate credentials.  This is either done in the grider config file or as part of the --database option for "girder serve" -- I'd have to check Mongo documentation to determine how the authentication is included in that uri.

Closing without further information.  Feel free to reopen if you have more details.


Search in Mountain leetcode problem.

Hey Aditya,
This is  my hacktoberfest contribution this year I add 1 leetcode solution into java repo the code is working properly.
Please approve my PR.

I complete your contribute criteria i star the repo also solve and add 2 leetcode problem solution in Java repo

Hey, @ar7aditya  My submission was excluded which rules have you not followed?


runtime: cargo test fails with modern GCC

cargo test fails to build with newer gcc (11.1.1) due to C dependencies pulled in by our ancient version of grpcio-sys.
  /home/yawning/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.4.7/grpc/third_party/boringssl/third_party/fiat/curve25519.c:1905:57: error: argument 2 of type ‘const uint8_t *’ {aka ‘const unsigned char *’} declared as a pointer [-Werror=array-parameter=]
   1905 | void x25519_ge_scalarmult_base(ge_p3 *h, const uint8_t *a) {
        |                                          ~~~~~~~~~~~~~~~^
  In file included from /home/yawning/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.4.7/grpc/third_party/boringssl/third_party/fiat/curve25519.c:41:
  /home/yawning/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.4.7/grpc/third_party/boringssl/third_party/fiat/internal.h:125:56: note: previously declared as an array ‘const uint8_t[32]’ {aka ‘const unsigned char[32]’}
    125 | void x25519_ge_scalarmult_base(ge_p3 *h, const uint8_t a[32]);
        |                                          ~~~~~~~~~~~~~~^~~~~
  cc1: all warnings being treated as errors
  gmake[3]: *** [third_party/boringssl/third_party/fiat/CMakeFiles/fiat.dir/build.make:82: third_party/boringssl/third_party/fiat/CMakeFiles/fiat.dir/curve25519.c.o] Error 1
  gmake[2]: *** [CMakeFiles/Makefile2:3544: third_party/boringssl/third_party/fiat/CMakeFiles/fiat.dir/all] Error 2
  gmake[2]: *** Waiting for unfinished jobs....
  gmake[1]: *** [CMakeFiles/Makefile2:1727: CMakeFiles/grpc.dir/rule] Error 2
  gmake: *** [Makefile:300: grpc] Error 2
  thread 'main' panicked at '
  command did not execute successfully, got: exit status: 2

Can you check if it works with #3950?

Can you check if it works with #3950?

  /home/yawning/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.8.1/grpc/third_party/abseil-cpp/absl/synchronization/internal/graphcycles.cc: In member function ‘void absl::lts_2020_09_23::synchronization_internal::GraphCycles::RemoveNode(void*)’:
  /home/yawning/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.8.1/grpc/third_party/abseil-cpp/absl/synchronization/internal/graphcycles.cc:451:26: error: ‘numeric_limits’ is not a member of ‘std’
    451 |   if (x->version == std::numeric_limits<uint32_t>::max()) {
        |                          ^~~~~~~~~~~~~~
  /home/yawning/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.8.1/grpc/third_party/abseil-cpp/absl/synchronization/internal/graphcycles.cc:451:49: error: expected primary-expression before ‘>’ token
    451 |   if (x->version == std::numeric_limits<uint32_t>::max()) {
        |                                                 ^
  /home/yawning/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.8.1/grpc/third_party/abseil-cpp/absl/synchronization/internal/graphcycles.cc:451:52: error: ‘::max’ has not been declared; did you mean ‘std::max’?
    451 |   if (x->version == std::numeric_limits<uint32_t>::max()) {
        |                                                    ^~~
        |                                                    std::max
  In file included from /usr/include/c++/11/algorithm:62,
                   from /home/yawning/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.8.1/grpc/third_party/abseil-cpp/absl/synchronization/internal/graphcycles.cc:38:
  /usr/include/c++/11/bits/stl_algo.h:3467:5: note: ‘std::max’ declared here
   3467 |     max(initializer_list<_Tp> __l, _Compare __comp)
        |     ^~~
  gmake[3]: *** [third_party/abseil-cpp/absl/synchronization/CMakeFiles/absl_graphcycles_internal.dir/build.make:82: third_party/abseil-cpp/absl/synchronization/CMakeFiles/absl_graphcycles_internal.dir/internal/graphcycles.cc.o] Error 1
  gmake[2]: *** [CMakeFiles/Makefile2:3340: third_party/abseil-cpp/absl/synchronization/CMakeFiles/absl_graphcycles_internal.dir/all] Error 2
  gmake[2]: *** Waiting for unfinished jobs....
  gmake[1]: *** [CMakeFiles/Makefile2:1009: CMakeFiles/grpc.dir/rule] Error 2
  gmake: *** [Makefile:261: grpc] Error 2
  thread 'main' panicked at '
  command did not execute successfully, got: exit status: 2

It managed to move the error around at least.

This is an upstream issue, see https://github.com/tikv/grpc-rs/issues/524.

@Yawning can you check if #3968 fixes this issue for you?

Yeah it does.  While I'm complaining about this test battery, can we do something about
error: environment variable `OASIS_STORAGE_PROTOCOL_SERVER_BINARY` not defined
  --> runtime/src/storage/mkvs/interop/protocol_server.rs:24:46
   |
24 | const PROTOCOL_SERVER_BINARY: &'static str = env!("OASIS_STORAGE_PROTOCOL_SERVER_BINARY");
   |                                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: this error originates in the macro `env` (in Nightly builds, run with -Z macro-backtrace for more info)

As well?  (Eg: auto-skip/fail the tests if it isn't defined)

You could run make test-unit-rust to set these up for you. But I agree that we should fix the above to not fail compilation in case the env variable is unset.


ci: fix rhel >= 8.4 nm-cloud-setup related error when setup k3s cluster.

ci: fix rhel >= 8.4 nm-cloud-setup related error when setup k3s cluster.
For https://github.com/longhorn/longhorn/issues/4333
Signed-off-by: Yang Chiu<EMAIL_ADDRESS>

@yangchiu We need to backport this to 1.2 and 1.3.


Bug/flex 4879 fix device replacement

SonarQube analysis reported 2 issues

 2 major

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 SsldConverter.java#L31: Remove this unused "LOGGER" private field.

SonarQube analysis reported 2 issues

 2 major

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 SsldConverter.java#L31: Remove this unused "LOGGER" private field.

SonarQube analysis reported 2 issues

 2 major

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 SsldConverter.java#L31: Remove this unused "LOGGER" private field.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 SsldConverter.java#L31: Remove this unused "LOGGER" private field.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 SsldConverter.java#L31: Remove this unused "LOGGER" private field.


Grammar fix

A small fix for description

Thanks!
We have to chnage it in the NetKAN repository first, else our bot would overwrite it again.
The NetKAN change will automatically be backported to the newest version (1.2.3).
It's basically the same as this PR, you would just need to edit this file instead.
It's not absolutely necessary to also change 1.2.2, because the GUI will take the description from the latest release, however if you want to, we can merge this PR afterwards.


Typo in the Lazy instructions?

Hello,
I think there is an error in the Lazy instructions:
	{
		"nvim-zh/colorful-winsep.nvim",
		config = function()
			asynrequire("packers.nvimsep")
		end,
		event = { "WinNew" },
	}

Should asynrequire("packers.nvimsep") be require("colorful-winsep").setup()?

Thanks for reminding

I am not sure about the fix you applied.
The name of the package seems incorrect and also we don't need to call the setup function?

I am not sure about the fix you applied.
The name of the package seems incorrect and also we don't need to call the setup function?

no problem


add logging middleware for server

fixes #784

Here's how the logs look :
./bin/terrascan server
2021-05-18T18:33:14.591+0530	info	http-server/start.go:61	registering routes...
2021-05-18T18:33:14.591+0530	info	http-server/start.go:65	Route GET - /health
2021-05-18T18:33:14.591+0530	info	http-server/start.go:65	Route POST - /v1/{iac}/{iacVersion}/{cloud}/local/file/scan
2021-05-18T18:33:14.591+0530	info	http-server/start.go:65	Route POST - /v1/{iac}/{iacVersion}/{cloud}/remote/dir/scan
2021-05-18T18:33:14.591+0530	info	http-server/start.go:65	Route GET - /k8s/webhooks/{apiKey}/logs
2021-05-18T18:33:14.591+0530	info	http-server/start.go:65	Route GET - /k8s/webhooks/logs/{uid}
2021-05-18T18:33:14.592+0530	info	http-server/start.go:65	Route POST - /v1/k8s/webhooks/{apiKey}/scan/validate
2021-05-18T18:33:14.592+0530	info	http-server/start.go:95	http server listening at port 9010
2021-05-18T18:33:19.345+0530	info	logging/logwriter.go:25	::1 - - "GET /health HTTP/1.1" 200 0
2021-05-18T18:33:42.464+0530	info	logging/logwriter.go:25	::1 - - "GET /health HTTP/1.1" 200 0
2021-05-18T18:33:53.427+0530	info	logging/logwriter.go:25	::1 - - "GET /health HTTP/1.1" 200 0
2021-05-18T18:34:01.815+0530	info	logging/logwriter.go:25	::1 - - "GET / HTTP/1.1" 404 0

In JSON fomat :
./bin/terrascan server --log-type json
{"level":"info","time":"2021-05-18T18:34:47.494+0530","file":"http-server/start.go:61","msg":"registering routes..."}
{"level":"info","time":"2021-05-18T18:34:47.494+0530","file":"http-server/start.go:65","msg":"Route GET - /health"}
{"level":"info","time":"2021-05-18T18:34:47.494+0530","file":"http-server/start.go:65","msg":"Route POST - /v1/{iac}/{iacVersion}/{cloud}/local/file/scan"}
{"level":"info","time":"2021-05-18T18:34:47.494+0530","file":"http-server/start.go:65","msg":"Route POST - /v1/{iac}/{iacVersion}/{cloud}/remote/dir/scan"}
{"level":"info","time":"2021-05-18T18:34:47.494+0530","file":"http-server/start.go:65","msg":"Route GET - /k8s/webhooks/{apiKey}/logs"}
{"level":"info","time":"2021-05-18T18:34:47.494+0530","file":"http-server/start.go:65","msg":"Route GET - /k8s/webhooks/logs/{uid}"}
{"level":"info","time":"2021-05-18T18:34:47.495+0530","file":"http-server/start.go:65","msg":"Route POST - /v1/k8s/webhooks/{apiKey}/scan/validate"}
{"level":"info","time":"2021-05-18T18:34:47.495+0530","file":"http-server/start.go:95","msg":"http server listening at port 9010"}
{"level":"info","time":"2021-05-18T18:34:52.123+0530","file":"logging/logwriter.go:25","msg":"::1 - - \"GET /health HTTP/1.1\" 200 0"}
{"level":"info","time":"2021-05-18T18:34:53.262+0530","file":"logging/logwriter.go:25","msg":"::1 - - \"GET /health HTTP/1.1\" 200 0"}
{"level":"info","time":"2021-05-18T18:34:54.702+0530","file":"logging/logwriter.go:25","msg":"::1 - - \"GET /health HTTP/1.1\" 200 0"}
{"level":"info","time":"2021-05-18T18:35:01.599+0530","file":"logging/logwriter.go:25","msg":"::1 - - \"GET / HTTP/1.1\" 404 0"}
{"level":"info","time":"2021-05-18T18:35:03.543+0530","file":"logging/logwriter.go:25","msg":"::1 - - \"GET / HTTP/1.1\" 404 0"}
{"level":"info","time":"2021-05-18T18:35:10.529+0530","file":"logging/logwriter.go:25","msg":"::1 - - \"GET /health HTTP/1.1\" 200 0"}

Updated the PR as per the reviews. PTAL

This is how the logging output looks now.


Use a released version of cborg

The dependency on cborg was pinned to an unreleased revision to bring in
https://github.com/well-typed/cborg/pull/223. That PR has been included in
cborg-<IP_ADDRESS>, so we can remove the dependency pin in favour of bumping the
lower version bound on the dependency.
Instead of using 0.2.3 as the lower version bound, bump it to >= 0.2.4, as this
version is compatible with the last version of the primitive package, which
will soon be needed by cardano-ledger-specs, see
https://github.com/input-output-hk/cardano-ledger-specs/pull/1785.
Another advantage of using a released version is that the package will now be
stored in the global cabal store and shared between projects (although that will
also be the case for source dependencies in Cabal <IP_ADDRESS>).

The commits from this PR are also included in https://github.com/input-output-hk/cardano-node/pull/1711, as they're needed to fix a dependency issue (primitive >= <IP_ADDRESS>). So if #1711 gets merged first, this PR can be closed.

No longer needed, since #1696 was merged

Ughh, I mentioned the wrong PR in my last comment :facepalm:. Thanks for cleaning this up.


Convert Time of Plenty and Early Frost to use an effect

I thought there may be a bug introduced by this:
Reveal Time of Plenty, effect applied drawPhaseCards += 1
Full round goes by
Start of new plot phase, drawPhaseCards is explicitly set to 2 in `Player.startPlotPhase`
New plot is revealed, Time of Plenty's effect is unapplied by drawPhaseCards -= 1
drawPhaseCards is at 1 for the new round

However, it seems to work fine and I'm not sure why. Just something to keep our eyes open for...

Because of the condition property, the effect will already be unapplied when the marshalling phase starts?

Oops, missed the condition. Yup, that'd be why! 👍


Response status code: 503

HttpRequestException: Response status code does not indicate success: 503 (Service Temporarily Unavailable).
any help ?

Website just used cloudflare, I'll try to update it today.

Bypassed Cloudflare protection in version 1.1.3 .

Still happening, any ideas?

I tested it and it works good for me.
The problem that made that exception is the website uses cloudflare which uses two types of challenges.
One is javascript challenge and the other one is reCaptcha so the problem happens if it's a captcha or your IP is banned, the javascipt one can be solved easily.
I use this library CloudflareSolver to bypass cloudflare, it provides the fail reason if any of the two previous problems happened.
I provides solving the captcha challenge if you provided a 2Captcha key (it's not free).
I'll throw a descriptive exception telling if it needs 2captcha key or your IP is banned if there's a problem.
If the problem is your IP so you should change it.
I'll add the new exception to make it more clearer (my bad) today.
Until I update it try it using another IP to see if this is the problem.

Added 2Captcha in version 1.1.4.
Try it, and tell me if there's any problem.

I'll do ASAP.
Thank you very much!
El sáb., 23 mar. 2019 22:32, Hazem Ahmed Amr<EMAIL_ADDRESS>escribió:

Added 2Captcha in version 1.1.4
https://www.nuget.org/packages/TempMail.API/1.1.4.
Try it, and tell me if there's any problem.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/RyuzakiH/Temp-Mail-API/issues/2#issuecomment-475906471,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFZUeJeRzxBjQjTa9fYY8aOeC3J1nsMXks5vZp1rgaJpZM4b3nTz
.

Hi,
I'm afraid there is a different error now:

Cloudflare.CloudflareException: Cloudflare (JS): response cookie not found
en TempMail.API.Client.StartNewSession()

The code:
        static void Main(string[] args)
        {
            if (Environment.Is64BitOperatingSystem)
            {
                PrepararEntornoWin64();

                var client = new Client();
                client.StartNewSession();

                Console.WriteLine(client.Email);
            }
        }

It's an exception related to CloudflareSolver library, but it doesn't happen with me so I can't debug it, so I'll need your help.
Have you tried opening the site in browser just to make sure it's not an IP problem.
Then tell me if the response contains the below text or not (having captcha for example):

Checking your browser before accessing temp-mail.org.
This process is automatic. Your browser will redirect to your requested content shortly.
Please allow up to 5 seconds…

Another way is:
Try using Hotspot Shield (it ran successfully for me).
Try these and keep me updated.
I hope we can solve it.

Response contains that text, it lasts 5 seconds to get to my mail account. However, I'd prefer not to use a VPN (i'm trying to get some stuff working for everyone, not just for us).
Thank you in advance.

Since it's a CloudflareSolver problem, so I think it's Jint (Javascript engine that parses the JS challenge and solve it).
I'm saying Jint as this library only depends on 3 libraries:

Newtonsoft.Json
it can't be it I think.
2CaptchaAPI
you don't use a 2captcha api key.
Jint
maybe Jint interprets JS differently on different machines.

I'm just thinking with you, what about your OS?

If you have some time, can you use fiddler and run your code again.
Only 3 requests are issued, the first two are what we need.

GET request to temp-mail returns 503 Service Temporarily Unavailable
GET request to https://temp-mail.org/cdn-cgi/l/chk_jschl?s=S&pass=PASS&jschl_answer=JSCHL_ANSWER returns 302 Moved Temporarily
with the 4 capitalized words replaced with other values.

I need you to send me the responses of the previous two requests (response heades and body).
I'll run them on my machine and compare to identify which part is leading to that error.


pollingIntervalInS; where S means String

The pollingIntervalInS should be formatted as a String. In the example it is formatted as an Integer. The typeHandlerVersion should be formatted as a String. in the table it is defined as an Integer.

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

ID: a52b0724-6b3b-0daf-7749-d18104d6a40b
Version Independent ID: 56e5c7c4-f456-fc2e-99da-2771eb162947
Content: Azure Key Vault VM extension for Windows - Azure Virtual Machines
Content Source: articles/virtual-machines/extensions/key-vault-windows.md
Service: virtual-machines
Sub-service: extensions
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

https://github.com/MicrosoftDocs/azure-docs/issues/41837

@msmbaldwin
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

emailed author

contacted author by email and Teams

Hello @rdeveen and thank you for catching this. We will integrate your proposed changes into the documentation. We appreciate your contribution. #sign-off

[like]  Robert de Veen reacted to your message:

From: Courtney Wales @.>
Sent: Thursday, February 15, 2024 3:45:55 PM
To: MicrosoftDocs/azure-docs @.>
Cc: R. de Veen @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] pollingIntervalInS; where S means String (PR #115966)
Merged #115966https://github.com/MicrosoftDocs/azure-docs/pull/115966 into main.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/115966#event-11817373044, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLXLDABSVE6CFKJBDSGPH3YTYUTHAVCNFSM6AAAAAA57DA6T2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRHAYTOMZXGMYDINA.
You are receiving this because you were mentioned.Message ID: @.***>


production files use tailwind requirement which are 3mb

Hi! Yes, nova-tailwind is pretty big and as I try to reduce duplicates, I hope to get it to be smaller. However, it does provide an extension to Nova's default functionality. It's a must-have to keep the unified styling provided by the default Nova installation.
GZipped, the nova-tailwind is 285kb, about 10x smaller.

first of all huge fan of all your packages and your work 👍 .
i removed tailwind from package and it still worked, i havent noticed any differences in presentation , i dont remember if i added custom stylings to my shop but as far as i remember i didnt.
the second thing i wanted to say , i used purge css to remove 99% of tailwind and only remain with 10kb file so maybe you can try also.

the purge i used on other project that is

@Tarpsvo I think this package should not depend on nova-tailwind. It is better to use purgecss and include only styles it needs.


Could you add info of test settings of the networks, whether single view or multi view predictions?

Are given test values relying on single view predictions or multi-view values?

All score is based on single view prediction (center crop)
On Fri, Dec 4, 2015 at 10:43 AM, Eren Golge<EMAIL_ADDRESS>wrote:

Are given test values relying on single view predictions or multi-view
values?
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet-model-gallery/issues/5.


Video files being downloaded multiple times

I am seeing blogs redownload videos each time I run the program. This is causing lots of extra bandwidth to be taken up. I don't know if it is every video, or just reposts that point to the same file, but it's definitely happening. The exact same file names as the previous videos are being downloaded. I can generally cut all the new ones into the folder with the old and 95% of them are overwrites.

Thanks for sharing. I'll look into that at sometime this week.

Looks like you fixed this too. Thanks!


volumes_from is deprecated in the latest docker-compose: what to do?

Hi!
Trying to run compose with docker-gen - got the error: volumes_from is deprecated

have solved - add to docker gen the volume with nginx.conf


Tracks ip and user agent

This rehashes my previous PR.  We had to revert and rework some of the business decisions around this feature.  We're in a good position now.
For identify, alias, group, page, and track calls, users can track ip and user agent by supplying some information in the context object.  The required information for these calls is that an ip or user agent be present, and also a corresponding Kissmetrics flag is set to true, e.g.
{
  ip: '<IP_ADDRESS>',
  userAgent: 'Mozilla/5.0',
  Kissmetrics: {
    shouldTrackIp: true,
    shouldTrackUserAgent: true
  }
}

Furthermore, for track calls, ip and user agent will be tracked if the event is visited site.

Please squash the commits into one.

This is our master branch of our fork.  I'll reopen this PR with a branch instead.

Re-opened here: https://github.com/segmentio/integration-kissmetrics/pull/19


critest against containerd under selinux fails 'runtime should support HostIpc is true'

What happened:
With SELinux=Enforcing and updated container-selinux policy, critest is failing 'Security Context / NamespaceOption / runtime should support HostIpc is true'. I believe this is because of the fact that the container run for this assertion is not privileged (process label container_t instead of spc_t) which is needed to read host shared memory via ipcs -m.
See:

https://github.com/containers/container-selinux/releases/tag/v2.143.0
https://github.com/containerd/containerd/issues/4460

What you expected to happen:
containerd should pass critest with SELinux enforcing.
How to reproduce it (as minimally and precisely as possible):

checkout https://github.com/containerd/containerd/pull/4451
export INSTALL_PACKAGES=https://github.com/dweomer/container-selinux/releases/download/v2.143.0/container-selinux-2.143.0-2.dev.gite2d5a9e.fc32.noarch.rpm
vagrant up --provision-with=shell,selinux,test-cri
witness the single failure:

    default: Summarizing 1 Failure:
    default:
    default: [Fail] [k8s.io] Security Context
    default: NamespaceOption [It] runtime should support HostIpc is true
    default: /root/go/src/github.com/kubernetes-sigs/cri-tools/pkg/validate/security_context_linux.go:159
    default:
    default: Ran 90 of 97 Specs in 68.387 seconds
    default: FAIL! -- 89 Passed | 1 Failed | 0 Pending | 7 Skipped

vagrant ssh  -c 'sudo ausearch -m avc -ts recent -c ipcs'

----
time->Tue Aug 11 08:49:09 2020
type=AVC msg=audit(1597135749.972:7071): avc:  denied  { unix_read } for  pid=64429 comm="ipcs" key=-222557646  scontext=system_u:system_r:container_t:s0:c453,c1010 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=shm permissive=0
----
time->Tue Aug 11 08:49:09 2020
type=AVC msg=audit(1597135749.972:7072): avc:  denied  { unix_read } for  pid=64429 comm="ipcs" key=-227491983  scontext=system_u:system_r:container_t:s0:c453,c1010 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=shm permissive=0
----
time->Tue Aug 11 08:49:09 2020
type=AVC msg=audit(1597135749.972:7073): avc:  denied  { unix_read } for  pid=64429 comm="ipcs" key=1625385622  scontext=system_u:system_r:container_t:s0:c453,c1010 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=shm permissive=0
----
time->Tue Aug 11 08:49:09 2020
type=AVC msg=audit(1597135749.972:7074): avc:  denied  { unix_read } for  pid=64429 comm="ipcs" key=-528880287  scontext=system_u:system_r:container_t:s0:c453,c1010 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=shm permissive=0
----
time->Tue Aug 11 08:49:09 2020
type=AVC msg=audit(1597135749.972:7075): avc:  denied  { unix_read } for  pid=64429 comm="ipcs" key=1981708331  scontext=system_u:system_r:container_t:s0:c453,c1010 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=shm permissive=0
----
time->Tue Aug 11 08:49:09 2020
type=AVC msg=audit(1597135749.972:7076): avc:  denied  { unix_read } for  pid=64429 comm="ipcs" key=-1466713841  scontext=system_u:system_r:container_t:s0:c453,c1010 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=shm permissive=0
----
time->Tue Aug 11 08:49:09 2020
type=AVC msg=audit(1597135749.972:7077): avc:  denied  { unix_read } for  pid=64429 comm="ipcs" key=-2056816126  scontext=system_u:system_r:container_t:s0:c453,c1010 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=shm permissive=0
----
time->Tue Aug 11 08:49:09 2020
type=AVC msg=audit(1597135749.972:7078): avc:  denied  { unix_read } for  pid=64429 comm="ipcs" key=-63844995  scontext=system_u:system_r:container_t:s0:c453,c1010 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=shm permissive=0

Anything else we need to know?:
I am happy to work on a fix but wanted to submit the issue to elicit discussion beforehand.
Environment:

Container runtime or hardware configuration:
containerd 1.4.x (https://github.com/containerd/containerd/pull/4451)

OS (e.g: cat /etc/os-release):
Fedora 32

Kernel (e.g. uname -a):
5.6.6-300.fc32.x86_64

Others:
container-selinux from https://github.com/dweomer/container-selinux/releases/download/v2.143.0/container-selinux-2.143.0-2.dev.gite2d5a9e.fc32.noarch.rpm

/cc @mikebrow @crosbymichael

In a nut.. the idea is container runtimes should nil the processLabel and mountLabel returned from selinux returned InitLabels when the pod-sandbox linux security policy is privileged, hostIPC, hostPID (what about hostNetwork?)....  Seems if that was supposed to be common behavior, we should document it (here)[https://kubernetes.io/docs/concepts/policy/pod-security-policy/#host-namespaces], and in the above InitLabels() call.

I surmise (from the crio code) container runtimes should nil (not use) the processLabel and mountLabel returned from selinux-InitLabels when the pod-sandbox linux security policy is privileged, or hostIPC, or hostPID (what about hostNetwork?).... Seems if that was supposed to be common behavior, we should document it (here)[https://kubernetes.io/docs/concepts/policy/pod-security-policy/#host-namespaces], and in the above InitLabels() call.


mm: mm_heap: Remove critical section in mm_sem.c

Summary

This commit removes critical section in mm_sem.c which was
added to stabilize the NuttX SMP kernel in Mar 2018.

Impact

SMP only

Testing

Tested with ostest with the following configs

maix-bit:smp (QEMU), esp32-devkitc:smp (QEMU)
sabre-6quad:smp (QEMU), spresense:smp, sim:smp

Tested with nxplayer with the following configs

spresense:wifi_smp, spresense:rndis_smp

I noticed that nxplayer with spresense:rndis_smp stopped after 5.3hrs.
Let me check if the issue relates to this PR.

I noticed that nxplayer with spresense:rndis_smp stopped after 5.3hrs.
Let me check if the issue relates to this PR.

Sure.

I noticed that nxplayer with spresense:rndis_smp stopped after 5.3hrs.
Let me check if the issue relates to this PR.

Finally, I found the root cause of the issue.
I will test this PR again once #3254 is merged.

@xiaoxiang781216
Please merge this PR.


[import/order] "import type" always should be recognized as 'type' group

Problem
import type {
  RouteName,
} from '~/enums';

should be recognized as type.
If import type then don't need to use pathGroups. Becuase sometimes I can use RouteName as value, sometomes only as type. Then import type keywords must take precedence on group recognition.
Full rules
'import/order': ['error', {
      'newlines-between': 'always',
      'groups': [
        'type',
        'external',
        'builtin',
        'parent',
        'object',
        'sibling',
      ],
      'pathGroups': [
        {
          'pattern': '~/types/**',
          'group': 'type',
        },
        {
          'pattern': '**/types',
          'group': 'type',
        },
        {
          'pattern': '~/**',
          'group': 'builtin',
        },
      ],
    }],

Please check in the newly released v2.23.3 (#2092).

@ljharb not helped

The following test passes successfully in the plugin repo using the config and code you provided plus the option I mentioned above:
test(
  {
    code: `
      import type { ViewStyle, TextStyle } from 'react-native';
      import type { Color } from '~/enums';
    `,
    parser,
    options: [
      {
        'newlines-between': 'always',
        'groups': [
          'type',
          'external',
          'builtin',
          'parent',
          'object',
          'sibling',
        ],
        'pathGroups': [
          {
            'pattern': '~/types/**',
            'group': 'type',
          },
          {
            'pattern': '**/types',
            'group': 'type',
          },
          {
            'pattern': '~/**',
            'group': 'builtin',
          },
        ],
        'pathGroupsExcludedImportTypes': ['type'],
      },
    ],
  },
  parserConfig,
),

I'm not sure why that not applied at first time (tried twice). It realy working nice now. Thank you


background removal implemented within the flow_utils and compute_flow

it looks suspect to me... but maybe it works better for other videos... im too tired right now to test it more.

it could be replaced with BackgroundSubtractorMOG or BackgroundSubtractorGMG instead of BackgroundSubtractorMOG2, maybe they will be better, i dunno

Thanks a lot! I'm gonna try it today and merge if everything works fine.


Adds functional tags for game size

Description

Resolves #89

Checklist

[x] Add tests for these changes
[ ] Test coverage is not decreased
[ ] Entire test suite passes

Codecov Report

Merging #91 into master will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #91      +/-   ##
==========================================
+ Coverage   96.96%   97.08%   +0.11%
==========================================
  Files          21       21
  Lines        1221     1236      +15
  Branches      191      195       +4
==========================================
+ Hits         1184     1200      +16
  Misses         21       21
+ Partials       16       15       -1

Impacted Files
Coverage Δ

src/spellbot/__init__.py
95.31% <100.00%> (+0.22%)
:arrow_up:

Continue to review full report at Codecov.

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


Aidl not found, please install it.  E: Unable to locate package aidl

I am trying to run buildozer. While running it gave the following error:
Aidl not found, please install it.
And I use sudo apt-get install aidl
E: Unable to locate package aidl
Any one could help me? Thanks very much.

Ok,thanks~


Fixed issues

Chenged class to "col-lg-10 col-lg-offset-1" in div container content

Great! Are the issues tracked here?
If yes let me know the ids so we can automatically resolve them when the merge is commited :wink:
I will checkout the PL now :smile:

@joariasl do we need the same changes in the post and tag layouts? :smile:


[Question]: How to run HuggingFaceEmbedding on multiple available GPUs?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I'm trying to load an embedding model from HuggingFace on multiple available GPUs using this code:
embed_model = HuggingFaceEmbedding(self._base_model, parallel_process = True, target_devices=["cuda:0", "cuda:1", "cuda:2","cuda:3"])
However, it still uses the first GPU and throws cuda out of memory error. Could someone help me with that?

Any updates on this? I am facing the same issue.


Add Coveralls, Tox

Packages updated, some fixes related to that as well.

Changes Unknown when pulling 4550a7b5b1c6ec5edd82dd02b2954fe7ebf4af93 on makyo:coveralls into ** on OpenFurry:master**.


Login Flow: API - Get Current User Endpoint

Goal: Implement an API endpoint to retrieve the current authenticated user's details.
Background: Currently, we do not have a dedicated endpoint for fetching the details of the currently authenticated user. This endpoint is important for allowing the client to fetch user details after login or page refresh, etc.
Tasks:

Design and implement a new API endpoint
The endpoint should return the currently authenticated user's details.
Ensure proper authentication checks are in place - only the logged-in user should be able to access their own details.
Implement error handling to cover cases like unauthenticated access, server errors, etc.
Write tests for the new endpoint

This task is required for the completion of user story - TrueSparrowSystems/AI-SalesBridge-iOS-App#3


Fix/test router reboot

Short version:
[root@cs1 smoke]# tail -f results.txt
Test for reboot router ... === TestName: test_reboot_router | Status : SUCCESS ===ok

----------------------------------------------------------------------
Ran 1 test in 440.146s

OK

Long version:
[root@cs1 smoke]# nosetests --with-marvin --marvin-config /data/shared/marvin/mct-zone1-kvm1-kvm2.cfg  -s -a tags=advanced,required_hardware=true test_network.py:TestRebootRouter
[INFO] 2016-05-30 16:16:26,985 - marvin - === Marvin Init Logging Successful ===
[INFO] 2016-05-30 16:16:26,985 - marvin - === Marvin Init Started ===
[INFO] 2016-05-30 16:16:26,988 - marvin - === Marvin Parse Config Successful ===
[INFO] 2016-05-30 16:16:26,988 - marvin - === Marvin Setting TestData Successful ===
[INFO] 2016-05-30 16:16:26,999 - marvin - Parsing Test data successful
[INFO] 2016-05-30 16:16:27,015 - marvin - === Test Client Creation Successful ===
[INFO] 2016-05-30 16:16:27,015 - marvin - === Marvin Init Successful ===
[INFO] 2016-05-30 16:16:27,114 - test_reboot_router (integration.smoke.test_network.TestRebootRouter) - === Started Test test_reboot_router ===
SSH into VM (ID : 0c79ea48-cd5a-4221-8e5f-7f59b0c7e4b3 ) before reboot of router
[INFO] 2016-05-30 16:20:08,504 - ssh - Trying SSH Connection to host <IP_ADDRESS> on port 22 as user root. RetryCount: 20
[ERROR] 2016-05-30 16:20:10,868 - ssh - Failed to create connection: [Errno None] Unable to connect to port 22 on <IP_ADDRESS>
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/marvin/sshClient.py", line 99, in createConnection
    timeout=self.timeout)
  File "/usr/lib/python2.7/site-packages/paramiko/client.py", line 324, in connect
    raise NoValidConnectionsError(errors)
NoValidConnectionsError: [Errno None] Unable to connect to port 22 on <IP_ADDRESS>
[INFO] 2016-05-30 16:20:20,879 - ssh - Trying SSH Connection to host <IP_ADDRESS> on port 22 as user root. RetryCount: 19
[ERROR] 2016-05-30 16:20:22,884 - ssh - Failed to create connection: [Errno None] Unable to connect to port 22 on <IP_ADDRESS>
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/marvin/sshClient.py", line 99, in createConnection
    timeout=self.timeout)
  File "/usr/lib/python2.7/site-packages/paramiko/client.py", line 324, in connect
    raise NoValidConnectionsError(errors)
NoValidConnectionsError: [Errno None] Unable to connect to port 22 on <IP_ADDRESS>
[INFO] 2016-05-30 16:20:32,895 - ssh - Trying SSH Connection to host <IP_ADDRESS> on port 22 as user root. RetryCount: 18
[ERROR] 2016-05-30 16:20:34,900 - ssh - Failed to create connection: [Errno None] Unable to connect to port 22 on <IP_ADDRESS>
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/marvin/sshClient.py", line 99, in createConnection
    timeout=self.timeout)
  File "/usr/lib/python2.7/site-packages/paramiko/client.py", line 324, in connect
    raise NoValidConnectionsError(errors)
NoValidConnectionsError: [Errno None] Unable to connect to port 22 on <IP_ADDRESS>
[INFO] 2016-05-30 16:20:44,910 - ssh - Trying SSH Connection to host <IP_ADDRESS> on port 22 as user root. RetryCount: 17
[INFO] 2016-05-30 16:20:45,557 - ssh - Connection to host <IP_ADDRESS> on port 22 is SUCCESSFUL
Public IP: <IP_ADDRESS>
Public IP: <IP_ADDRESS>
Rebooting the router (ID: 96b5dd2f-bb12-4315-9586-41d1c0c2f4c2)
VM state: Running
SSH into VM (ID : 0c79ea48-cd5a-4221-8e5f-7f59b0c7e4b3 ) after reboot
[INFO] 2016-05-30 16:22:56,224 - ssh - Trying SSH Connection to host <IP_ADDRESS> on port 22 as user root. RetryCount: 20
[INFO] 2016-05-30 16:22:56,785 - ssh - Connection to host <IP_ADDRESS> on port 22 is SUCCESSFUL
[INFO] 2016-05-30 16:23:47,259 - test_reboot_router (integration.smoke.test_network.TestRebootRouter) - === TestName: test_reboot_router | Status : SUCCESS ===
[INFO] 2016-05-30 16:23:47,259 - test_reboot_router (integration.smoke.test_network.TestRebootRouter) - TestCaseName: test_reboot_router; Time Taken: 440 Seconds; StartTime: Mon May 30 16:16:27 2016; EndTime: Mon May 30 16:23:47 2016; Result: SUCCESS
[INFO] 2016-05-30 16:23:47,260 - test_reboot_router (integration.smoke.test_network.TestRebootRouter) - === finalize does nothing!  ===

Ping @borisroman @wilderrodrigues


Annotations showing on multiple pages

I'm adding annotations to reveal.js presentations . The code im using is based on the example as well as the two other issues posted.
The problem is that an annotation from one presentation still loads on another.
jQuery('Lecture 1').text(); is changed to  jQuery('Lecture 2').text(); in the other reveal presententaion.
	<script>
    var content = jQuery(".reveal").annotator();
    content.annotator('addPlugin', 'Offline', {
	  setAnnotationData: function (ann) {
    if (!ann.pageName) {
      ann.pageName = jQuery('Lecture 1').text(); // Use the page title
    }
  },
  shouldLoadAnnotation: function (ann) {
    return ann.pageName === jQuery('Lecture 1').text();
  },
      online:  function () {
        jQuery("#status").text("Online");
      },
      offline: function () {
        jQuery("#status").text("Offline");
      }
    });
    var annotator = content.data('annotator');

    jQuery("#clear-storage").click(function () {
      if (annotator) {
        annotator.plugins.Offline.store.clear()
      }
    });
	</script>

I'm not sure what you want jQuery to be doing here. You need to pass it a CSS selector not a piece of text. If you want to get the title of the page you can use jQuery('title').text() otherwise you can just use the string "Lecture 1" as you have it without the jQuery part.
Please feel free to re-open this if you're still having issues.


Can't get address by connecting and disconnecting to device

Expected behavior:
When finding a new device (that will lack the address) I can connect and then disconnect. Then the device will show up in the scan and it will contain the address.
Actual behavior:
Doesn't matter if I disconnect the device still has no address when found again. Has to restart the whole script in order for it to find the address.
Long version:
I have a script that finds a device based on name. If the device didn't have the address I only had to connect to the device and then disconnect and the next time the device was found in the scan the address was available. But now it doesn't matter if I disconnect and then find the device again, it will still not find the address.

Hi, I think this is a problem with noble, as you can see here the address is only updated if the peripheral is not yet discovered.
You say that it worked before:

... If the device didn't have the address I only had to connect to the device and then disconnect and the next time the device was found in the scan the address was available. But now it doesn't matter if I disconnect and then find the device again, it will still not find the address.

Which noble/noble-mac version where you using?

Interesting, I'm using 0.0.2

And where you using noble before (when it worked) or was that another module?

@vikeri I'll close this, as it is most likely the problem of how noble handles the device address. If you feel this is still an issue with noble-mac please reopen this or create a new issue, thanks!


docs(behavior_path_planner): update avoidance module's documentation

Signed-off-by: Muhammad Zulfaqar Azmi<EMAIL_ADDRESS>Description
This PR aims to update avoidance_module's parameter list and some of the contents.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[X] I've confirmed the contribution guidelines.
[X] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@zulfaqar-azmi-t4 Is this still a draft?

@TakaHoribe
Sorry, I didn't realize it is in draft.
It is ready.


Joins for Subject Map

Add support for joins for Subject Maps as well, useful to construct IRIs from multiple tables or CSVs, especially for GTFS files.
Currently, joins are only possible in the Predicate Object Maps.

@DylanVanAssche in some cases, joining on the subject map could be facilitated or supported by the ability to apply functions on rr:template (e.g., using fnml:functionValue and so on; see https://github.com/elsdvlee/rml-jc/blob/patch-1/ProposalsAndIssues.md#lionel-static-rrtemplates for more context). Does this fall under the need to extend the RML specification, or is it solely related to the implementation of mapping engines? More broadly, is this feasible?

@DylanVanAssche rml:logicalView will allow the combination of data from different sources in one template. Can this issue be closed?


Update dockerSetup.sh

Replace old flag to change docker data dir in service file setup.

Already fixed :-)


[voice/video bug] voice is acting strangely, doesn't always establish a connection properly

Tested and reproduced on Quarrel Insider 20.6.25, including on my fork of the repository.

As far as I can tell this is fixed as of 20.7.5. Will close, but remind me to reopen this if it starts getting reported again.

This has not been fixed

Problem still happens to me!!
First 15-30 seconds of communication ok, but then communication is lost.

Problem still happens to me!!
First 15-30 seconds of communication ok, but then communication is lost.

Please don't spam here, we're aware.

Hello. Do we have any news about this issue?

hello there! From my recent testing, optimization appears to have made this less common, but there are still alot of reports that come in about this. it'll be closed if and when a proper fix is implemented.

Yesterday I tried to use the application twice on XBOX, but unfortunately in less than 5 minutes I lost the audio.
Is there anything I can do, any configuration, to improve?

try using the latest build of Quarrel Insider

try using the latest build of Quarrel Insider

Sorry, but how can I do this on XBOX?


support scoped variable via proper compile pass

Scoped-variable is now implemented via a proper compile pass. It is enabled by default. But it is not stable enough, you can toggle it off in the sidebar.
The following visual appearance is added to the editor:

function definition
(global) variable definition
function call site
variable use
unbound variables

Screenshot:

UPDATE: hide annotations by default, and add a "show annotations" setting to toggle it.