[Bug] Hotfix changelogs exclude PRs to dev

Bug description
The "Wasm Create Hotfix PR" GitHub Action fails to include changelog entries on a support/* branch when those PRs were made to the dev branch.
Cause
The cause of this bug is due to passing the --release-branch ${{ inputs.branch }} argument to github_changelog_generator:

--release-branch RELEASE-BRANCH
Limit pull requests to the release branch, such as master or release.

However, the argument was added a while ago to fix a different issue (unclear what exactly that was or if we still suffer from it): https://github.com/iotaledger/identity.rs/pull/560
The current workaround is to remove that argument in the workflow, but only in the hotfix/support branch, and to run the workflow from the support/wasm-v0.5 branch rather than dev.
Example
For the support/wasm-v0.5 branch, the dev branch commits were pushed directly to it rather than re-doing the PRs targeting the support/wasm-v0.5 branch as a base. This was done because no breaking changes were made and the PRs from those commits were intended for 0.5.1.
However, when the "Wasm Create Hotfix PR" action was executed, it failed with an error that the changelog file was unchanged. This is because it excluded the PRs to dev, even though they were made after the most recent tag, wasm-v0.5.0.
Specifically, the following PRs were not included (the rest have the No changelog label):

https://github.com/iotaledger/identity.rs/pull/789
https://github.com/iotaledger/identity.rs/pull/794

Steps To reproduce the bug
Explain how the maintainer can reproduce the bug.

Push a PR to dev with changelog labels, e.g. Wasm, Patch.
Create a hotfix branch from dev, e.g. support/wasm-v0.5.
Run the "Wasm Create Hotfix PR" workflow.

Expected behaviour
"Wasm Create Hotfix PR" workflow executes without errors, including the PR from dev in the new changelog.
Actual behaviour
An error is thrown by the workflow that the changelog is unchanged, because the PR from dev is excluded.
Errors
Paste any errors that you see, including logs, errors, or screenshots.
https://github.com/iotaledger/identity.rs/runs/5855428737?check_suite_focus=true

This really isn't ideal but I think that is unsolvable with the github-changelog-generator library. The problem being that release-branch controls which target branch is to be considered for changes to be included. If hotfix workflows would not limit release-branch to themselves they will include all changes on dev up to the point the job is run, even if they aren't merged on the hotfix branch.
The problem in the case of https://github.com/iotaledger/identity.rs/tree/support/wasm-v0.5 was that the PR was merged to dev as if it was a regular feature, it will therefore be included in the next regular changelog. To make it work with support branches another PR must be created for the support branch or vice-versa, to then be included in the hotfix changelog since then a PR targeting the support branch can be detected.
Long story short: removing release-branch will include merges to dev not on the support branch. To make it work PRs must be created against support branches, sometimes requiring the same code as a PR against dev and potentially multiple support branches. A support branch must therefore be created from the commit with the associated tag not a later commit.


40 - Add project with Infinite Scroll example.

New project to show a basic example for infinite scroll.

@alexventuraio Please resolve the conflicts.

@alexventuraio Please resolve the conflicts.

@swapnilsparsh addressed!

@alexventuraio Please resolve the conflicts.😅

@alexventuraio Please resolve the conflicts.😅

@swapnilsparsh one more, up-to-date! 👍

@alexventuraio Please resolve the conflicts.sweat_smile

@swapnilsparsh one more, up-to-date! +1

I merged your PR #34 and then now conflicts has to be resolved kn this PR 🙃

@alexventuraio Please resolve the conflicts.sweat_smile

@swapnilsparsh one more, up-to-date! +1

I merged your PR #34 and then now conflicts has to be resolved kn this PR 🙃

@swapnilsparsh  No worries, I'll be here until we get this done!

@alexventuraio Please resolve the conflicts.sweat_smile

@swapnilsparsh one more, up-to-date! +1

I merged your PR #34 and then now conflicts has to be resolved kn this PR 🙃

@swapnilsparsh No worries, I'll be here until we get this done!

@swapnilsparsh  Ready for you to review! 😎

@alexventuraio Thank you for your contribution. Looking forward to more such amazing contributions 🌟


[Bug]: Current jsonld files do not store jsonld graphs

batch name and line
No response
What happened?
this is taking one of the examples to jsonld playground.
https://tinyurl.com/ynhdr9ek
you'll see all forms of it are empty. i haven't tracked the context changes, but likely the culprit. we should also add tests for jsonld expansion and compactness in the github workflow.
Describe the expected ground-truth ?
to be able to visualize the graph
What soft is concerned
No response

Hi @satra ! Thanks for spotting this! You are right: this example is outdated and do not match the current context.
Am I right this is the one you looked at: https://github.com/bids-standard/BEP028_BIDSprov/tree/master/examples/spm_default ?
All examples created by the parsers work fine, see for example spm_default on JSON-LD playground that is a copy of https://github.com/bids-standard/BEP028_BIDSprov/blob/master/examples/from_parsers/spm/spm_default_batch.jsonld.
I'll into this and will likely remove this outdated example as we now have many others generated by the parsers.


[API] add comments endpoint for single comment

close #9486
usefull at #4811 (#9488)
add GET /repos/{owner}/{repo}/issues/comments/{id}
GitHub reference spec: link

Codecov Report

Merging #9494 into master will increase coverage by <.01%.
The diff coverage is 59.03%.

@@            Coverage Diff             @@
##           master    #9494      +/-   ##
==========================================
+ Coverage   41.53%   41.54%   +<.01%
==========================================
  Files         572      572
  Lines       74757    74840      +83
==========================================
+ Hits        31053    31093      +40
- Misses      39852    39892      +40
- Partials     3852     3855       +3

Impacted Files
Coverage Δ

models/issue_comment.go
42.68% <0%> (-0.91%)
:arrow_down:

routers/api/v1/api.go
73.84% <100%> (+0.03%)
:arrow_up:

routers/api/v1/repo/issue_comment.go
56.02% <68.57%> (+2.48%)
:arrow_up:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

services/pull/temp_repo.go
31.62% <0%> (-2.57%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

models/repo_list.go
73.14% <0%> (-0.93%)
:arrow_down:

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 3e166bd...1bfe4ed. Read the comment docs.

@zeripath I thought comment.APIURL() is a genaral function wich #9488 will use - but i dont know when this get ready ... so I thought add this smal func to this one ... but yes its unrelated - removed

@techknowlogick @guillep2k done

ready for merge :rocket:


How to establish a bond under dhcpcd?

How to establish a bond under dhcpcd?  Love the idea of the failback profile logic, but when I setup dhcpcd for dhcp with static failback profile, and then create a bond under networking itnerfaces, I trip over the start up sequence order, such that no interfaces actually exist on reboot.    So clearly something is stepping on something during system start up.  Any insight or suggestions?  Is there a way to create a bond strictly under dhcpcd control?  From a configuration perspective?

Yes and no, I hoped there was a more native or supported way to do such via dhcpcd.  But it does such one way to address the issue.  Since my goal is to get away from having to use older /etc/network/interfaces methodology, I was thinking more towards dhcpcd... since I like the idea of a fallback method.  I guess I could also go the systemd route, eh, no pun intended.

dhcpcd's job is just to configure addressing and routing on a given interface.
Setting the interface up is the job of something else.

True, but there are many scenarios outlined via google where traditional bonding does not work well with dhcpcd.  I have experienced this corundum myself, where the typical load sequence at boot trips up between dhcpcd and the networking service.  So my idea was to figure out a way to try to address this, I happen to love the idea of the static failback feature of dhcpcd.  So, I first tried using dhcpcd with a typical bond configuration in /etc/network/interfaces configuration file.  The method worked if I ran the command sequence manually but on boot, the bond was never established.  So the idea occurred to me, that maybe not using a typical bonding configuration could be eliminated, hence my question above.  Using the exit idea is the next thing to try.

I fail to see how dhcpcd can trip up with bonding - you start it with no interfaces specified or with the -m flag and dhcpcd will react to interfaces arriving and departing. A bond interface by itself is nothing special.
Now there might be some extra delay when dhcpcd spots the bond interface and starts up on it, but if the OS is half decent it will report no carrier on it until an interface joins it with a carrier and thus there is no delay. But this is no different from a bridge interface from dhcpcd's perspective.

There seems to be some odd interaction.  A race condition or sequence issue with how the two services interact.  For example, if I run dhcpcd -k then run systemctl restart networking, by bond comes up and the bond0 interface has an IP via dhcp, because I have denied the slave interfaces and only allowed the bond interface.  All good.
However, if I reboot the same device, with no changes, at all, the bond interface fails, and thus no dhcp results.  If I then run dhcpcd -k and then systemctl restart networking on the just rebooted device, the bond interface works, dhcp assignment happens.
I can reproduce this on buster and bullseye, I happen to be testing on a Raspberry Pi device, but I have created the same scenario on Intel/AMD based hardware, so this is not a ARM based hardware quirk IMHO.

#76 has been merged into master and may help here if DHCPv6 or IPv6 was in play.

When I am working on a bond configuration, typically I leave it enabled as default so it starts with system start, i.e. the unit file under systemd.  But when I am explicitly testing a bond configuration I will at times stop via -k and then let it restart when I do systemctl restart networking.  This manual restart of networking is not 'that' clean, so usually I just do a graceful restart of the system to be sure the mimic a typical system start/boot.
The original goal was to let dhcpcd work with a DHCP server, and configured with a static fall back, so the bond0 interface is always a know ip assignment, in the DHCP server, using a reservation of course.  But some distros of Linux don't always handle this well, they sometimes expect the entire bond0 setup to be in the interfaces file, regardless of a static or dhcp ip assignment elsewhere.
There does not seem to be any explicit denial that dhcpcd can't be used with a bond configuration, so that is why I was testing and trying to figure out if it is possible... it should be because a bond is just another interface from dhcpcd perspective.  If I set the 'bonded' interfaces to denyinterface in dhcpcd.conf, and only allow dhcpcd to handle bond0, this works in some cases, on some distros.
When I opend the question, I was thinking that maybe dhcpcd could own the bond configuration, feature enhancement maybe, and this would completely retire the need for using the interfaces file.  But I realized that dhcpcd is really just a dhcp client, so it should accept whatever interface if finds available, i.e. bond0.  But that does not explain why when using bonds, and the older interfaces file, and explicitly disabling dhcpcd, everything seems to work. but when using dhcpcd, the bond fails to come up successfully at times, even if configured to use dhcp configuration.  Feels like something is stepping on something or something is not handing off control (or back to something) in the network stack.
Most of my testing has been around or on Debian 10, 11 x64 and Pi OS 10 and 11 (i.e. Debian 10, 11 ARM), just for reference.


french translation - javafx-8-tutorial-fr-part1.md

Hello!
Thank you so muck for this great tutorial.
I believe that we don't have a lot of french tutorials about JavaFX and i am happy i found it!
I'm trying to help a bit translating a small part.
I don't know if i have the time for the rest. I will try.
Perhaps can you delete this branch after merging if you accept my changes.
I can always create a new one (one a part).
All the best!
David

Hi @Davidmue. That is awesome, thank you! I will integrate it tonight.

Very nicely done.
I've also added you to the list of contributors at the bottom of the first part of the French tutorial.


⚠ Cioos Pacific ERDDAP has degraded performance

In 9cff951, Cioos Pacific ERDDAP (https://data.cioospacific.ca/erddap/index.html) experienced degraded performance:

HTTP code: 200
Response time: 9678 ms

Resolved: Cioos Pacific ERDDAP performance has improved in c3ac320 after 5 minutes.


Context access might be invalid: ACT

Describe the bug
I believe this is a quality of life issue rather than a bug.
It would be nice for there to be a documented way, perhaps a quick actions, to ignore certain warnings.
The issue is that if: ${{ !env.ACT }} warns that ACT might be invalid. Which it will be when not running in an environment that defines ACT. Not sure if warning that an environment variable might not be declared is useful.
To Reproduce
Steps to reproduce the behavior:

Create any workflow block that contains an if statement referencing an environment variable that isn't declared in an env block.

Expected behavior
Either no warning, or a way to set particular environment variables to be ignored in this context.
Screenshots

Extension Version
v0.25.3

After further searching, appears to be a duplicate of both #61 and #47

I'm also experiencing this issue. It makes no sense to me that this warning is being issued. Either the extension can validate all context or if it cannot do some then don't do any. Given that variables and secrets are viewable on the left I would imagine  if can validate.
Not sure if this is the cause but the workflow doesn't show in the left pane either.

Fixed in next release: https://github.com/github/vscode-github-actions/issues/61#issuecomment-1498265231

@tylerclark:

Fixed in next release: #61 (comment)

I can't see a commit which fixes this - even an unreleased one. Where did you see it was fixed?

Still seeing this issue FYI

I'm also still seeing this!

definitely not fixed in the latest version.

I would love to have no yellow underline on my workflow files :)

i'm almost disabling the extension... problem showing in v.0.25.8 here too :(

I am disabling the extension for now. i'll check back in about 5 months 😞

@felipesu19 @cschleiden
Ya'll are posting in a closed issue, no one is going to see this. Pinging for visibility.

Have you tried declaring ACT as an environment variable? Something like
name: Demo workflow

env:
  ACT:

on:
  push:
  workflow_dispatch:
    inputs:
      name:
        type: string

jobs:
  say-hello:
    runs-on: ubuntu-latest

    steps:
      - run: echo "Hello ${{ env.ACT }}"

I uninstalled and reinstalled and then the warnings went away for me.

Have you tried declaring ACT as an environment variable? Something like

That does not fix the issue.  The "secrets" object is a simple dictionary of name / value pairs.  Is it possible to add a config section for what we expect the object to look like ?

No. The issue is being able to add custom values.
It's good that it already has the values from the documentation.

So unless we only use the prepopulated attributes on secrets or env, our scripts are filled with error warnings. The purpose of these objects is to extend them.
A way to add known values to the shape of these objects, like adding words to a spell checker, would resolve this issue.

It uses the actual secrets configured for the repository, not just from the documentation.

Possible issue with that feature.  What's the best way for me to help ?  How can I generate a reproduction ?

One reproduction would be following the steps from https://app.codecov.io/
They have you create a:
secrets.CODECOV_TOKEN

Then when you go to edit that .yml file, it will always show this error where CODECOV_TOKEN is used.

Other plugins, like the spell checker, allow us to add words to the workspace settings to resolve similar issues.

I can confirm this works and is populated in github and that the plugin has no knowledge of the secrets configured for the repository.

It would be a very impressive feature to have GH aware context. Just the keys mind you, values would not be good.

@Zageron that is supported by the extension:

Nothing, just login and open the repository.

Since those Azure ones are well known and might be populated by the plugin staticly, could you create a secret named CODECOV_TOKEN and see if it appears in the list and allows it without warning ?
Thank you kindly.

@epreston I had to uninstall/reinstall the extension for the secret names autocomplete to start working. Also, fixed the yellow underlines.

Any possible fix?

@epreston I had to uninstall/reinstall the extension for the secret names autocomplete to start working. Also, fixed the yellow underlines.

This resolved the issue for me, can't believe I didn't try this first myself 😑

I think this issue need to be re-opened

Just installed the extension (0.26.2) and immediately stumbled over this, which is not helpful.

If I restart vscode, these warnings go away until I open a file under .github. But the instant I open one, all the warnings come back.
In my case the underlying cause seems to be a lack of alias support (#239).


Too many package dependencies

Hi there.
I just noticed when using your module that a lot of modules got pulled in.
In particular, the netifaces module is pulled in, which is a binary extension.  And others.
Browsing your code, I can't see that there are other direct dependencies than fastapi and python-socketio.  But the Pipfile includes a host of extensions, and so does setup.py.   (python-engineio is automatically added as part of python-socketio)
Not sure why the module needs the various "extras" in setup.py, since there is no extras code.

Hey, thanks for addressing the issue. I'll try tidy up the dependencies file in next couple of days 🙏🏻

Hey, thanks for addressing the issue. I'll try tidy up the dependencies file in next couple of days 🙏🏻


PinsMixin.pins returns None

Describe the bug
Any component which inherits PinsMixin will return None when calling .pins
There is a return missing from the pins property.
e.g.
class PinsMixin:
    """
    Mixin used by devices that use multiple pins
    """

    @property
    def pins(self):
        """
        Returns a tuple of pins used by the device
        """
        self._pin_nums

should be:
        return self._pin_nums

Fixed in dev

Release with v0.3.0


sonictt16b: Fix broken startup on ext4 filesystems.

Commit e811c41697678e37e2512067eb6d1add964bd9f8 breaks sonictt16b when installed on (case-sensitive) ext4 filesystems.
The problem is that the game tries to load language JSON files using titlecase (data/English.json, data/Spanish.json, etc.)¸ but those files are stored in the original .apk as lowercase (data/english.json, data/spanish.json, etc.).  This works fine when the files are stored in game.apk (as they were prior to e811c41697678e37e2512067eb6d1add964bd9f8) since the filename comparisons are case insensitive, but when they're stored bare in the gamedata directory, the loads fail and the game won't boot.
The easiest fix here is to put them back into game.apk, which the updated patchscript does.

Hello, thanks for reporting this. I have created my own simpler solution and will be submitting a pull for that, standby.


(RUBY) Could not find a JavaScript runtime. See https://github.com/rails/execjs for a list of available runtimes. (ExecJS::RuntimeUnavailable)

I hope to activate rails server, but it couldn't.... please help me ... i don;t know what i have to do;;;;;
/home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/execjs-2.6.0/lib/execjs/runtimes.rb:48:in autodetect': Could not find a JavaScript runtime. See https://github.com/rails/execjs for a list of available runtimes. (ExecJS::RuntimeUnavailable)         from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/execjs-2.6.0/lib/execjs.rb:5:inmodule:ExecJS'
from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/execjs-2.6.0/lib/execjs.rb:4:in <top (required)>'         from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/uglifier-2.7.2/lib/uglifier.rb:3:inrequire'
from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/uglifier-2.7.2/lib/uglifier.rb:3:in <top (required)>'         from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/bundler-1.10.6/lib/bundler/runtime.rb:76:inrequire'
from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/bundler-1.10.6/lib/bundler/runtime.rb:76:in block (2 levels) in require'         from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/bundler-1.10.6/lib/bundler/runtime.rb:72:ineach'
from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/bundler-1.10.6/lib/bundler/runtime.rb:72:in block in require'         from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/bundler-1.10.6/lib/bundler/runtime.rb:61:ineach'
from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/bundler-1.10.6/lib/bundler/runtime.rb:61:in require'         from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/bundler-1.10.6/lib/bundler.rb:134:inrequire'
from /home/ec2-user/config/application.rb:7:in <top (required)>'         from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/railties-4.0.1/lib/rails/commands.rb:74:inrequire'
from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/railties-4.0.1/lib/rails/commands.rb:74:in block in <top (required)>'         from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/railties-4.0.1/lib/rails/commands.rb:71:intap'
from /home/ec2-user/.rvm/gems/ruby-2.0.0-p643/gems/railties-4.0.1/lib/rails/commands.rb:71:in <top (required)>'         from bin/rails:4:inrequire'
from bin/rails:4:in `'

which node
And then

If you run into this issue, you may be tempted to add therubyracer to your gemfile. Prefer to use NodeJS instead, if your application runs in a multithreaded environment. Sidekiq itself considers therubyracer as a non thread-safe gem.


Unable to upgrade from the Settings screen to a beta version

Describe the bug
Unable to upgrade to beta
To Reproduce
Steps to reproduce the behavior:

Go to Settings
Click on 'Include beta' versions
Click on 'Check for updates'
Nothing happens - even though the github page says there are betas available. (currently 5.4.0-beta.4 is available)

Expected behavior
I should have been able to upgrade to the 5.4.0-beta.4 beta
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS Catalina
Browser Ferdi
Version 5.3.3

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

As far as I know Franz has the same problem. It sometimes takes a while or a restart for it to see beta updates.

I think it's because we have introduced the app signing/notarizing in 5.3.4-beta.3. The in-app updater doesn't work for unsigned app on macOS Catalina. The 5.3.3 being not signed/notarized, you will have to manually update to 5.4.3-beta.3 or greater for the in-app updater to work.
Please re-open if you think this is something else or if it did work since you opened the issue, which would mean the issue lays somewhere else.


Branch Motorways in Italy

Branch motorways in Italy are numbered based on the two motorways that they connect:

The shields on these roads use a stacked configuration as follows:

In the database, these route relations use the form ref=A7/A26.  These will need to be parsed and split by the slash in a vertical configuration.

Would '/' --> '\n' replacement work or are special characters not supported for text rendering?

Javascript canvas doesn't have multi line support, so that wouldn't work out of the box.

Stacked refs tracked in #366.


How to add all pages under a route parameter to the public pages list?

Using this repository as a template, I have an indefinite number of pages inside a route parameter (/src/routes/people/[personName]), and I need all those pages to be publicly viewable without needing to sign in. I'm curious if this is possible with publicPages in src/hooks.server.ts. Thanks so much for creating such a great project!

I found a workaround by adding another condition in which requestAllowed can be true:
function isPersonPage(path: string) {
  const pathSegments = path.split('/').filter(segment => !!segment); // Split and remove empty strings
  if(pathSegments.length === 2 && pathSegments[0] === 'people') return true;
}
const requestAllowed =
  publicPages.includes(event.url.pathname as (typeof publicPages)[number]) ||
  isPersonPage(event.url.pathname) ||
  (userNeedsSessionRefresh && event.url.pathname === commonRoutes.refreshSession);


Change SystemCall wrapper

Remove QEventLoop from it. Use signals and slots. It should be fully asynchronous. Now in fact system calls are blocking. Need to redesign everything related to system calls.

Done


Fix resolved header in email template

Background
We use a custom email template based on alertmanager/template/email.tmpl.
Problem
The email template for resolved messages currently has a styling issue. Due to an extra 	, the green heading is only shown in the top right corner, instead of covering the full width.

Solution
Removing the extra 	 solves the issue.

Can anyone help with the DCO check, please?

template/email.tmpl gets generated from template/email.html via the template/inline-css.js script which uses juice under the hood to inline the CSS properies into the style attributes.
IIUC juice is confused by this part and adds the extra
https://github.com/prometheus/alertmanager/blob/dcc92eda2ecbb96948c1e864338bfbf32ce8885b/template/email.html#L326-L334
One way to fix it would be to duplicate the range loop like this:
 {{ if gt (len .Alerts.Firing) 0 }}

   {{ .Alerts | len }} alert{{ if gt (len .Alerts) 1 }}s{{ end }} for {{ range .GroupLabels.SortedPairs }}
     {{ .Name }}={{ .Value }}
   {{ end }}

 {{ else }}

   {{ .Alerts | len }} alert{{ if gt (len .Alerts) 1 }}s{{ end }} for {{ range .GroupLabels.SortedPairs }}
     {{ .Name }}={{ .Value }}
   {{ end }}

 {{ end }}

Can this be merged? We are having the same problem and is annoying.

@cod-r this should be fixed in the latest v0.25.0 thanks to #3166


Typed and Friendly error messages for user visible errors

@cpuguy83 has some valid comments on error messages in : https://github.com/docker/docker/pull/16645#issuecomment-145719089.
These require changes in libnetwork errors and also type them appropriately so that docker could potentially check and change the message at will.

Patches linked merged, closing it


Let client perform the hashing but on custom pool?

Hi, thanks for this. So far, I've gotten this to work on a Ubuntu 16/17 server pointing to a custom pool. Been having issues seeing up the coin hive proxy.
Is there a way to use this code so a web page you run on a browser points to a custom pool?

Maybe this is the answer? https://minero.pw/#/

You can, you will need to setup a proxy and then configure the CoinHive miner to point to your proxy.


Will NTK RoPE be supported?

A discord community member asked if NTK RoPE will be supported in the future, and offered the example given below:
def build_rope_cache(
    seq_len: int,
    max_seq_length:int,
    n_elem: int,
    dtype: torch.dtype,
    device: torch.device,
    base: int = 10000,
    condense_ratio: int = 1,
    scaling_factor: float = 1.0
) -> RoPECache:

    if seq_len > max_position_embeddings:
        base = base * (
            (scaling_factor * seq_len / max_seq_length) - (scaling_factor - 1)
        )  (n_elem / (n_elem - 2))

    theta = 1.0 / (base  (torch.arange(0, n_elem, 2, device=device) / n_elem))

    seq_idx = torch.arange(seq_len, device=device) / condense_ratio

    idx_theta = torch.outer(seq_idx, theta).repeat(1, 2)

    cos, sin = torch.cos(idx_theta), torch.sin(idx_theta)

    if dtype in (torch.float16, torch.bfloat16, torch.int8):
        return cos.half(), sin.half()
    return cos, sin

Associated paper is Extending Context Window of Large Language Models via Positional Interpolation
https://arxiv.org/abs/2306.15595

It is already implemented in the transformers library: https://github.com/huggingface/transformers/blob/main/src/transformers/models/llama/modeling_llama.py
Here is the original reddit post about NTK Scaled RoPE: https://www.reddit.com/comments/14lz7j5
I think it would be important to feature this or Linear Scaled RoPE to enable Long-Context -Training.

Are there any "popular" checkpoints that use this technique? We'd only accept this contribution in that case.

Yes, Metas new Code Llama uses that technique: https://about.fb.com/news/2023/08/code-llama-ai-for-coding/
As well as pretty much every other Llama model claiming above 4k context for example https://huggingface.co/upstage/Llama-2-70b-instruct. It is recommended to train with the desired NTK setting but it can be also applied without training.

Can you specifically point where in the code-llama paper this is mentioned? The hub config doesn't specify anything: https://huggingface.co/codellama/CodeLlama-34b-hf/blob/main/config.json#L18 and AFAIK the only used a custom rope theta parameter, but not this scaling technique mentioned here.
We already support changing the condensation ratio: https://github.com/Lightning-AI/lit-gpt/blob/main/lit_gpt/config.py#L51 which is what the SuperHOT blogpost described.
If this technique is useful without training, then we could add support for it without any relevant checkpoint that requires it. It would be nice if you could run try it out in this codebase and run some experiments to demonstrate its performance with longer context lengths.

I'm hesitant about adding this since It's not clear to me what is the expected and popular formula. However, can play now modify the base to your liking after #464
config = Config.from_name("falcon-7b")
config.rope_base = ...  # scale using whichever formula you want, you can access the config for values

Yes, by changing the block size. If you use the technique described in https://kaiokendev.github.io/context, you'll need to adjust the rope_condensation_ratio. See our existing configs to find some examples
Also - fresh out of the oven - yet another rope extension technique
https://github.com/jquesnelle/yarn


Clarify in spec that there will be multiple model types

I've clarified that "glm" is the only model type currently supported but that we expect this to expand as software implementations expand.


⚠ Fetchers has degraded performance

In f78af9c, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 313 ms

Resolved: Fetchers performance has improved in 0237153.


integrate bicep CLI into az CLI and az powershell

The following commands should all work in Az CLI and Azure PowerShell respectively:
Compile and deploy a bicep file without emitting a main.json file
az deployment group create -f ./main.bicep -g my-rg

New-AzResourceGroupDeployment -TemplateFile ./main.bicep -ResourceGroupName my-rg

Run what-if and ask for confirmation before deploying:
az deployment group create -f ./main.bicep -g my-rg -c

New-AzResourceGroupDeployment -TemplateFile ./main.bicep -ResourceGroupName my-rg -Confirm

Transpile a .bicep file to its equivalent .json file
az bicep build ./main.bicep

Invoke-AzBicepBuild -File ./main.bicep

Both command line tools should check if the bicep exe is installed on the machine. If it's not, the user should be prompted to do so in an easy way. Similar to how kubectl is handled today for AKS command-line scenarios.
*** added 1/6 ***
Decompile an ARM Template into it's equivalent .bicep file
az bicep decompile ./main.json

Invoke-AzBicepDecompile -File ./main.json

Needs to support source-mapping of some kind. Since end user's wont' be interfacing with template JSON anymore, we need to make sure that errors returned during a deployment are mapped to the relevant spot in the bicep file.  It should degrade gracefully.

If there is a line number in the JSON, that should map to a line number in bicep
If there is no line number, but there is a resource name and type, we should be able to point you there in the bicep file

Described in more detail in #855

Discussed with PowerShell and Az CLI team, and our plan will be to follow the AKS/kubectl model which does the following:

There is a dedicated cmdlet/command for installing kubectl (az aks install-cli)
Any commands that need kubectl will expect the binary to exist in a well-known location. If there, it will call the binary. If it is not there, the user will be prompted to run the above command.

As part of this issue, we should aim to start #800 in parallel.

Discussed with @rynowak, who brought up that it would be helpful to be able to point to a custom install location of bicep so that you can use private or pre-release builds of bicep if you want to opt in. Not sure how AKS implements it, but if it is just pointing to well known PATHs, then it should be straightforward for user's to replace the binary manually and not build anything additional.

@alex-frankel - will bicep be included in cli docker image?

Good question. I think if we do nothing, it'll have the default version of Az CLI installed, so you will be prompted to install bicep when a relevant deployment command is invoked. I will add a note to see if we can have this installed on the image so that step doesn't need to happen.

@alex-frankel Would this also include support for validating deployments (az deployment group validate) or just the creation of deployments?

@alex-frankel Would this also include support for validating deployments (az deployment group validate) or just the creation of deployments?

It should include any cli command/cmdlet that uses the --template-file/-TemplateFile, so the validation commands should be covered. Thanks for the reminder as we have a bad tendency to forget about this command!

It should include any cli command/cmdlet that uses the --template-file/-TemplateFile, so the validation commands should be covered. Thanks for the reminder as we have a bad tendency to forget about this command!

What is the progress with supporting deploying .bicep files using Az PowerShell and AZ CLI? What support can we expect as a first step? I looked at the code implemented so far and open PRs, and it seems that Azure CLI will support a lot more things thanks to @shenglol.
Also created an issue in the azure-powershell repo with some findings on the code implemented so far, https://github.com/Azure/azure-powershell/issues/14289.

The initial scope of support is to install the bicep CLI if it is not present and to automatically transpile the bicep code if it is provided as the value for the -TemplateFile parameter. PowerShell team is a bit more resource constrained so their implementation is going to be less feature rich than Az CLI at least to start. Let's see what powershell team has to say about the issue you filed - thank you for reporting the issues you've found.

Both PS and CLI work for this has been merged and will be available on 3/2.
There is a slight difference in functionality. In Az CLI, we will install a self-contained version of bicep for CLI to use the first time you use a command that requires bicep. In Az PS, you need to manage the install of bicep yourself and ensure it is available on the PATH. We are planning to add an Install-AzBicep cmdlet, but this will not be in the first release.
@tfitzmac - I think it will be important to doc these discrepancies. Do you want me to update the installation instructions that are in this repo or will you plan to take care of this as part of the move to MS Docs?

@alex-frankel - I think we should update in the Bicep repo. When the doc has been published, we can decide if we want to maintain both places.

Just tested the Azure CLI code using the installer from the PR build. It works great! Only found one minor issue, https://github.com/Azure/azure-cli/issues/17083.
@alex-frankel I agree that it's very important to doc the difference between Azure CLI and Az PowerShell. And also write a note about the install path for the self-contained version installed by Azure CLI in %userprofile%\.Azure\bin\.  Depending on how you've been installing Bicep CLI you might end up with bicep.exe in three different directories, and possibly different versions for Windows users (PowerShell script installer, Windows installer and Azure CLI installed).


dependencies: update the template to 0.7

closes #28
Signed-off-by: Marko Baricevic<EMAIL_ADDRESS>

Thanks Marko!


v2: Minimum margin requirement info

Subject of the issue
People may miss depositing margin, and be confused why nothing works, probably 0 margin should make every other action not available / display a notification on press.
The minimum margin needed to trade is $100. When depositing <100$ and/or having a margin <100$, the interface should show an information message regarding this fact.
Temporary solution

[x] disable place order button if margin below $100 and some sort of clear status message near it
[x] display warning if depositing <100$

"Minimum margin to trade is 100$"
Ideas:

[ ]  Show Info Box under 'Deposit' / Withdrawal boxes when Margin <100$
[ ] On Deposit Modal, show information message when (current margin + deposit input amount) <100$.

modal:
https://www.figma.com/file/YWU3PMsnqZaqr1gABgAFJK/UI-Refresh?node-id=1925%3A10763

futures ui:
https://www.figma.com/file/YWU3PMsnqZaqr1gABgAFJK/UI-Refresh?node-id=1935%3A12431


blob where VHDX files are stored not accessible

Seems that both URI's to the two VHDX files are unavailable.
https://aka.ms/AAbbhkn
https://aka.ms/AAbclsv

Looking at this now, thanks for the heads up.

This has been corrected, please test @BartRoels  and confirm it is working as expected. Then I will close this issue. Thanks.

Confirmed links are working and access to VHDs was restored. Closing Issue. Thanks.


Notebook Tab/ mode and RAG on / off switch

What would you like to see?
(Doesn’t have to be, Again, just an idea that could be super cool)
Notebook mode: a tab where you can write notes, or Stories, etc, and then use either text completion, or simple command that enables instruct mode(to make the LLM write something specific in the notebook), and then the option to send it to the documents or save as text file (which can then be added to the RAG)
inspired by an oobabooga text gen extension:
Specifically the notebook and ‘quick instruct’ feature:
https://github.com/FartyPants/Playground?tab=readme-ov-file
And then on top of that: to add an option to turn RAG on or Off, so that you can ask an LLM normally without relying on added context optionally.
Just a rough example I can imagine in “notebook mode”:
‘[RAG mode: off]
(Written by user:)
Character A: 32, male, brown hair, blue eyes, powers:  teleportation, has a great friendship with character B, (etc.
character B: (more notes)
They are a (….)[(generate text completion; LLM)]
….
(Written by user ⬇)
(instruct) (example command): $$write me a summary of character A and B
Summary: …(created by LLM)
…
[RAG mode: ON](e.g: a pinned a document|
about some summary of a story/ concept)
(Written by user ⬇)
$$how can I add the concept of (from pinned document) and connect it with Character A’s powers and Character B’s… etc.
…
‘
———————
just ideas, but it would most definitely be a super Overpowered, useful and awesome feature.

I feel like this feature would be great as without it, I’d have to have 3 applications running and it would get messy and use up more power, etc.
Anyway, AnythingLLM is great project and love it a lot, and is my favorite overall application.

Dang, i really like the idea behind what you are getting at there. I see the overall point but im having trouble seeing how we can fit it into the app. I wonder if there is some opt-in mode or control we can surface to the frontend if desired to fullfill that kind of concept. Cool idea!


Fix nondeterminstic crash when initializing xnnpack tensors

See https://github.com/tensorflow/tensorflow/issues/55331

Anything I need to do to make the build pass or get this reviewed? Not sure why it's failing since the link doesn't work for me.

@nutsiepully Can you please review this PR ? Thank you!

@nutsiepully Can you please review this PR ? Thank you!

Bump

@nutsiepully Can you please review this PR ? Thank you!

@nutsiepully Can you please review this PR ? Thank you!

Is there a reason the build fails for macOS CPU and Windows Bazel? Looking at the logs, I don't see anything quite informative. Are there more informative logs on Google's side?

@richhx Can you please resolve conflicts? Thank you!

Should be fixed. Looks like someone else also found the issue and finally fixed it.

Hi @yishuangP Can you please review this PR ? Thank you!

Should be fixed. Looks like someone else also found the issue and finally fixed it.

Yes this is fixed in https://github.com/tensorflow/tensorflow/commit/b842657860e85490dfc074541a6ce83198476b33. Thanks for this PR anyway. I'm gonna close this one.


[SUPPORT] Current state of Java 17 support

I saw https://issues.apache.org/jira/browse/HUDI-6506 ("Java 11 and 17 support with Hudi") and it seems like this is only targeted towards 1.x release, though it seems like some work has been completed already for certifying runtime/compile support for Hudi on   Java 17:
https://github.com/apache/hudi/pull/9136 - it seems like CI has been running against Java 17 since 0.14.0
https://github.com/apache/hudi/pull/11604 - "Make Hudi compilable with Java 17"

What is the current state of Java 17 support for 0.14.x, 0.15.x, etc? We've done some tests on EMR and there's been no issues (and significant perf improvements in some test workloads), but wanted to check if there are any known issues.
Has the EMR team backported Java 17 support to their forks of Hudi? On EMR 7.x the default JVM became Java 17 - is Hudi certified to work with Java 17 on EMR?

@CTTY I saw you worked on supporting Java 17 in those tickets - is this something you could help answer? Appreciate the work on this

Hi @mzheng-plaid, we've been certifying Java 17 runtime support since 0.14 and we haven't noticed any major issues so far. Java 17 compilation time has been done as well: https://github.com/apache/hudi/pull/11604 . But the default Java version stays to be Java 8.
On EMR, Hudi 0.14/0.15 run on Java 17 by default.
Hopefully this can help answer the ask. Please feel free to create a ticket to EMR Hudi team if you have more questions regarding Hudi on EMR

Thanks appreciate the answer @CTTY that's very helpful


TSDuck acting as a hardware DCM?

Hello,
It seems like TSDuck embed plugins tlike ZAP on or more service on live streal or file, but can you consider acting as an hardware DCM (for e.g enbale / disable only PIDs in particular), then produce a new MPEG-TS live or file output with the a DVB tables (PMT, PSI, etc.) updated for this purpose? For e.g, be able to consider only the audio or subtitle PIDs? Thanks.

Hi @Vincent-Dabouineau
I have no idea of what you call a DCM. Based on your description, yes, you can filter in or out PID's or packets based on various combinations of conditions, you can modify the signalization on the fly, remove tables, add tables, patch tables, etc.
TSDuck was designed to allow any form of transformations on the transport stream.
What TSDuck cannot do is audio / video processing, transcoding or muxing. There are other tools for that.
I suggest that you browse through the description of the plugins in the users's guide, as well as the example chapter. If you have a specific operation in mind, please provide a precise description.


hello omo

you are on T.V.!!!!! https://lambdaurora.dev/optifine_alternatives/#:~:text=More Block Predicates,Available for%3A 1.17

opem

how dare close

this is cookieism


Move to new spec format for dex/sso

Via https://issues.redhat.com/browse/GITOPS-2761 we are told that the
dex configuration has a new format.
Old format:
spec:
  dex:
    openShiftOAuth: true
    resources:
    ...

New format:
spec:
  sso:
    provider: dex
    dex:
      openShiftOAuth: true
      resources:
      ...

This format is only supported starting with gitops-1.8.0, so we should
merge this only when we are absolutely sure that no pattern in no
situation needs an older gitops version.
Tested on MCG with gitops-1.8.2
Note: with this change gitops < 1.8 is not supported. Starting with
gitops-1.9 the old format will be unsupported.

Let's merge this in a few weeks, when we're reasonably sure that gitops versions < 1.8 won't be needed at all.


Register

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/41045
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.9.0 -m "<description of version>" 674bfd03b62d481bf4e79531cd3eedec01c35dc3
git push origin v0.9.0

Also, note the warning: This looks like a new registration that registers version 0.9.0.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/41047
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.9.0 -m "<description of version>" 6d6fd7f9e6f333035c15c28cac19694a131be813
git push origin v0.9.0

Also, note the warning: This looks like a new registration that registers version 0.9.0.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.


Thumb approximation for Index Emulation

Allow some degree of thumb motion in index-emulated games, by setting touch on the A button when the finger is pushed beyond a certain threshold.

firmware doesn't currently have a label for this 'touching a button' value but it would be nice for folks who might want to turn it off for some reason, however needs a change both places in that case.  this may be easier if always using this gestural behavior is going to be generally preferrable

Yeah, I think this was a bit of a misunderstanding caused by a typo I had in the code. The only purpose for doing this is specifically for the render model of hands to sort of show the thumb moving in index emulation-only games, so I can't think of any situation where it would be favorable for the firmware to have control over this.

Sure, will do


Support direct release library on maven central

I've used this template to create library KDeviceInfo and for publishing the library on maven central using the ./gradlew publishAllPublicationsToSonatypeRepository command. But with this command library is being created on staging repository and for every release I've to open nexus site and have to close and release process manually.
Also checked other available gradle command under publishing section but couldn't found any command for direct publishing.
So my question is there any option so could automate this library releasing process ?

As I know, there is no way to publish on the maven central in a single command. If you find it, post it here, please

There are two options to automate the release process :

gradle-maven-publish-plugin using this plugin with one single command we can publish new version of the library on maven central.
Command  : ./gradlew publishAndReleaseToMavenCentral --no-configuration-cache

publish-plugin not much popular but can consider this as well.


When the number of table more than eight, there will be no consumption of kafka

More than 8 tables in the cluster number, then the next table can't consumption kafka.
I tested many times, discover finally create table does not generate the data properly.

Created in front of the table, respectively table1_realtime,table1_offline,
                                            table2_realtime,table2_offline,
                                            table3_realtime,table3_offline,
                                            table4_realtime,table4_offline,
                                            table5_realtime
table5_realtime cannot consumption kafka.

@dongyanfeng  can how tell us how many instances are you using for these tables...
thanks

If there is a limited number of servers relative to ZK nodes, it could be the case that this exhausts the limit of ZK connections from a single node. The default value for maxClientCnxns in ZK is 10, so if there is only one ZK host and all the cluster is running on one machine, it could prevent the kafka client from connecting to zk.


chore: Remove google fonts imported from googleapis

Description
Google fonts are served by the appsmith instance and change that was introduced in this PR. This PR removes redundant the import of google fonts from googleapis.
Fixes #Issue Number
or
Fixes Issue URL

[!WARNING]
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!IMPORTANT]
🟣 🟣 🟣 Your tests are running.
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/10247572619
Commit: 58e5b2c97c64058334dca9c79db1749b3d09ca47
Workflow: PR Automation test suite
Tags: @tag.All
Spec: ``
Mon, 05 Aug 2024 11:02:12 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[ ] No

Summary by CodeRabbit

New Features

Transitioned to a fully local/custom font strategy, enhancing branding consistency and potentially improving load times.

Bug Fixes

Resolved issues related to external font loading dependencies, promoting a more stable application performance.

/build-deploy-preview skip-tests=true

@dvj1988 Can you ascertain airgapped users are not impacted by these changes?

@dvj1988 Can you ascertain airgapped users are not impacted by these changes?

@vsvamsi1 I have tested it on my local using this document. Everything works as expected on airgapped.


Mechanism for OCR correction

Fixes #40
The page is pretty spartan, but it gets the job done. When you submit a correction, it's logged in the usual OldNYC feedback queue. I'll build out something to use all this information after the launch. After submitting a correction, you're presented with another image. Hopefully users will go on a text correction rampage!

Never cross someone on a text correction rampage.
On Mon, May 18, 2015 at 11:06 AM, Dan Vanderkam<EMAIL_ADDRESS>wrote:

Fixes #40 https://github.com/danvk/oldnyc/issues/40
The page is pretty spartan, but it gets the job done. When you submit a
correction, it's logged in the usual OldNYC feedback queue. I'll build out
something to use all this information after the launch. After submitting a
correction, you're presented with another image. Hopefully users will go on
a text correction rampage!
[image: ocr correction]
https://cloud.githubusercontent.com/assets/98301/7683562/e8d9acd2-fd4d-11e4-9ce9-c0c8975e976e.png
You can view, comment on, or merge this pull request online at:
https://github.com/danvk/oldnyc/pull/84
Commit Summary

Mechanism for OCR correction

File Changes

M viewer/app.yaml
https://github.com/danvk/oldnyc/pull/84/files#diff-0 (3)
A viewer/static/ocr.html
https://github.com/danvk/oldnyc/pull/84/files#diff-1 (146)

Patch Links:

https://github.com/danvk/oldnyc/pull/84.patch
https://github.com/danvk/oldnyc/pull/84.diff

—
Reply to this email directly or view it on GitHub
https://github.com/danvk/oldnyc/pull/84.

--
David W. Riordan
mobile:<PHONE_NUMBER> | im: daveriordan | email<EMAIL_ADDRESS>|
@riordan http://twitter.com/riordan | http://magicschoolb.us


Admin UI: "A user must be supplied"

The output from this exception is going to catalina.out instead of cdradmin.log.

This is fixed with the error message and others like it routed back to the page for the user to see.


[ionic v4 with angular] Calling Slides methods (ex. slideTo) in ngAfterViewInit throws error

Bug Report
Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.
Ionic:

   ionic (Ionic CLI)          : 4.0.1 (/Users/bilal/.config/yarn/global/node_modules/ionic)
   Ionic Framework            : @ionic/angular 4.0.0-beta.0
   @angular-devkit/core       : 0.7.0-rc.3
   @angular-devkit/schematics : 0.7.0-rc.3
   @angular/cli               : 6.0.8
   @ionic/ng-toolkit          : 1.0.0
   @ionic/schematics-angular  : 1.0.1

Cordova:

   cordova (Cordova CLI) : 8.0.0
   Cordova Platforms     : none

System:

   Android SDK Tools : 26.1.1
   ios-deploy        : 1.9.2
   ios-sim           : 6.1.2
   NodeJS            : v8.9.4 (/Users/bilal/.nvm/versions/node/v8.9.4/bin/node)
   npm               : 6.2.0
   OS                : macOS High Sierra
   Xcode             : Xcode 9.4.1 Build version 9F2000

Environment:

   ANDROID_HOME : /Users/bilal/Library/Android/sdk

Describe the Bug
When call slides methods (ex. slideTo) in ngAfterViewInit throws following error:
Error: Uncaught (in promise): TypeError: Cannot read property 'slideTo' of undefined
TypeError: Cannot read property 'slideTo' of undefined
Steps to Reproduce
Steps to reproduce the behavior:

Go to stackblitz example.
Check console.

Related Code
https://stackblitz.com/edit/ion-slides-afterviewinit
Expected Behavior
Slides methods (ex. slideTo) should work in ngAfterViewInit.

Internally the ion-slides are still starting Swiper when you call the methods.
I suggest creating the ionSlidesLoaded event for ion-slides.

This is not yet documented, but this must have been fixed by # 14865

Now we can use ionSlidesDidLoad event in  "v4.0.0-beta.1" for calling slides methods.


Update Masthead to follow High Fidelity Mockups

User Story
Impacts
UI
UI Details
Need to update masthead to match styles of mockup here:
https://redhat.invisionapp.com/share/BJN9KGLTUD8#/screens
Acceptance Criteria

[ ] - Masthead styles match high fidelity mockup
[ ] - Background image matches the one in the mockup.

Merged via #78


Anaconda packaging does not track master branch.

Actually, this appears to be working fine -- I misread the logs.


[集] A collection of confusing problems met when developing JavaScript under IE

This is a task for collecting some confusing problems during developing under IE, mostly when we really get confused with IE8!! To avoid struggling with IE, it's smart to do this as early as I can.
这个 Task 主要是为了收集 IE 开发中所遇到困惑，尤其当我们被 IE8 所折腾的时候。因此，为了避免陷入与 IE 的斗争，我理应尽早地去执行该任务。
Reference Articles:
引用文章：

Console is not defined under IE8
Predefined and limit some by UglifyJS via {pure_funcs : ['debug', 'console.debug', 'console.warn']}
引入实现并通过 UglifyJS 限制部分方法使用

Blur event is fired when clicking on the scrollbar under IE8
Calculate the position of the mouse through the event and stop the propagation when clicking on the scrollbar.
计算鼠标位置并通过阻止事件传递解决问题

Internet Explorer's CSS rules limits
Use css-split-webpack-plugin to solve.
使用 css-split-webpack-plugin 来解决

Content-Editable Elements under IE 10
In the case when we implementing an editor, we may need to consider about such a case which I have met recently. Assume that there is an element which has the attribute contenteditable with true as its value, and the element has contained a large table at the same time:

A strange thing may happen when the table is followed after an empty tag
 under IE10. With inputing words by clicking the keyboard, the browser will not render characters immediately after you release the keyboard key. That's terrible!

In order to work around such a wierd occasion, we may have two ways:

Since a Content-Editable element contains a large table, it is recommended to replace all empty elements
 with a paratraph element
.
Another simple way is to set an CSS property overflow: auto to make a trick.

Detect the type of data in clipboard under IE (IE 下粘贴板数据类型检查)
Recently, I have met a requirement where I need to implement a pop-up notification for suggesting users to install upload plugins under IE, when they are copying a picture. Under webkit browsers, it can be easily implemented by atttaching a paste event onto a content-editable element, and get items via (e.clipboardData || e.originalEvent.clipboardData).items;. What confuses me is that IE does not support such a feature. Therefore, we should use another way to check whether a user is copying a picture.
最近，我遇到了这样的一个需求：检查用户是否粘贴图片，并以此提醒用户安装上传插件。在 Webkit 内核的浏览器下，我可以轻松地通过对可编辑元素进行 paste 事件绑定，以获取相应的粘贴数据 (e.clipboardData || e.originalEvent.clipboardData).items; 来判断粘贴板是否粘贴有一图片来实现。然而，困扰我的是 IE 并没有这样的一个特性。因此，我们需要另外想一些办法来判断。
Actually, under IE, we can only use the method window.clipboardData.getData('Text') to access the cliplboard of system. What it means is that we should know how it acts when users copy different things:
实际上，在 IE 下我们只能通过方法 window.clipboardData.getData('Text') 来访问系统级粘贴板。这就意味着，我们需要了解用户在拷贝不同东西的情况下该方法到底会返回什么给我们：

When copying a string, the method will return the text content after being called.
When copying nothing, the method will return an empty content after being called.
When copying a file or a picture through some screen shot tools, the method will return null after being called.

当拷贝一段字符串时，方法会原封不动地返回该文本
当粘贴板为空时，方法会返回一个空字符串
当粘贴一个文件或使用截图工具粘贴一张图片时，方法会返回 null

After my testing between IE8 to IE11, the method should return accroding to the occasions above. Therefore, if we want to implement such a requirement under IE, we can just check whether window.clipboardData.getData('Text') returns null.
经过 IE8 到 IE11 下的验证，该方法都会按照以上情况来返回不同的结果。因此，为了实现 IE 下检测用户粘贴的是否为一张图片，我们只需要判断方法是否返回 null 即可。

Horriable event for property changing under IE (IE 下监听属性改变的事件是何其可怕)

Automatically adjust table under IE8
Assume that there is a table with being set as table-layout: fixed, which has resized its content by hiding a column. Under IE8, such a table will not adjust its width to fit the table container outside, while it does under Chrome, Firefox, or even IE7.
To work around this problem, we can simply used the following snippet, so that we can make the table adjustable for its changing contents:
table.style.display = 'inline-block';
setTimeout(function () {
    /** macrotasks executed in the end of current event loop */
}, 0);

Two global functions definition ways may be different under IE8（IE8 两种全局函数定义会有所不同）
As a common sense of coding JavaScript, it is not a wise choice to define functions (or variables) globally, as developers struggle with definning conflict problems when using shared functions, especially when the project has become larger and larger, like SPA.
不要定义全局函数（或变量）早已成为 JavaScript 开发中的一个常识。源于使用共享函数时，大量的全局定义可能会使得开发者深陷于变量定义冲突的问题，尤其是当项目像单页面应用（SPA，Single-page Applications）那样越来越庞大的时候，身同体会。
However, it seems to be the only one choice when you have to communicate with IE add-ons. For example, I have a task needing add-ons to handle asynchronously, and tell the result to me with a callback method named done(result). In such a case, the most common way you may meet is where add-ons developers use FireBreath to implement this. With this framework, you may have to define a global function, and pass the string of the function name to add-ons, so that they can callback result for you with calling your defined functions.
然而，当你需要跟 IE 插件进行数据交互时，定义全局函数似乎是唯一可行的方法。举个例子来说，我需要插件异步处理一个任务，并通过回调告诉我结果。这种情况下，最寻常的做法是插件开发者会使用 FireBreath 来实现，而该框架下，你需要定义一个全局函数，并把函数名以字符串的形式传递给插件。这样，插件才能把结果回调至你所定义的函数。
function scope() {
    /** function scope but not global one */
    document.getElementById('#ie-plugin').runJSFunc('test', result); /** nothing happens */

    /** throw "Method Invoke Failed" */
    function test(result) {
        /** handle with result */
    };
}

Ooops, there is something wrong under IE8!!
不会吧，IE8 怎么这么傲娇！！
You can't define the function globally with window:
竟不能通过 window 对象来定义全局函数：
try {
    document.getElementById('#ie-plugin').runJSFunc('test', result);
} catch (e) {
    console.log(e.message); /** => Method Invoke Failed. */
}

/** throw "Method Invoke Failed" */
window.test = function (results) {
    /** handle with results */
};

That's why I marked this with documenting here, in order to tell you how to solve this with following workarounds:
这就是为何我要在此进行文档记录，以告知诸位如何用下面的方式解决问题：

Define it directly (only be suggested when not defining in global scope)
直接定义（仅建议在非全局情况下使用）
test = function (results) {
    /** handle with results */
};

Use the official simple way (only use when it is in the global scope)
使用官方简单的函数定义（仅局限于全局作用域下使用）
function test(results) {
    /** handle with results */
}

Wierd phenomenas during loading IE add-ons（加载 IE 插件时出现的奇象）
Recently, I have met some wierd things when trying to load IE add-ons under IE 8 and 9. Since the IE add-on was designed as an object element in HTML, developers around this add-on had guided me to load their plugin within front-end developing following this tutorial:
最近，在 IE8 和 IE9 下尝试加载 IE 插件时，又遇到了一些奇象。既然 IE 插件是被设计成 HTML 中的 object 对象，当时那边的开发者们就引导我用以下的方法去加载该插件：

Firstly, define function to handle after the plugin has been loaded:
首先定义插件加载完成后的处理函数：
window.pluginLoaded = function () {
    /** the plugin has been loaded */
};

Append the object element into your HTML files:
添加元素到 HTML 文件里
<object id="ie-addon" type="application/x-xxxplugin">
    <param name="onload" value="pluginLoaded" />
</object>

However, due to some real developing requirements, I had to dynamically inject elements into DOM, and the first plan I designed was to use appendChild method to do so:
然而，基于现实中的若干开发需求，我只能动态地往 DOM 插入该对象元素。而使用 appendChild 则是我首先想到的方案：
const obj = document.createElement('object');
obj.id = 'ie-addon';
obj.setAttribute('type', 'application/x-xxxplugin');
const param = document.createElement('param');
param.setAttribute('name', 'onload');
param.setAttribute('value', 'pluginLoaded');
obj.appendChild(param);

/** `pluginLoaded` has not been invoked under IE9 */
document.body.appendChild(obj);

The code snippet above seemed to run well under IE8, while something wrong happened when ran it under IE9. What I found was that the pluginLoaded method was not invoked at all, and I could still call inner methods defined inside this plugin object, like plugin.xxx(). After calling the inner method, another wierd thing happened, and the method did not do return data as usual. As a workaround, I changed to use innerHTML to load this plugin:
上述的代码片段在 IE8 下似乎毫无瑕疵，而在 IE9 中却发生了奇怪的事情。我当时发现，pluginLoaded 函数根本就没有被触发到，而 plugin 内所定义的内部函数却能被调到，如 plugin.xxx()。即便如此，内部函数在调用后也没有像以往一样返回数据。因此，我转而尝试通过 innerHTML 来加载插件：
const obj = document.createElement('object');
obj.id = 'ie-addon';
obj.setAttribute('type', 'application/x-xxxplugin');
const param = document.createElement('param');
param.setAttribute('name', 'onload');
param.setAttribute('value', 'pluginLoaded');
obj.appendChild(param);

document.body.innerHTML += obj.outerHTML;

Cool, it really worked under IE9, BUT the plugin has broken down under IE8, as it was loaded more than once. After some confusing inspections, I doubted that the plugin had been already initialized once the object element was created. That's why I finally only use the following plain:
果真，方案在 IE9 确实行得通，但 IE8 下竟由于插件被多次加载导致插件崩溃。经过多番检查，我怀疑是因为插件在 object 元素创建后，即被初始化，这也就为何说明我最后用了以下唯一可行的方案：
document.body.innerHTML += `
    <object id="ie-addon" type="application/x-xxxplugin">
        <param name="onload" value="pluginLoaded" />
    </object>
`;

Notify downloading JSON responses under IE8/IE9 ?（IE8/IE9 下提示下载 JSON 返回请求？）
When posting a file to the server, like uploading, we usually use multipart/form data to construct our requests, and wait for responses from the server, formatted with JSON in common. However, there will be troubles when requesting under IE8/IE9, where users will receive notification to download the response as a file after requesting. This is also one of the most common problem arisen when using jQuery-File-Upload. That's because you have received a response with setting application/json or test/x-json as its Content-Type header. To avoid undesired download dialog, we must change our server to response with text/plain or text/html. And then, the client should receive a string value, which can be parsed as a JSON object, to check responses of the server. (Note: after responding with text contents, the client should also set text or html as its requesting data types.)
当我们想上传一个文件给服务器时，大多数情况下我们通用的方法是使用 multipart/form 的方式来构造请求，并等待服务器返回一个 JSON 对象以判断其响应结果。然而在 IE8/IE9 下，用户会为此收到一个下载通知，提示下载返回的信息。这也是使用 jQuery-File-Upload 最为常见的问题之一。原因在于返回头设置了其 Content-Type 头部为 application/json 或 test/x-json。为了避免该非预期内的下载框，我们必须改造我们的服务器返回一个 text/plain 或 text/html 的响应，并由客户端去解析 JSON。（注：响应文本后，客户端在请求时需同时更改其请求数据类型为 text 或 html。）

Unable to select text throuth a content-editable element under IE?（IE 下无法在可编辑元素内选择文本？）
When implementing a editor, which should be compatible through different browsers, especially IEs, you may need to take this issue tracker as a consideration, as I rencently found that I could not select text from bottom to top, through a content-editable element within iframes under IE11 when the area was scrollable (such like a quite long paragraph). After debugging, this issue was resulted because the implementation of the editor has ignored to set elements as content-editable, rather than switching on the designed mode.
倘若你要实现一个兼容不同浏览器，尤其是 IE 浏览器的编辑器，你就要考虑以下我所遇到的问题。最近，我发现在 IE11 下一个 iframe 内的可编辑元素在可滚动的情况下，无法自下而上选择文本。后来通过调试发现，该问题是因为编辑器在实现元素可编辑化时，针对 IE11 错误把元素的 designMode 打开而非设置 contentEditable。
var iframeBody = docuemnt.querySelector('.j-editor').contentWindow.document.body;
iframeBody.designMode = 'on'; /** result in the problem of selecting text */

Instead of setting designMode with on value, you may have to use the following snippet to work around (even under all versions of IE):
为了解决该问题，需要设置 contentEditable 为 true，而非打开 designMode（所有 IE 都需要如此处理）：
iframeBody.contentEditable = true;

Similaly, I also found a documented issue under the Stack Overflow here.
此外，我在 Stack Overflow 上也找到类似问题记录，详情可参考此处。

Lose comments between select option tags under IE8/IE9

Naming anonymous function problems under IE8

Unexpected toString  prototype of Object under IE8

Don't try to prevent default events and cancel events bubbling in another event loop

Keyup events won't fire when there is a button within the same Form under IE8/IE9/IE10（IE8/IE9/IE10 下若 input 标签所属 Form 含有 button 元素则无法监听 Keyup 事件）
Recently, I found that when I was focusing on an input element, the Form of which had a button elements, I couldn't get it fired on listening keyup events after pressing the enter key. To reproduce such a bug, I have simplified it as the following case:
最近，我发现在同一个 Form 表单内，含有 button 及 input 元素的时候，input 标签输入文字后按回车并不会触发所监听的 keyup 事件。为了重新构造该问题，下面举一个较为简单的例子：
<input type="text" class="input-wrapper" value="" />
<button>test</button>

$('input.input-wrapper').on('keyup', e => {
    console.log(e.keyCode); /** cannot not print 13 when pressing "enter" key */
});

It seemed like the button element had captured the event and did not propagate to the input element.
从现象来看，像是 button 元素捕获了事件而没有往 input 元素传递。
To workaround this, there are two ways:
为了解决该问题，有两个办法：

Used keypress or keydown event instead:
改用 keypress 或 keydown 事件进行监听：
$('input.input-wrapper').on('keydown', e => { console.log(e.keyCode); });

$('input.input-wrapper').on('keypress', e => { console.log(e.keyCode); });

Wrap the input elements with an empty form:
在 input 元素外包裹一层空 Form 表单：
<form action="javascript: void(0);">
    <input type="text" class="input-wrapper" value="" />
</form>
<button>test</button>

Some problems around IPv6 address you may need to know under IE8

Form action need to be set with correct address format constructed with [::1] rather than ::1. Otherwise, the browser should throw an error: "Invalid Arguments."
When calling postMessage() during iframes, the origin paramter should also be passed with correct address format like the case 1.

So if you want to avoid those kinds of problems, the smart way to construct url is:
/**
 *  -----------------------------------------------------------------
 *  | location.host | document.domain |  location.port |   browser   |
 *  | ::1:8080      |     [::1]       |      8080      |     IE8     |
 *  | ::1:8080      |     [::1]       |      8080      |     IE9     |
 *  | [::1]:8080    |     [::1]       |      8080      |     IE10    |
 *  | [::1]:8080    |     [::1]       |      8080      |     IE11    |
 *  | [::1]:8080    |     [::1]       |      8080      |   Chrome    |
 *  | [::1]:8080    |     [::1]       |      8080      |   Firefox   |
 *  -----------------------------------------------------------------
 */
const fullRequest = path =>
    `${location.protocol}//${document.domain}${location.port ? `:${location.port}` : ''}/${path}`;

Array.prototype.splice is not fully supported under IE8

Array.prototype.splice(start[, deleteCount[, item1[, item2[, ...]]]])

As defined in MDN, Array.prototype.splice should be supported under IE8. However, I recently found that Array.prototype.splice method did not act as expected when passing only one argument (start). As mentioned in MDN, when deleteCount is omitted, all of the elements from start through the end of the array will be deleted, but it seemed like it was not under IE8.
console.log([1, 2, 3].splice(0)); /** => [], under IE8 */
console.log([1, 2, 3].splice(0)); /** => [1, 2, 3], under Chrome */

To work around this wierd situation, I have wrapped this method with AOP:
const splice = Array.prototype.splice;
Array.prototype.splice = function (start, deleteCount, ...args) {
    return splice.apply(this, [start, arguments.length === 1 ? this.length : deleteCount, ...args]);
};

Within ES3:
var splice = Array.prototype.splice;
Array.prototype.splice = function (start, deleteCount) {
    for (var _len = arguments.length, args = Array(_len > 2 ? _len - 2 : 0), _key = 2; _key < _len; _key++) {
        args[_key - 2] = arguments[_key];
    }

    return splice.apply(this, [start, arguments.length === 1 ? this.length : deleteCount].concat(args));
};

clearTimeout and clearInterval is not a Function under IE8
// => [object Object] under IE8
({}).toString.call(clearTimeout);
({}).toString.call(clearTimeout);

// => [object Function] under IE9/Chrome
({}).toString.call(clearInterval);
({}).toString.call(clearInterval);

It means that we should notice that do not use Function.prototype.bind() at all like the following case:
const timer = setTimeout(() => {});
const _clear = callback => callback && callback();

_clear(clearTimeout.bind(window, timer));

Scrollbar style crash IE9?
The following case will crash IE9:
<!doctype html>

    <button class="target"
                  onclick="alert(window.getComputedStyle(this, ':before').getPropertyValue('scrollbar-3dlight-color'))">
      click to crash IE9
    </button>

Wrong base64 images with text from the clipboard under IE11 (IE11 下图文粘贴时错误的 base64 编码图片)
When I tried to copy a snippet of texts and an image from a Word document at the same time, I found that the image from the clipboard under IE11 would specify a wrong Content-Type in the image base64 code like data:image/png;xxx, but that image was converted into JPEG formats. It means that if we upload it via Blob, IE11 fails to render such an image from the server by throwing DOM7009: Unable to decode image at URL: '[some unique url]' when the server responses with a header X-Content-Type-Options: nosniff;.
当从 Word 文档同时粘贴一段图文时，就会发现 IE11 粘贴板内所拿到的图片 base64 地址含有错误的 Content-Type 如 data:image/png:xxx，而实际上该图片已经被转换成 JPEG 格式。这就意味着，如果我们把 base64 地址转成 Blob 上传到服务器时，IE11 就会由于服务器返回的 X-Content-Type-Options: nosniff; 头部而编码不了该图片显示裂图，并抛出异常 DOM7009: Unable to decode image at URL: '[some unique url]'。

Wrong base64 images with text from the clipboard under IE11 (IE11 下图文粘贴时错误的 base64 编码图片)
When I tried to copy a snippet of texts and an image from a Word document at the same time, I found that the image from the clipboard under IE11 would specify a wrong Content-Type in the image base64 code like data:image/png;xxx, but that image was converted into JPEG formats. It means that if we upload it via Blob, IE11 fails to render such an image from the server by throwing DOM7009: Unable to decode image at URL: '[some unique url]' when the server responses with a header X-Content-Type-Options: nosniff;.
当从 Word 文档同时粘贴一段图文时，就会发现 IE11 粘贴板内所拿到的图片 base64 地址含有错误的 Content-Type 如 data:image/png:xxx，而实际上该图片已经被转换成 JPEG 格式。这就意味着，如果我们把 base64 地址转成 Blob 上传到服务器时，IE11 就会由于服务器返回的 X-Content-Type-Options: nosniff; 头部而编码不了该图片显示裂图，并抛出异常 DOM7009: Unable to decode image at URL: '[some unique url]'。

We cannot access some attributes of DispHTMLCurrentStyle under IE8 (IE8 下无法直接访问 DispHTMLCurrentStyle 的部分属性)
In IE8, there is no Window.getComputedStyle() method for extracting styles from a DOM element. In such a case, we need to use Element.currentStyle or ElementCSSInlineStyle.style as a workaround. These attribute should be defined as a DispHTMLCurrentStyle object.
在 IE8 下, 我们无法通过  Window.getComputedStyle() 方法来获取 DOM 元素的样式。这时候，我们可以使用 Element.currentStyle 或 ElementCSSInlineStyle.style 来替代。两者属性所返回的对象，理论上命名为 DispHTMLCurrentStyle。
Nevertheless, I recently found that we can directly access some attributes of it like outline or outlineWidth under IE8. It should throw an error: Unspecified error.
然而，最近我发现在 IE8 下我们如果直接访问其中的部分属性如 outline 或 outlineWidth，浏览器将会抛出类似异常：Unspecified error。

URL() Constructor in old Edge (旧版 Edge 中的 URL() 构造)
Before Edge 79, query arguments in the base URL argument are removed when calling the URL() constructor event passing empty URL as the first argument.
在 Edge 79 版之前，即便第一个参数传空 URL，base 参数所传的 URL 变量仍然会被移除。
// Chrome, Firefox, Node, New Edge
new URL('', 'http://www.test.com?sid=1').searchParams.get('sid'); // => "1"
// Old Edge
new URL('', 'http://www.test.com?sid=1').searchParams.get('sid'); // => null

If you can't generate Blob with Uint8Array / Int8Array ... under IE10 / IE11 (若你在 IE10 / IE11 无法生成 Uint8Array / Int8Array ... 的 Blob)
Sometimes, if you get such a error when trying to generate Blob with a given Uint8Array / Int8Array ...:
new Blob([new Uint8Array(0)]); // => "InvalidStateError"

You can workaround by using MSBlobBuilder:
var bytes = new Uint8Array(0);
try {
    new Blob([bytes]); // => "InvalidStateError"
} catch (ignore) {
    var blobBuilder = new MSBlobBuilder();
    blobBuilder.append(bytes.buffer);
    blobBuilder.getBlob();
}


Introduce custom user model

Still putting this up as a draft. I have tested AWX with this, and have a corresponding patch for aap-gateway.

Likely to be closed per @AlanCoding

yeah, b/c I am throwing in the towel on doing a custom user model in AWX


Add dtype, fweights, aweights to cupy.cov

Fixes #6383
This is a revival of #6401, which applies the last comments that @emcastillo left.

/test mini

Test failures seem to be real

Can we please not bump Jitify in this PR? Is it needed?

@andfoy sorry, could you take a look at the test failures?
https://ci.preferred.jp/cupy.linux.cuda102/106932/
let us know if you don't understand any of these :)
Thanks!

/test mini

The test failures are unrelated, as they are NCCL-related


Has pretrained model?

@huangyangyu,Hi if has the pretrained model?

@huangyangyu,Hi if has the pretrained model?

L2-softmax model is trained without pretrained model.
ArcFace model is trained with the corresponding L2-softmax model as pretrained model. These two models are both trained with the same noisy dataset.


Use new host hooks for importing modules

This updates this proposal to use the hook introduced by https://github.com/tc39/ecma262/pull/2905.

I updated this PR now that https://github.com/tc39/ecma262/pull/2905 has been merged. This is necessary for whatwg/html#8253, since HTML links to this proposal.


Remove IO.puts logging on Startup

I couldn't bear the IO.puts screaming at me every time I start my app. http://dsh.re/ef5f3

👍

@hassox can we merge this one?


bugfix: 插件管理-节点列表页面加载异常 (closed #334)

Codecov Report

:exclamation: No coverage uploaded for pull request base (V2.1.358-rc@65ec512). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             V2.1.358-rc     #342   +/-   ##
==============================================
  Coverage               ?   62.62%
==============================================
  Files                  ?      246
  Lines                  ?    19953
  Branches               ?        0
==============================================
  Hits                   ?    12496
  Misses                 ?     7457
  Partials               ?        0

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 65ec512...65e0377. Read the comment docs.


Timer on Training Page

Shouldn't be too much work but a timer defaulting to 20 sec would be nice instead of using another device or estimating in your head.

I considered it, but we should be careful here. 20 seconds isn't a magical number. What's important is that you deliberately practice a set.
I've spent several minutes working on sets that were difficult for me. Ultimately, we want the user to do however much would challenge them. But 1) it's hard to convey that in the UI, and 2) it actually requires some practice in of itself to understand when you are effectively practicing.

For some more context, Smash Training is based on the Stronglifts weightlifting app. In that app, you always do five reps of an exercise, and then the rest time is variable.
There's a slider at the bottom that goes through three phases. It's suggested you rest 1:30 if the set was easy, 3:00 if normal, or 5:00 if hard. However, the official guidance is to do your next set just as your heart rate and breathing return to normal.
Unfortunately, there's no simple physiological signal that we can wait on in our case 😓


Remove duplicated code between arvo and hoon

Also removes some dead code from arvo which lets us clean up some
^ in dojo.

Yeah, scaffolding doesn't belong on(/the way to) master obviously, but might be good to put a revert in a "future-marks" branch or similar.
(It'll probably be useful as part of staging a live network update, but like, with the specific types the new features turn out to actually need)


Release 27-08-19

rename of flybase file and new odk release

Not sure why this is failing on Travis - it says 'duplicate definition', but this is only supposed to be a warning, so what is the error?

Oh, just spotted this at the top:
ERROR	missing_label	GO:0048018	rdfs:label

this is 'receptor ligand activity' but I cant find the id or label in any of the files!

I don't think the fail comes from this missing GO term label.  It looks like a robot fail on report generation:
2019-08-27 13:38:26,588 ERROR org.obolibrary.robot.ReportCommand - Report failed!
make: *** [fbcv.Makefile:236: obo_qc_fbcv.owl] Error 1
How urgent is this? May take some time to debug.  If we need to get a release out ASAP, we may be able to fix temporarily by commenting out report generation from the MakeFile.

Yes, this is an irrelevant bug by the robot report generation process (which does not distinguish between native and foreign terms) that I am circumventing by injecting some assertions (in this case, bogus labels) into the artefact on which the QC is run.. I have fixed it now; if this happens again, and the error message at hand is unrelated to FBCV terms, you can try to inject an assertion here. Or ping me again and I will do it!


Llama 3.1 Structured Response

To follow the naming convention, could you move it to examples/structured/structured_response_llama_3_1.ipynb?

@pgarbacki can you please approve this? I'll remove the additional file. Somehow could not delete it rightaway

Could you please move it to the correct path per earlier suggestion?

As for file removal:
git rm examples/Structured_Response_Llama_3_1.ipynb
git commit -a --amend
git push -f origin llama-3.1

Done, I've made necessary changes. @pgarbacki