add check for attrs attribute in MyHTMLParser

Add a check for parser.attrs
fixes #10

Thanks for the PR! However I don't think this is the right solution. It seems like the HTML parser doesn't detect a start tag. Could you add a print for tag_match[0] and let me know what value it has in the case that fails?

Thank you Danmou for your incredibly fast response, and sorry for my delay. You are probably right, I just hacked together something, so that it would work.
I think I have to abandon using the api, since I do not get any of my hand-drawings out of it. I think I'll stick to something like .mht export and then extracting the base64-images.

Here's results of my tag_match[0] print, commit b098dc6e553d64725aa46b19ae352c20f5099d75
...snip...
    Opening page 7 Fossil CMS
      HTML file already exists; skipping this page
    Opening page 8 Fossil learning TH1
      Got content of length 19954
parser.feed tag_match:
<IP_ADDRESS> - - [04/Jan/2022 07:48:47] "GET /getToken?code=M.R3_BAY.225081db-b1a8-f945-ac69-3ee570dcadd4&state=a6efcfda-9d5f-4e28-98dd-93b766dd6b55 HTTP/1.1" 500 -
Traceback (most recent call last):
  File "C:\ProgramData\scoop\apps\miniconda3\4.10.3\Lib\site-packages\flask\app.py", line 2091, in __call__
    return self.wsgi_app(environ, start_response)
  File "C:\ProgramData\scoop\apps\miniconda3\4.10.3\Lib\site-packages\flask\app.py", line 2076, in wsgi_app
    response = self.handle_exception(e)
  File "C:\ProgramData\scoop\apps\miniconda3\4.10.3\Lib\site-packages\flask\app.py", line 2073, in wsgi_app
    response = self.full_dispatch_request()
  File "C:\ProgramData\scoop\apps\miniconda3\4.10.3\Lib\site-packages\flask\app.py", line 1518, in full_dispatch_request

    rv = self.handle_user_exception(e)
  File "C:\ProgramData\scoop\apps\miniconda3\4.10.3\Lib\site-packages\flask\app.py", line 1516, in full_dispatch_request

    rv = self.dispatch_request()
  File "C:\ProgramData\scoop\apps\miniconda3\4.10.3\Lib\site-packages\flask\app.py", line 1502, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)
  File "C:\Users\Matt\code\onenote_export\onenote_export.py", line 240, in main_logic
    download_notebooks(graph_client, app.config['output_path'], app.config['select_path'], indent=0)
  File "C:\Users\Matt\code\onenote_export\onenote_export.py", line 179, in download_notebooks
    download_sections(graph_client, sections, path / nb_name, select, indent=indent + 1)
  File "C:\Users\Matt\code\onenote_export\onenote_export.py", line 200, in download_sections
    download_pages(graph_client, pages, path / sec_name, select, indent=indent + 1)
  File "C:\Users\Matt\code\onenote_export\onenote_export.py", line 216, in download_pages
    download_page(graph_client, page['contentUrl'], page_dir, indent=indent + 1)
  File "C:\Users\Matt\code\onenote_export\onenote_export.py", line 229, in download_page
    content = download_attachments(graph_client, content, path, indent=indent)
  File "C:\Users\Matt\code\onenote_export\onenote_export.py", line 150, in download_attachments
    content = re.sub(r"", download_image, content, flags=re.DOTALL)
  File "C:\ProgramData\scoop\apps\miniconda3\4.10.3\Lib\re.py", line 210, in sub
    return _compile(pattern, flags).sub(repl, string, count)
  File "C:\Users\Matt\code\onenote_export\onenote_export.py", line 112, in download_image
    props = parser.attrs
AttributeError: 'MyHTMLParser' object has no attribute 'attrs'

Here's a screenshot of the portion of the page it seems to be having trouble with. It's two images embedded in a table

the complete tag match text is:

(updated the debug statement to if debug: print(f"parser.feed tag_match: '''{tag_match[0]}''' ") to make it easier to see )


S3 backend and some Win32 cherry-picks from upstream

We're using git-fat at our company and we found the two binary mode commits necessary for the smudge filer to work on Windows platforms.
I also made a few changes that abstract the backend push/pull operations to allow for different implementations because we tend to store our files in S3.

@jedbrown After seeing the HN discussion, I'll work on getting this PR updated.  We've made some progress on our fork since this was created.
Regarding the config file format, do you have any specific preferences?

@willkelleher That's great news. For the config file, I really want named remotes in every way analogous to Git remotes.  I think the syntax could be like
[remote "foo"]
  url = rsync://user@host/path/
[remote "bar"]
  url = s3://.......

I don't know if I've talked with you about it, but I think it's worth making a Git backend for fat files (just tagged objects so they can be pushed and pulled independently) and I feel like `url = user@host:path/storer.git" should have that meaning (for consistency with normal Git remotes).
What do you think?

@jedbrown Do you mean you want to evolve git-fat into a custom Git object backend that would store the large file objects in some other backend but leave other files in the flat file object store?
I'm not very familiar with Git's support for custom backends, but it looks like libgit2 enables something relevant.  How would you 'tag' the specific objects that you want to use custom storage vs. the traditional backend?
Let me know if I totally missed your point, but this sounds interesting.

@willkelleher I have noticed people using git-fat to store somewhat compressible files, such that the "dumb" object store takes a lot more space than a git packfile (and more network bandwidth). I also see an overhead associated with maintaining two access control lists (one for the Git repository and one for the fat object store). The mechanics can be done with Git's core command line tools or with libgit2. There is more discussion in Issue #1.

Hi guys, im really interested on this feature, is this usable 'as is' now. I would like to migrate a current repo which uses git-fat into S3. Is this possible right now with this branch and without so much hacking??

+1

@magec you can try to use https://github.com/PersonifyInc/git-fat (a fork) we used it for years and we can pull/push files to S3 rather well, it can also support large file.


Provide good error message when the type is capitalized

Hot newcomer path probably.

type something = {
  types: array<SuperType>
}

@chenglou yeah, it tripped me up almost immediately starting out with rescript. The  Did you forget a `.` here? message is quite misleading. It seems that message shows up in a bunch of cases where it isn't very helpful to newcomers.


Feature request: simple playback control

Get some simple playback control (play, pause, forward, rewind) in the menu which appears when clicking on the text in the menubar.

There is a analogue project name bytecontroller doing that.

+1

For anyone else looking, there's another app available called SmashTunes that gives this and track name as well as some other options. http://smashbits.nl

Thanks for the suggestion. As others have mentioned, there are other apps offering that sort of functionality, and Statusfy is deliberately more simplistic than that.
Personally, I use the dedicated keys on Apple keyboards for this type of thing.


#15409: Lock 'SignaturePlaceHolderLbl' to avoid translation

#15409

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Availability update: We will publish a fix for this issue in the next update for release 19.

Hi Jesper,
your last response was 14 days ago - at that time the "next update for
release 19" would have been BC19.02 which came out and does not yet contain
the fix. We have a customer that is waiting on this. Will this come with
BC19.03? Do you have any ETA?
Thanks
On Fri, 3 Dec 2021 at 09:38, Jesper Schulz-Wedde @.***>
wrote:

Availability update: We will publish a fix for this issue in the next
update for release 19.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/ALAppExtensions/pull/15410#issuecomment-985317866,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACD4LEFGLQ3QXP2RCUYE4TTUPB6YJANCNFSM5JCJL3FA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

The change went in on the 2nd of December and will hence ship with the January update, which is 2021 Wave II Update 3. As a rule of thumb, we branch for the upcoming monthly minor in the last week before the upcoming month and the update releases early in the month. Occasionally, internal or external issues prevent us from releasing at the planned time, but the release still remains locked for additions other than those which will resolve the release issues.


DisplayName is not generated when running in through webpack-karma

I am able to get the tranform-react-display-name plugin running when running webpack in my normal environment with this configuration.
module: {
    loaders: [
      { test: /(\.jsx|\.js)$/,
        exclude: /node_modules/,
        loader: 'babel',
        query: {
          env: {
            development: {
              plugins: [
                ["transform-react-display-name"],
              ],
            },
          },
        },
      },
    ]
  }

However, with a nearly identical configuration in my karma.conf with webpack-karma, it does not transform the displayName. I need this functionality to unit test some of my components.
module: {
        loaders: [
          {
            test: /(\.jsx|\.js)$/,
            exclude: /node_modules/,
            loader: 'babel',
            query: {
              env: {
                development: {
                  plugins: [
                    ["transform-react-display-name"],
                    ["babel-plugin-rewire"]
                  ]
                }
              }
            },
          },
        ],
      },

I was able to trace it down to this call in the isCreateClass function.
if (!isCreateClassCallExpression(node.callee)) return false;

Moving to #67.


slim instructions

Some of this is out of date
https://www.pombase.org/browse-curation/fission-yeast-go-slimming-tips
I will suggest revisions shortly

google doc with edits sent to Midori

replying to
@mah11
I do not have access to this document (or to google drive generally) via this email address. I would greatly prefer if you either put the text in the website ticket, or email it to me; when there are no other collaborators it doesn't make any sense to use google docs.

Hopefully you can see the edits her easily.
I use it because it is easier  to see the changes.
These points will help you understand GO slims, and highlight some features of the fission yeast slim terms and annotations.
Note that the counts in the “Genes” column of the GO slim table are not additive, because many gene products are annotated to multiple terms.
It not possible to create a slim with no overlaps between terms. Although the S. pombe slim has been defined to include biologically informative terms, and minimise overlaps between terms, large overlaps cannot be entirely avoided. For example, most of the gene products annotated to signal transduction are also annotated other terms in the slim.
As a consequence of points 1 and 2, GO slim annotation summaries should not be presented using pie charts. Although a pie chart could show the fraction of total annotations for any slim term, it can too easily be mistaken for the fraction of total annotated gene products, which is not the same.
It is difficult to define a slim which includes all annotated gene products without including terms with very small numbers of annotations (for example, the peroxisome organization cell ageing branch has very few annotations), or very high- level terms which are not particularly biologically informative (e.g. cellular process). Because we have opted not to include such terms, some gene products are annotated to process terms but do not appear in the slim annotation set.
Bear in mind that both proteins and RNAs can be annotated to GO terms. If you are working only with proteins you will need to make adjustments for this. For example, many tRNAs and rRNAs are annotated to cytoplasmic translation, and are therefore included in the slim set.
There is a difference between “unknown” and “unannotated”. All fission yeast and budding yeast gene products have been assessed and are classed as “unknown” for biological process if no biological process information is found (experimental or inferred). If you are making comparisons with other organisms, remember that it is possible that not all gene products have been assessed and that the “unknown” set is underestimated.
The default S. pombe slim includes all evidence codes for fission yeast. The evidence code IEA (inferred from electronic annotation) is often considered to be less accurate than other evidence codes, but it is very useful for increasing the coverage of some of the high level GO terms. Accurate annotation counts for some terms currently depends on including this evidence code (for example, there are 126 gene products annotated to transmembrane transport with IEA evidence, which are not yet covered by a manual annotation). For fission yeast, the IEA annotations improve slim coverage, but only represent a small number of annotations (519654 biological process annotations as of May September 202011), and have a low rate of false positives. We therefore recommend that you include them.
If you are making comparisons with budding yeast (or other organisms), you should consider excluding the evidence code RCA. This evidence code is used for functional predictions, and has a very high rate of false positives (for example, including RCA for budding yeast hugely will greatly and artificially inflate the number of annotations to translation).
Creating a user-defined slim
You can create your own slim, or retrieve slim annotations for a gene set, using online slimming tools such as the GOTermMapper at Princeton, or the QuickGO tool at the EBI (not that you need to use UniProt, not PomBase ID’s in Quick|GO.
When creating a slim for the entire genome, you should try to ensure that it covers as many annotated genes in your set as possible (see #3 in list above). You should be aware of how many genes are annotated but not in your slim, and how many are “unknown” (i.e., annotated only to the root node; see #5 in list above).
For display purposes, you usually want to keep the number of terms as small as possible to convey your results. However, you should ensure that the terms you include are specific enough to capture biologically relevant information. Many terms (e.g. metabolic process (3229 2915 annotations), cellular process (4581 083 annotations)) are too general for the purpose of most slim-based analyses.
On a related note, if you are using your slim for data analysis (e.g. to summarize an enrichment), you should ensure that the terms are specific enough to demonstrate their relevance to the biological topic of interest. For example, lumping all genes involved in transport my mask overrepresentation of transmembrane transport vs. underrepresentation of vesicle-mediated transport in your results set, so you need to ensure that the slim has categories to represent your results effectively.
Most current implementations of software to create “GO slims” include the regulates relationship by default, so that (for example) genes involved in regulation of cytokinesis will be included with the set of genes annotated to cytokinesis. See the GO Ontology Relations documentation for further information about relationships in GO. The annotation totals presented for the PomBase default S. pombe slim, by contrast, are calculated both explicitly including or excluding the genes which are involved in a process via regulation only. We expect this distinction to be available in future versions of slimming software.

No the edits didn't show

Just plug in what you want as the new text; I can diff it.

These points will help you understand GO slims, and highlight some features of the fission yeast slim terms and annotations.
Note that the counts in the “Genes” column of the GO slim table are not additive, because many gene products are annotated to multiple terms.
It not possible to create a slim with no overlaps between terms. Although the S. pombe slim has been defined to include biologically informative terms, and minimise overlaps between terms, large overlaps cannot be entirely avoided. For example, most of the gene products annotated to signal transduction are also annotated other terms in the slim.
As a consequence of points 1 and 2, GO slim annotation summaries should not be presented using pie charts. Although a pie chart could show the fraction of total annotations for any slim term, it can too easily be mistaken for the fraction of total annotated gene products, which is not the same.
It is difficult to define a slim which includes all annotated gene products without including terms with very small numbers of annotations (for example, the peroxisome organization branch has very few annotations), or very high-level terms which are not particularly biologically informative (e.g. cellular process). Because we have opted not to include such terms, some gene products are annotated to process terms but do not appear in the slim annotation set.
Bear in mind that both proteins and RNAs can be annotated to GO terms. If you are working only with proteins you will need to make adjustments for this. For example, many tRNAs and rRNAs are annotated to cytoplasmic translation, and are therefore included in the slim set.
There is a difference between “unknown” and “unannotated”. All fission yeast and budding yeast gene products have been assessed and are classed as “unknown” for biological process if no biological process information is found (experimental or inferred). If you are making comparisons with other organisms, remember that it is possible that not all gene products have been assessed and that the “unknown” set is underestimated.
The default S. pombe slim includes all evidence codes for fission yeast. The evidence code IEA (inferred from electronic annotation) is often considered to be less accurate than other evidence codes, but it is very useful for increasing the coverage of some of the high level GO terms. Accurate annotation counts for some terms currently depends on including this evidence code (for example, there are 16 gene products annotated to transmembrane transport with IEA evidence, which are not yet covered by a manual annotation). For fission yeast, the IEA annotations improve slim coverage, but only represent a small number of annotations (519 biological process annotations as of May  2020), and have a low rate of false positives. We therefore recommend that you include them.
Creating a user-defined slim
You can create your own slim, or retrieve slim annotations for a gene set, using online slimming tools such as the GOTermMapper at Princeton, or the QuickGO tool at the EBI (not that you need to use UniProt, not PomBase ID’s in Quick|GO
When creating a slim for the entire genome, you should try to ensure that it covers as many annotated genes in your set as possible (see #3 in list above). You should be aware of how many genes are annotated but not in your slim, and how many are “unknown” (i.e., annotated only to the root node; see #5 in list above).
For display purposes, you usually want to keep the number of terms as small as possible to convey your results. However, you should ensure that the terms you include are specific enough to capture biologically relevant information. Many terms (e.g. metabolic process (3229  annotations), cellular process (4581  annotations)) are too general for the purpose of most slim-based analyses.
On a related note, if you are using your slim for data analysis (e.g. to summarize an enrichment), you should ensure that the terms are specific enough to demonstrate their relevance to the biological topic of interest. For example, lumping all genes involved in transport my mask overrepresentation of transmembrane transport vs. underrepresentation of vesicle-mediated transport in your results set, so you need to ensure that the slim has categories to represent your results effectively.
Most current implementations of software to create “GO slims” include the regulates relationship by default, so that (for example) genes involved in regulation of cytokinesis will be included with the set of genes annotated to cytokinesis. See the GO Ontology Relations documentation for further information about relationships in GO. The annotation totals presented for the PomBase default S. pombe slim, are calculated both explicitly including the genes which are involved in a process via regulation only. We expect this distinction to be available in future versions of slimming software.


Predictions attached to comments weren't migrated

It's possible to attach predictions to comments, like this:

The data is stored in latest_prediction of metac_question_comment of the old database.
I understand it's not in the same format, but that's no reason to not import them.
Here's a dataclip that gives you the data in the new format.

@SylvainChevalier Migrated, please confirm

@SylvainChevalier Migrated, please confirm


WIP Add KodeinPreferenceFragment

Added this for convenience. Works identically to a KodeinFragment except it's a PreferenceFragment

This needs more changes like I have made to #55

Great idea,
I'll merge this when it's working like KodeinFragment ;)

KodeinPreferenceFragment has been added and will be released in Kodein 4.1


Fix mistake in the description of prisma migrate deploy

The first paragraph mistakenly states that the prisma migrate deploy command is meant to be used in "development" environments, while it should in fact say "production".
This command is meant to be used in non-development environments, such as production, as clearly stated here and here.

Thanks for catching this, @aradalvand !!


Add options for custom linker and compiler.

Add options for custom linker and compiler in the stages that will be implemented from #4.
[[bin]]
default = true
name = "Test"
compiler = "gcc"
linker = "gcc"

Resolved in #14


refactor(release): use release-it as release automation tool

Pull Request Test Coverage Report for Build 475

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 463:
0.0%

Covered Lines:
693

Relevant Lines:
693

💛  - Coveralls


Advantage not being handled properly for player characters

Checklist

[x] I have disabled GM Toolkit and the issue doesn't happen.
[x] I have disabled all other modules (except GM Toolkit) and the issue still happens.
[x] I go not get a Spectator Notification related to actors affected by this issue.
[x] I can be reached on Discord by my handle: clapo

Versions

Foundry VTT: v11.311
WFRP4e System: v7.0.2
GM Toolkit:  v6.0.5
Macro (if applicable): Add Advantage

Description
What should happen
Adding or removing advantage using the macro or automated system would work on player characters.
What is happening
Adding and removing advantage using the macro or automated system only works for npcs. (Edit: this only happens if the gm wants to edit the advantage, the macro works fine for players, the automation is still broken)
Steps to reproduce the issue

Select / Target Player character
Execute macro Add/Remove advantage or enable automated advantage
Notification comes up but with no update to the advantage number

Images

Thanks for raising the issue. Think I'm going to need more info.

Can you share any console log output from when the process please, whether successful or errors. A screencast would be handy.
Does this happen with jjust Isaiah or other player characters?
The character is in an active combat right?
Does the same happen whether you have Group Advantage enabled or disabled?
Do you have these options all enabled?

Thanks

Sorry for the late reply, didnt see this will get back tocyou tomorrow when
I have done these tests :)
On Thu 12 Oct 2023, 07:05 Jag Goraya, @.***> wrote:

Thanks for raising the issue. Think I'm going to need more info.

Can you share any console log output from when the process please,
whether successful or errors. A screencast would be handy.
Does this happen with jjust Isaiah or other player characters?
The character is in an active combat right?
Does the same happen whether you have Group Advantage enabled or
disabled?
Do you have these options all enabled?

[image: image]
https://user-images.githubusercontent.com/521096/274497390-b0753dc2-22f9-41ed-90a1-dd98ea1b1219.png
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/Jagusti/fvtt-wfrp4e-gmtoolkit/issues/250#issuecomment-1758963132,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUR5TIVXMSFXB544XC6D3GDX66CBBANCNFSM6AAAAAA52XHAXE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Take your time. No rush at my end. Cheers!
On Sat, 14 Oct 2023, 01:35 Clapolad, @.***> wrote:

Sorry for the late reply, didnt see this will get back tocyou tomorrow
when
I have done these tests :)
On Thu 12 Oct 2023, 07:05 Jag Goraya, @.***> wrote:

Thanks for raising the issue. Think I'm going to need more info.

Can you share any console log output from when the process please,
whether successful or errors. A screencast would be handy.
Does this happen with jjust Isaiah or other player characters?
The character is in an active combat right?
Does the same happen whether you have Group Advantage enabled or
disabled?
Do you have these options all enabled?

[image: image]
<
https://user-images.githubusercontent.com/521096/274497390-b0753dc2-22f9-41ed-90a1-dd98ea1b1219.png>

Thanks
—
Reply to this email directly, view it on GitHub
<
https://github.com/Jagusti/fvtt-wfrp4e-gmtoolkit/issues/250#issuecomment-1758963132>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AUR5TIVXMSFXB544XC6D3GDX66CBBANCNFSM6AAAAAA52XHAXE>

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Jagusti/fvtt-wfrp4e-gmtoolkit/issues/250#issuecomment-1762426282,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD7HCBYLE3MUHAU5DYVZ7DX7HM4VANCNFSM6AAAAAA52XHAXE
.
You are receiving this because you commented.Message ID:
@.***>

I had the same issue with player characters after installing Up-in-Arms and oddly the issue went away after I duplicated the characters and used the duplicates instead of the originals. Why? I have no idea. :D

Issue is caused by improper checking of ownership and several wrong assumptions (that there is only one GM and all Documents were manualy and directly created by said GM)
Let me explain.
In Foundry, ownership is an object, which in it's most concise form can simply be {default: 3}. 3 represents a level of Ownership as can be checked in CONST.DOCUMENT_OWNERSHIP_LEVELS.
Then, any deviation from default must be applies to specific user by user id. For example: {default: 3, "QkizEELLge2OGlxk": 0}.
When user creates a Document, Foundry gives the Document the following ownership:
{
  "default": 0,
  "QkizEELLge2OGlxk": 3
}

What about GM?
GM is considered Owner of every and any Document in the World, no matter the default or explicit level.
So what was the issue?
When Player creates a Document (like an Actor), or an Actor is created programatically, there is no guarantee it will show GM's ID in there, which shows only as a side effect of how Foundry generates default ownership, and only if the specific GM creates that Document.
The issue would also appear in Worlds with multiple GMs, if one was that created the Document, then the other would not explicitly appear in the Ownership object.
And this Module simply checks for ownership[game.user.id], which has multiple issues:

It may and often will return false for GMs if they are not the ones who created the Document
It will return true for Limited and Observer levels, who have no permissions to edit Actors

Instead it is better to use Foundry's built in getter ClientDocument.isOwner, which code is as follows:
    get isOwner() {
      return this.testUserPermission(game.user, "OWNER");
    }

    testUserPermission(user, permission, {exact=false}={}) {
      const perms = DOCUMENT_OWNERSHIP_LEVELS;
      const level = user.isGM ? perms.OWNER : this.getUserLevel(user);
      const target = (typeof permission === "string") ? (perms[permission] ?? perms.OWNER) : permission;
      return exact ? level === target : level >= target;
    }

I made Pull Request to adress the issue:

#262


Clarify temporal alignment of all Permafrost layers

Right now it's unclear if the Jorgensen, Obu, datasets are referring to a present-day assessment or future conditions.  This should be clarified in the legends, and also in the data report (above the bulleted list of results).

I think the temporal alignment is clear enough at this point, but we do now have a disconnect between the Climate Maps page and any real links to the supporting academic references.  Adding another ticket for that.


'module' object is not callable

I thought I'd experiment with this alpha utility because I'm finding the REST service has too much overhead for the many thousands of quick queries I need to run.
I tried this with both Python 2.7 and python 3.4, since it isn't clear which python flavor it is supposed to be used with.
After cloning the repo, I started by following the steps in the readme:
'''
$  python3
Python 3.4.0 (default, Jun 19 2015, 14:20:21)
[GCC 4.8.2] on linux
Type "help", "copyright", "credits" or "license" for more information.

import sys
sys.path.append('/home/rotten/src/neo4j-python-driver')
import neo4j
driver = neo4j.driver("bolt://myneo4jserver")
Traceback (most recent call last):
File "", line 1, in
TypeError: 'module' object is not callable
'''

I get the same complaint about it being a module with both python versions.
'''

type(neo4j.driver)
<class 'module'>
neo4j.driver.file
'/home/rotten/src/neo4j-python-driver/neo4j/driver.py'
'''

What am I missing?

The package name is wrong. You need:
from neo4j.v1 import GraphDatabase
driver = GraphDatabase.driver("bolt://localhost")


Feature request: Add ingress annotation to enable upstream "keepalive" option

In the latest NGINX release, 0.9.0-beta.7, there is no mention of the keepalive option from the upstream module. It'd be nice to be able to configure this, presumably with an annotation on an ingress resource. I'm thinking something like ingress.kubernetes.io/upstream-keepalive: {integer} (explicitly including upstream in the name to differentiate between this and the client<->server keepalive settings, though this could simply be ingress.kubernetes.io/keepalive).
I might wind up trying to implement this myself, but I figured I should go ahead and open the feature request regardless. I searched open issues and don't think I saw anything like this; apologies if this is a duplicate feature request.

@vdavidoff Let me know if you need any help implementing this :)

@vdavidoff let's start adding a global parameter in the configuration. We can add an annotation after we verify this works without issues.

Thanks!


Set up a "Hello world" project with two apps

Set up a "Hello world" project with two apps
In this milestone, I implement the following:

Create a new React app called 'hello-react-front-end'
Initialize the project with Git.
Create the App component with react-router.
Create the Greeting component that will display a greeting.
Create a store, an action, and a reducer that will connect to my API endpoint to get the random greeting.
Display the random greeting in my Greeting component.

Hi @aubinleyoung 😃,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

Well done on making the required changes from the previous reviewer for the back end repo

Cheers and Happy coding!👏👏👏
Feel free to mention @MahmoudBakr23 to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Getting permission denied while trying to setup cortx-k8s

Error while trying to follow the steps in the below document :
https://github.com/Seagate/cortx-k8s/blob/stable/doc/cortx-aws-k8s-installation.md
While trying to execute step number 2.7 I am getting the following error :
# ssh $SSH_FLAGS centos@$ClusterControlPlaneIP "kubectl create -f https://docs.projectcalico.org/manifests/tigera-operator.yaml; kubectl create -f https://docs.projectcalico.org/manifests/custom-resources.yaml"
Warning: Permanently added 'XX.XXX.XX.XXX' (ED14408) to the list of known hosts.
error: error loading config file "/home/centos/.kube/config": open /home/centos/.kube/config: permission denied
error: error loading config file "/home/centos/.kube/config": open /home/centos/.kube/config: permission denied

Please check if /home/centos/.kube/config exists and is accessible by centos user.
Have you completed the previous step?
ssh $SSH_FLAGS centos@$ClusterControlPlaneIP "mkdir -p .kube; sudo cp -i /etc/kubernetes/admin.conf .kube/config; sudo chown $(id -u):$(id -g) .kube/config"

Please check if /home/centos/.kube/config exists and is accessible by centos user.
Have you completed the previous step? ssh $SSH_FLAGS centos@$ClusterControlPlaneIP "mkdir -p .kube; sudo cp -i /etc/kubernetes/admin.conf .kube/config; sudo chown $(id -u):$(id -g) .kube/config"

Yes I ran the previous step and it was successful, I didn't get any errors or warning there.
Also, the directory /home/centos/.kube/config does exist and is accessible by centos user

Please check if /home/centos/.kube/config exists and is accessible by centos user.
Have you completed the previous step? ssh $SSH_FLAGS centos@$ClusterControlPlaneIP "mkdir -p .kube; sudo cp -i /etc/kubernetes/admin.conf .kube/config; sudo chown $(id -u):$(id -g) .kube/config"

Yes I ran the previous step and it was successful, I didn't get any errors or warning there.
Also, the directory /home/centos/.kube/config does exist and is accessible by centos user

Could you run the following commands:
ssh $SSH_FLAGS centos@$ClusterControlPlaneIP   - to connect to the master node
On the master node:
ls -ld /home/centos/.kube /home/centos/.kube/config
kubectl get nodes
kubectl get pods -ALL
And share the output?

Could you run the following commands:
ssh $SSH_FLAGS centos@$ClusterControlPlaneIP   - to connect to the master node
On the master node:
ls -ld /home/centos/.kube /home/centos/.kube/config
kubectl get nodes
kubectl get pods -ALL
And share the output?

Could you run the following commands: ssh $SSH_FLAGS centos@$ClusterControlPlaneIP - to connect to the master node
On the master node:
ls -ld /home/centos/.kube /home/centos/.kube/config
kubectl get nodes
kubectl get pods -ALL
And share the output?

I ran the commands and this was the output:
[root@fedora cortx-k8s-cl03]# ssh $SSH_FLAGS centos@$ClusterControlPlaneIP
Warning: Permanently added 'XX.XX.XX.XXX' (ED12408) to the list of known hosts.
Last login: Mon Dec 27 11:20:42 2021 from XXX.XXX.XX.XX
[centos@ip-XXX-XX-XX-XXX ~]$ ls -ld /home/centos/.kube /home/centos/.kube/config
drwxrwxr-x. 2 centos centos   20 Dec 27 10:50 /home/centos/.kube
-rw-------. 1 root   root   5641 Dec 27 10:53 /home/centos/.kube/config
[centos@ip-XXX-XX-XX-XXX ~]$ kubectl get nodes
error: error loading config file "/home/centos/.kube/config": open /home/centos/.kube/config: permission denied
[centos@ip-XXX-XX-XX-XXX ~]$ kubectl get pods -ALL
error: error loading config file "/home/centos/.kube/config": open /home/centos/.kube/config: permission denied

please run on the master node:
sudo chown centos:centos /home/centos/.kube/config

please run on the master node: sudo chown centos:centos /home/centos/.kube/config

This worked !! Thanks alot!

Great, glad it helped. Please share your experience with CORTX.


Opacity stopped working inside Overlay on iOS Impeller.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create OverlayPortal
Run animation inside overlayChildBuilder
Pass animation value to Opacity

Expected results
Widget below opacity gradually fadesIn
Actual results
Opacity waits for animation to complete and then jumps to opacity=1.
Though it rebuilds
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  late final controller = OverlayPortalController();

  @override
  Widget build(BuildContext context) {
    return OverlayPortal(
      controller: controller,
      overlayChildBuilder: (context) => Positioned.fill(
        child: GestureDetector(
          onTap: () => controller.hide(),
          child: TweenAnimationBuilder(
            duration: const Duration(milliseconds: 2000),
            tween: Tween(begin: 0.0, end: 1.0),
            builder: (context, value, child) {
              return Opacity(
                opacity: value,
                child: Container(
                  color: Colors.black.withOpacity(value),
                ),
              );
            },
          ),
        ),
      ),
      child: Scaffold(
        body: Center(
          child: OutlinedButton(
            onPressed: () => controller.show(),
            child: const Text("Hello, world"),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/flutter/flutter/assets/35890159/c42256fd-bef8-486f-86b2-084f134dc11b

Logs
Logs
No error logs

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.6, on macOS 14.0 23A344 darwin-arm64, locale en-RU)
    • Flutter version 3.13.6 on channel stable at /Users/alexander/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ead455963c (2 weeks ago), 2023-09-26 18:28:17 -0700
    • Engine revision a794cf2681
    • Dart version 3.1.3
    • DevTools version 2.25.0

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A240d
    • CocoaPods version 1.13.0

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 75.1.4
    • Dart plugin version 232.9559.10

[✓] Connected device (1 available)
    • iPhone 15 Pro Max (mobile)    • 93696608-FDB8-498F-8BCC-26DA524864DA • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Worth mentioning: if you run the same code in Chrome, it works as expected

It's fixed on beta:
Flutter 3.15.0-15.2.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision 0d074ced6c (3 weeks ago) • 2023-09-21 10:24:15 -0700
Engine • revision 453411bcf3
Tools • Dart 3.2.0 (build 3.2.0-134.1.beta) • DevTools 2.27.0

Hi @AlexanderFarkas. Thanks for filing this. I see this has been fixed in the latest channels on my end as well.
https://github.com/flutter/flutter/assets/53122008/246783f3-828f-46d3-bc84-e85156670d05
Closing this as fixed.


Narrow what the pip3. logic drops from entry points.

Summary
The hack in pip itself only modifies entry points called pip<number>.<number> and easy_install-<number>.<number>, uv previously dropped too many items including any of the form foo.<number>.
Found while trying to install memray which somewhat notably does not provide an abi3 wheel, so the installed, suffixed script matches.  At a minimum, this makes the installed files match the entry_points.txt more than it did previously, which makes pickley happy.
Test Plan
New test provided for previously-untested code.

Ah ok, so memray3.13 was previously matching. Makes sense.

Ah ok, so memray3.13 was previously matching. Makes sense.

Right, bin/memray3.13 was previously being skipped (and not replaced with anything), but the installed entry_points.txt still referenced it.


Feature Request: Social Media Entries for Mastodon and Matrix

While testing vutuv I found out that one is unable to enter his Mastodon and Matrix address in the social entries and only the big data miners are listed.

We are currently in the process of finishing the next version of vutuv. So we do not open any new feature requests for the old version right now.
I can not tell you when the new version is going online. But if you are interested in the process: Have a look into our repository. It is a very open process.
For now I close this issue.

In addition to Mastodon and Matrix, Pixelfed, Hubzilla, Friendca and other Fediverse services would also make sense in my eyes.


Typo fix in owncloud

line 13:
a owncloud deployment->an owncloud deployment
PR #4985 failed to add cncf-cla label, so I recreate it. Thanks a lot!

/assign @carrodher

/approve
/ok-to-test
/lgtm


[ISSUE #3010]Method manually handles closing an auto-closeable resource [RegistryHandler]

Fixes #3010.
Modifications
Used try-with-resources for OutputStream
Documentation

Does this pull request introduce a new feature? (no)

@PickBas Please fix the conflicts


Address gcc warning

Passing size as a variable rather than literal 0 to vsnprintf causes
the following gcc warning:
asprintf.c: In function ‘vasprintf’:
asprintf.c:43:8: warning: null destination pointer [-Wformat-truncation=]
   size = vsnprintf(NULL, size, fmt, tmpa);
   ~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I know that gcc has its own library implementation of this function, but it's
still worth having a portable implementation that works with gcc.

Thanks!


Update logging and job error message when getting the service account

Change logging and error messages to reflect failures in service account retrieval.
Adheres to logging style guide[1]

[1] https://github.com/ManageIQ/guides/blob/d4108998ab7230a95f917dceac5aaf42e7a7dcf3/coding_style_and_standards.md#logging

@miq-bot add_label providers/containers, smart_state, bug, darga/yes

@enoodle please review

Travis is failing and outstanding questions in the review. Reassigning to the author.
@miq-bot assign moolitayer

Fixed tests and added a new one for the relevant failure (fails w/o this fix), changed logging
@simon3z ok to remove this from darga?

@simon3z @enoodle please review

@simon3z @enoodle ping

@simon3z @enoodle updated PTAL

@moolitayer still relevant? what's the status? I see that this is darga/yes, is it needed soon?

This is ready and includes a test. I've updated the pr message and title to reflect what this does.
@simon3z @enoodle please review.
@miq-bot remove_label darga/yes

This is ready and includes a test. I've updated the pr message and title to reflect what this does.
@simon3z @enoodle please review.

@moolitayer ah right we had to discuss that controversial block of code that is logging messages.

I am sorry, I forgot about this one. LGTM :+1: I am even ok with line [59] now that you are getting the service account right after it.
[59] https://github.com/ManageIQ/manageiq/pull/9075/files#diff-0a5ccb8a9fe991f71c0d38d1783ac03eR59

@moolitayer ah right we had to discuss that controversial block of code that is logging messages.

@simon3z Do you have any objection to this at the moment?

@simon3z Do you have any objection to this at the moment?

@moolitayer yes. Let's have a call early tomorrow morning on this (lots to point out).

LGTM :+1: @enoodle please review
@moolitayer there are failing tests

LGTM :+1: It seems like the best solution now. The error in travis suggests that another mock function is needed in the tests.

@simon3z PTAL

LGTM :+1:
@miq-bot assign chessbyte


Analytics.js 2.0 and Angular

Is a Angular a 2nd class citizen :[ ?
Could we have some docs on creating a nice angular service for the new version please?
Thanks!
p.s. I'm going to have a go at the service; will post it here.

Here you are fellow angularites
Bypass the thin segment and heap docco and perhapps adapt this to your needs....
(this is for a Heap integrated through the Segment console impl.)
npm i @segment/analytics-next

// see: https://developers.heap.io/reference/client-side-apis-overview
interface Heap {
  track: (event: string, properties?: Object) => void;
  identify: (identity: string) => void;
  resetIdentity: () => void;
  addUserProperties: (properties: Object) => void;
  addEventProperties: (properties: Object) => void;
  removeEventProperty: (property: string) => void;
  clearEventProperties: () => void;
  appid: string;
  userId: string;
  identity: string | null;
  config: any;
  loaded: boolean;
}

declare var window: { heap: Heap };

@Injectable()
export class SegmentAnalyticsService {
  private analytics?: Analytics;
  private context?: Context;

  // use an APP_INITIALIZER
  loadAnalytics = (): Promise<Analytics | undefined> => {
    return new Promise<Analytics>((resolve) => {
      AnalyticsBrowser.load({
        writeKey: environment.segmentKey
      })
        .then((response: [Analytics, Context]) => {
          this.analytics = response[0];
          this.context = response[1];

          this.analytics.ready().then((res) => {
            console.log('ANALYTICS READY: ', this.analytics);

            resolve(this.analytics);
          });
        })
        .catch((e) => {
          console.error('Error loading segment analytics');
          resolve(undefined);
        });
    });
  };

  identify = (activeUser: ActiveUser) => {
    this.analytics
      .identify(activeUser.cognitoId, {
        name: `${activeUser.forename} ${activeUser.surname}`,
        email: activeUser.email
      })
      .then();
  };

  addMemberProperties = (activeMember: Member) => {
    this.clearMemberProperties();

    // a user can switch the organisation in context. we want to track activity by organisation - DO NOT use user traits for this
    const properties = {
      'Organisation Id': activeMember.organisationId,
      'Organisation Name': activeMember.organisationName,
      'Cloud Flavour': activeMember.cintraCloudFlavour,
      'Roles': activeMember.roles?.join(', '),
      'Messaging Enabled': activeMember.messagingEnabled.toString(),
      'Groups Enabled': activeMember.groupsEnabled.toString()
    };
    window.heap.addEventProperties(properties);
  };

  clearMemberProperties = () => {
    window.heap.clearEventProperties();
  };

  track = (event: string, payload: any) => this.analytics.track(event, payload).then();

  logout = () => {
    this.clearMemberProperties();
    this.analytics.reset();
  };
}

@Injectable()
export class MockSegmentAnalyticsService {
  loadAnalytics = (): Promise<Analytics | undefined> => {
    return new Promise<Analytics>((resolve) => {
      resolve(undefined);
    });
  };

  identify = (activeUser: ActiveUser) => {};

  addMemberProperties = (activeMember: Member) => {};

  clearMemberProperties = () => {};

  track = (event: string, payload: any) => {};

  logout = () => {};
}

/// core.module.ts

    {
      provide: SegmentAnalyticsService,
      useClass: environment.analyticsDisabled ? MockSegmentAnalyticsService : SegmentAnalyticsService
    },

`

Awesome! thanks for the example implementation. Angular not a 2nd class citizen for sure, we just don't have Angular expertise in house to write-up an example, so should rely on the community 🙌
... ( and we will fix the @types/jquery type issue soon )


Add expectation tier function for non-members

Before this function implemented, user cannot know where they are
and what are the proper problems without tier
This function works following steps

Parse the solved problem in user page
Get Level information from Api server
Calculate the total exp point from level information
Display expectation tier

Thanks
Seokho

Please check resolved your review! :)

Fix missed K & R style in e21189947c693c84d66feb67f475d8c01c489ea5 to c467e18bfae2e51c6521f739bd4af462810423ce; a blank space.
:)

PTAL

I'm currently too busy preparing problems for Sogang Programming Contest. I'll take a look when I'm not busy. Thanks.

Ah Okay! Take a time!

And here we go!


VS2019 support

Please add

Which projects types do you still use it for? VS 2017+ has the edit buttons for Net Core/Standard projects, or any projects that use the SDK style csproj.

Old project files
Actually extension installs but does not appear as context menu item

https://blogs.msdn.microsoft.com/visualstudio/2018/09/26/how-to-upgrade-extensions-to-support-visual-studio-2019/

Just as a note - The extension in the Visual Studio Marketplace claims it does support 2019, But the installer doesn't add it to 2019, only 2017.

https://github.com/MisterJimson/LetMeEdit/pull/7

We still need this extension to edit the solution file.

@nvmkpk, you can checkout my branch and build extension for VS2019

Done in 1.2.0.


submit the cpu topology exporter code

Signed-off-by: huone1<EMAIL_ADDRESS>

@huone1
As the repo name is changed to "resource-exportor". We might support more kind of resource in the future.
So I'd suggest to restruct the directory, put cpu topology under directory/package.

@huone1
As the repo name is changed to "resource-exportor". We might support more kind of resource in the future.
So I'd suggest to restruct the directory, put cpu topology under directory/package.

currently， keep the directory struct。just restruct the directory when adding more kind of resource

/lgtm
/approve


Add Gnosis safe app manifest and snapshot icon+avatar

For these, should we create a png with the fitting sizes for each of them?


Secret for local_uri needs to be added to prod Secrets.toml for a successful deployment

When I use shuttle_shared_db I can make use of the local_uri parameter.
In conjunction with shuttle-secrets I can set this to local_uri = "{secrets.DATABASE_URL}".
Even though the local uri is only used for local development, I need to set DATABASE_URL not only in Secrets.dev.toml but also in Secrets.toml 🤨
If the parameter is missing (at least defined with an empty string) it results in an error during the loading state:

error="Failed to interpolate string. Is your Secrets.toml correct?"

Please add some clarification to the docs.

Closed in #187


Identify regional and global models

The model names dictionary in https://github.com/pyoceans/utilities/blob/master/utilities/secoora.py#L150 can be improved to provide the information if the model is a regional or global.  That information would help users interpreted small skill scores when a global model is compared to local observations.

That information is useful but I will leave it to the end user.  It is too cumbersome to provide an updated models list.


GPU support for ImplicitRanker

Type of change

[x] New feature (non-breaking change which adds functionality)

Description
Implement use_gpu argument to ImplicitRanker.rank method. Adapt tests to it.
This PR solves #200 and #199.
The GPU support works well but it does not pass all the test normally. The reason is behind implicit's implementation of topk on GPU and is not a bug: if predicted scores are equal, the order of elements is undefined. On CPU it is sorted in an ascending order but on GPU the order seems to be reverse. This limitation fails a lot of tests. Im not sure how to handle this because here are too much test cases to be rewritten without this thing.
Also I should mention changes in _get_neginf_score. On GPU the filtration score is slightly different than on CPU:  https://github.com/benfred/implicit/blob/main/implicit/gpu/knn.cu#L36
I changed the implementation and its tests.
So, questions like interface for user, GPU by default, tests for ranker on GPU keep open and feedback from mainterers will be fine.

Force-pushed with linter fix.

Nice! Thank you for RecTools!


Refactor index.html by Separating CSS and JavaScript into External Files

Currently, the index.html file includes inline CSS and JavaScript code, which can reduce code maintainability and readability. To improve the structure of the project and follow best practices, the inline styles and scripts should be moved to separate external files.
Tasks:

[ ]  Move Inline CSS to an External Stylesheet:

[ ]  Move Inline JavaScript to an External Script File:

@DhanushNehru  can i also work on this issue?

Hi @DhanushNehru , i would like to contribute on this issue


Update CHANGELOG.md for 1.6.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (9b9fc96) 79.08% compared to head (22cb750) 78.58%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #749      +/-   ##
==========================================
- Coverage   79.08%   78.58%   -0.51%
==========================================
  Files          86       86
  Lines        6121     6121
==========================================
- Hits         4841     4810      -31
- Misses       1075     1105      +30
- Partials      205      206       +1

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


Remove trivyignore

Removed from the trivyignore file two issues already fixed in pebble and added a new detected vulnerabilities

It's me or they are the same?

It's me or they are the same?

Ok! They are


react/prop-types: components with props that all have default values fails to validate

Problem
When using the react/prop-types rule when a component has default values for all of its props the rule fails to validate correctly.
Reproduction case
Run it in repl.it/@jvdl/eslint-plugin-react-default-value-prop-validation-bug or clone jvanderloo/eslint-plugin-react-default-value-prop-validation-bug and run locally.
eslint config:
  "rules": {
    "react/prop-types": "error"
  }

Example that does validate as expected:
function Foo({ fooProp1 }) {
  return (
{fooProp1}
);
}

Example that does not validate as expected:
function FooBar({ foobarProp1 = "foo" }) {
  return (
{foobarProp1}
);
}

Expected results
(Assuming a file that has both of the above examples in it)
./example.js
   4:16  error  'fooProp1' is missing in props validation  react/prop-types
  19:19  error  'foobarProp1' is missing in props validation  react/prop-types

Note in the above expected results that both functions fail props validation.
Actual results
./example.js
  4:16  error  'fooProp1' is missing in props validation  react/prop-types

Note that for the function FooBar the props validation passes, even though there is no explicit prop types declaration, only the implicit one by virtue of the types of the default value for the prop.
Additional notes
It appears that this case only occurs when all props have a default value, if only some of the props have one then all props will be properly validated as not having a prop type.
e.g. in the following example only one of the props has a default value
function Bar({ barProp1 = "foo", barProp2 }) {
  return (
{barProp1} and {barProp2}
);
}

This shows the expected result of:
  10:16  error  'barProp1' is missing in props validation  react/prop-types
  10:34  error  'barProp2' is missing in props validation  react/prop-types

I'm not sure what you're asking about. All of these components need explicit .propTypes = added to them, that's the point of this rule.

All of these components need explicit .propTypes = added to them, that's the point of this rule.

Agreed, however when a component has default values defined in the function signature for all of its props, then the rule doesn't validate that component.
For instance, if I have the following code:
function FooBar({ foobarProp1 = "foo" }) {
  return (
{foobarProp1}
);
}

This component will not be validated by react/prop-types despite the lack of an explicit .propTypes = ... definition.
Whereas the following code:
function Foo({ fooProp1 }) {
  return (
{fooProp1}
);
}

function Bar({ barProp1 = "foo", barProp2 }) {
  return (
{barProp1} and {barProp2}
);
}

Will trigger the rule for both components if no explicit propTypes definition is present.

Ah, thanks for clarifying. Indeed, those should both be equally warned upon.


merging two separated materialsCitation FA4FE02FFFDEFFEAFFB7FF9C4151197E

@gsautter In this article, the type locality and the holotype is separated by another paragraph and subSubSection. Is there now a way to merge the two? I remember you mentioned something like that
FA4FE02FFFDEFFEAFFB7FF9C4151197E

I did mention something like that, indeed ... for treatments that are spread out over multiple parts (be it by intermittent treatments of child taxa or because there is additional data in an appendix) ...
While a similar approach should be generally feasible for individual materials citations, things are a good bit more complicated in the latter case: as opposed to treatment annotations, which are groups of whole paragraphs, materials citations exist below the paragraph level, and thus connecting multiple annotations across paragraph boundaries would (at the very least) mean two things:

a really messed up and confusing text flow that retains no resemblance to paragraph boundaries
a deviation from the originally published treatment in terms of the verbatim text, so because paragraph snippets would be moved around when inlining the text streams for the XML

I don't think either effect is desirable, as faithful representation of the treatments is essential for wider acceptance of both TreatmentBank and BLR as reliable sources of primary published text and data.

On the other hand, I do see the need for normalizing occurrence data into individual records ... the best option I see here right now is the following:

mark the (type) locality as such
reference the type locality from the actual materials citations that contain the remaining information, and transfer any location related attributes

The whole approach is yet to be implemented, so no use marking the type locality at this point ... this is merely an idea how we could tackle this type of issue ... and how to automate it is yet another question that most likely requires another good deal of thought once we settle on a general approach.


Not obvious which preps have been added to EBI

Currently is not obvious, in any way, which preps within a study have been submitted to EBI. I think adding a logo beside the name or something like that should be good.

Excellent idea
On Fri, Oct 14, 2016 at 5:12 AM, Antonio Gonzalez<EMAIL_ADDRESS>wrote:

Currently is not obvious, in any way, which preps within a study have been
submitted to EBI. I think adding a logo beside the name or something like
that should be good.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/1971, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB69gInxnbjOxePtqnLHkX1wHGbYuZdyks5qz2O5gaJpZM4KW5k3
.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>


[Feature][Rabbitmq] Allow configuration of queue durability and deletion policy

Queue settings in Rabbitmq connector are now configurable to specify durable,exclusive, and auto-delete options. This enables users to define queue behavior
Check list

[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[x] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[x] Update the release-note.

Thanks @zqr10159 for contribute this! Could you add a test case in https://github.com/apache/seatunnel/blob/862e2055cec6a4428ec4cf67b472ba41ca364d05/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-rabbitmq-e2e/src/test/java/org/apache/seatunnel/e2e/connector/rabbitmq/RabbitmqIT.java#L4 ?

Thanks @zqr10159 for contribute this! Could you add a test case in https://github.com/apache/seatunnel/blob/862e2055cec6a4428ec4cf67b472ba41ca364d05/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-rabbitmq-e2e/src/test/java/org/apache/seatunnel/e2e/connector/rabbitmq/RabbitmqIT.java#L4 ?

Ok，I will add it later.

@Hisoka-X Done!

please fix ci

Sorry, I don't know how to fix this. I created a new branch and rolled back all my code, but still the local test doesn't pass, can you give me some guidance, thank you very much!

Sorry, I don't know how to fix this. I created a new branch and rolled back all my code, but still the local test doesn't pass, can you give me some guidance, thank you very much!

Have you try build package (eg: mvn clean package -DskipTests) before execute e2e test case?

Sorry, I don't know how to fix this. I created a new branch and rolled back all my code, but still the local test doesn't pass, can you give me some guidance, thank you very much!

In my opinion, you should be the window system, which is due to the package not copy to the container, or the script newline problem, you modify the idea newline environment

Sorry, I don't know how to fix this. I created a new branch and rolled back all my code, but still the local test doesn't pass, can you give me some guidance, thank you very much!

In my opinion, you should be the window system, which is due to the package not copy to the container, or the script newline problem, you modify the idea newline environment

Thanks,I'll try it

Could you retry the failed ci? Thanks.

Could you retry the failed ci? Thanks.

The previous local test was a Windows line break issue, the local modification has passed the test, but the github ci seems to have a different error.

Please merge from dev to retrigger it.


Can't run version 8.3

I build a 8.3 version from strach, using php-wasm-builder, and got that browser console error.
PhpWeb.mjs:162 TypeError: Cannot read properties of undefined (reading 'request')
    at PhpWeb._enqueue (PhpWeb.mjs:47:19)
    at PhpWeb.run (PhpBase.mjs:181:15)
    at PhpWeb.<anonymous> (PhpWeb.mjs:160:8)
    at postRun (PhpBase.mjs:42:10)
    at callRuntimeCallbacks (php-web.mjs:9:32725)
    at postRun (php-web.mjs:9:14900)
    at doRun (php-web.mjs:9:699687)
    at run (php-web.mjs:9:699844)
    at runCaller (php-web.mjs:9:698882)
    at removeRunDependency (php-web.mjs:9:15716)

Try running that part I found on README:
<script async type = "module" src =<EMAIL_ADDRESS>
<script type = "text/php" data-stdout = "#output" data-stderr = "#error" data-libs = '[
  {"url": "https://unpkg.com/php-wasm-yaml/php8.3-yaml.so", "ini": true},
  {"url": "https://unpkg.com/php-wasm-yaml/libyaml.so", "ini": false}
]'><?php
  print yaml_emit([1,2,3,"string",["k1" => "value", "k2" => "value2", "k3" => "value3"],"now" => date("Y-m-d h:i:s")]);
</script>

And got same error.
Tried run on Chrome, Brave and Edge.

I just found that navigator.locks.request dont work on insecure connections (like a simple Docker).
Just put a polyfill, and it runs:
<script src="https://unpkg.com/web-locks-polyfill/dist/web-locks-polyfill.js"></script>
Thank you.


getitem and setitem support for structured arrays and scalars

This adds a dedicated IR rewrite pass before type inference, attempting to detect getitem and setitem lookups using compile-time constants and replacing them with dedicated IR expressions/statements.

@sklam: Can you review this?

Looks good.  Thanks!


Creating a database table for the products and sellers

ERD Done
Database Schema Testing Done
Deployment Done
Insertion of Seed Data based on real Local Mechanic Stores and Local Auto Parts Stores Done
Total Time: 7hrs

Total Completion Time: 7 hrs


Release recent gem version

Hi there -- thanks for maintaining the gem -- it really helps smooth out payment integrations.  It seems that Rubygems is still at 2.3.0 from 2016 ( https://rubygems.org/gems/offsite_payments ) while the repo is at 2.5.0.  Are there any plans to release the current and future versions to Rubygems?

Yes, we'll release every version from now on. Cheers
https://rubygems.org/gems/offsite_payments/versions/2.5.0

Sweet - thanks!

2.4.0 was also release https://rubygems.org/gems/offsite_payments/versions/2.4.0


Capitalize list line in working-with-the-apache-maven-registry#authenticating-to-github-packages

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-apache-maven-registry#authenticating-to-github-packages
This issue is intended for newcomers to the OS Docs repo! 💛
If you are veteran of the OS Docs repo and are looking for issues to work on, take a look at our help wanted section ✨
What part(s) of the article would you like to see updated?

To authenticate to a GitHub Packages registry within a GitHub Actions workflow, you can use:

GITHUB_TOKEN to publish packages associated with the workflow repository.
a personal access token (classic) with at least read:packages scope to install packages associated with other private repositories (which GITHUB_TOKEN can't access).

Capitalize the second list line, per our style guide
Additional information
No response

@Piyush-r-bhaskar Thank you so much for opening a PR with this fix! ✨ We are currently in a repo freeze, but we'll get this merged as soon as the freeze ends on Jan 6th 💛


[FEATURE] Add conversion code for headings

Description
We don't have the code to convert headings of HTML to Markdown and vice versa.
Describe the solution you'd like
We can write the code in the existing functions to convert HTML headings to Markdown and vice versa.

Can I work on this issue? I think this could be good starting point for me to contribute

Sure @anandv07 Go ahead


RuntimeError: FAILED: Table is not recovered is faild

mysql  Server version: 5.7.21-log
innobackupex version 2.4.6 Linux (x86_64) (revision id: 54967d1)
root@<node1||/opt/auto-xtrabackup/MySQL-AutoXtraBackup>:#/opt/Python-3.5.3/bin/python3 autoxtrabackup.py -v -lf /data/xtrabackup/autoxtrabackup.log -l DEBUG --defaults_file=/etc/bck.conf --partial
2018-05-21 13:35:18 DEBUG    <pid.PidFile object at 0x2b62617f24a8> entering setup
2018-05-21 13:35:18 DEBUG    <pid.PidFile object at 0x2b62617f24a8> create pidfile: /tmp/autoxtrabackup.py.pid
2018-05-21 13:35:18 DEBUG    <pid.PidFile object at 0x2b62617f24a8> check pidfile: /tmp/autoxtrabackup.py.pid
Type Database name: zx_election
Type Table name: vote_info
2018-05-21 13:35:45 DEBUG    Running mysqladmin command -> /usr/local/mysql/bin/mysqladmin --defaults-file=/etc/my.cnf --user=root --password=**** status --host=<IP_ADDRESS> --port=3306
2018-05-21 13:35:45 DEBUG    OK: Server is Up and running
2018-05-21 13:35:45 DEBUG    Checking if innodb_file_per_table is enabled
2018-05-21 13:35:46 ERROR    FAILED: InnoDB file per-table Check
2018-05-21 13:35:46 ERROR    mysql: [Warning] Using a password on the command line interface can be insecure.
@mysql: [ERROR] Unknown suffix '.' used for variable 'port' (value '<IP_ADDRESS>')
mysql: [ERROR] /usr/local/mysql/bin/mysql: Error while setting value '<IP_ADDRESS>' to 'port'
2018-05-21 13:35:46 ERROR    FAILED: Table is not recovered
2018-05-21 13:35:46 ERROR    FAILED: InnoDB file per-table Check
2018-05-21 13:35:46 DEBUG    <pid.PidFile object at 0x2b62617f24a8> closing pidfile: /tmp/autoxtrabackup.py.pid
Traceback (most recent call last):
File "/opt/auto-xtrabackup/MySQL-AutoXtraBackup/partial_recovery/partial.py", line 359, in final_actions
self.check_innodb_file_per_table()
File "/opt/auto-xtrabackup/MySQL-AutoXtraBackup/partial_recovery/partial.py", line 71, in check_innodb_file_per_table
raise RuntimeError("FAILED: InnoDB file per-table Check")
RuntimeError: FAILED: InnoDB file per-table Check
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "autoxtrabackup.py", line 306, in
all_procedure()
File "/opt/Python-3.5.3/lib/python3.5/site-packages/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/opt/Python-3.5.3/lib/python3.5/site-packages/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/opt/Python-3.5.3/lib/python3.5/site-packages/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/opt/Python-3.5.3/lib/python3.5/site-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/opt/Python-3.5.3/lib/python3.5/site-packages/click/decorators.py", line 17, in new_func
return f(get_current_context(), *args, **kwargs)
File "autoxtrabackup.py", line 276, in all_procedure
c.final_actions()
File "/opt/auto-xtrabackup/MySQL-AutoXtraBackup/partial_recovery/partial.py", line 376, in final_actions
raise RuntimeError("FAILED: Table is not recovered")
RuntimeError: FAILED: Table is not recovered

Hi and thanks for report.
It seems to be mysql connection issue here:
/usr/local/mysql/bin/mysql: Error while setting value '<IP_ADDRESS>' to 'port'
Could you check if something wrong with config?

Thank you for your response!
This partial.py script maybe  problematic！
if  true   but    else  running!
else:
logger.error("FAILED: InnoDB file per-table Check")
logger.error(output)
raise RuntimeError("FAILED: InnoDB file per-table Check")

@weiyanwei412 please submit a pull request if you have code suggestions, improvements.
All kinds of help are appreciated.

Closing as there is no activity here.


Can't build on Windows due to nowhere to download Visual Studio 2008

The Windows build instructions tell you that VS Studio 2008 is needed, which MS no longer provides downloads for. Is there any chance that this project could be moved to Visual Studio 2015 for its solution, or even 2017?

Oh, thank you. The only download I had found was a .pptx file on their site. Are there any plans to move to a newer Visual Studio project file? Newer versions still support .NET 3.5

From https://github.com/SoftEtherVPN/SoftEtherVPN/wiki/SoftEther-VPN-Developers-Guideline-(DRAFT):

MUST support Microsoft Visual C++ 2008. Microsoft Visual C++ 2008 is the latest version to support Windows 9x and Windows NT 4.x binary linking. It is OK to add new Visual Studio's solution files (e.g. for Visual Studio 2017). In such a case, there must be dual solution files for both Visual C++ 2008 and the latest Visual Studio.

I'm working on adding Windows support to our CMake configuration.

That info seem to be good to add to BUILD_WINDOWS.MD
@BlackHoleFox , can you take care of that?

@chipitsine Assuming that you mean details on what C++ versions it must support and that a CMake config for windows is coming, sure.

With the merging of #588, this can be closed now right?


replace the word install with require as thats the keyword for composer

Signed-off-by: Victor Bala<EMAIL_ADDRESS>

This has been fixed. We appreciate the assist. Thanks


issue61 - adding a flag to turn off velocity logging for eclipse use

Signed-off-by: Brian Fitzpatrick<EMAIL_ADDRESS>

ok to test


Extension.EnteredInitialState has context.State == null

This prevents getting the IState<TState, TEvent>  of the initial status.
Example extension code:
private class CurrentStatusExtension : AsyncExtensionBase<States, Triggers>
{
	private readonly AppccelerateTestData _instance;

	public CurrentStatusExtension(AppccelerateTestData instance)
	{
		_instance = instance;
	}

	public override void EnteredInitialState(
		IStateMachineInformation<States, Triggers> stateMachine,
		States state,
		ITransitionContext<States, Triggers> context)
	{
		// context.State is null ... so there is no way to get the initial state?
		_instance.CurrentStatus = context.State;
	}
}

Use the state passed to the method directly. The state of the context is the source state, which is null in case of initialization.
Does this work for you?

No, sorry, I need the IState, because I need to get the valid triggers.

Maybe it might be useful to add a property containing the target state? My assumption was that the context state always contains the target state for all transitioned/initialized events.


Steps and colonized planet displayed wrongly

Use Tokens instead?

Interesting. Try full refresh? Using tokens might work too...

Full refresh usually works. I will take the update to token as my svg-vue exam ;)

@coyotte508 what's the best approach: try to have it in the current component or create a new one?

I would add <Token> to the current component. But you might have trouble with line height, alignment and all that.

Solved by 43aed36dac440620f0664c7a3f8ddd08bf5b501e


Update gcp plugin changelogs with v0.5.0 changes

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind documentation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

/area plugins
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: pirrofra
Once this PR has been reviewed and has the lgtm label, please assign ahmedameenaim for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

plugins/gcpaudit/OWNERS

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

LGTM label has been added.  Git tree hash: b754e9608d3eef7f55f579d5cd138f20e1888fa2

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, pirrofra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

plugins/gcpaudit/OWNERS [leogr]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra, pirrofra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

plugins/gcpaudit/OWNERS [LucaGuerra,leogr]

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


Panic on import "github.com/aws/aws-sdk-go-v2/aws"

repl panics when I try to import "github.com/aws/aws-sdk-go-v2/aws":
{/User/kaixia} go get -u "github.com/aws/aws-sdk-go-v2/aws"
{/User/kaixia} yaegi
> import "github.com/aws/aws-sdk-go-v2/aws"
panic: reflect.StructOf: field 0 has no type

goroutine 1 [running]:
reflect.StructOf(0xc00019e820, 0x3, 0x4, 0x0, 0x0)
        /usr/local/Cellar/go/1.12.6/libexec/src/reflect/type.go:2369 +0x357c
github.com/containous/yaegi/interp.(*itype).TypeOf(0xc000292680, 0xc000265200, 0xc0001ca360)
        /Users/kaixia/.xiaket/go/src/github.com/containous/yaegi/interp/type.go:720 +0x788
github.com/containous/yaegi/interp.(*itype).TypeOf(0xc000292d00, 0x1, 0x1)
        /Users/kaixia/.xiaket/go/src/github.com/containous/yaegi/interp/type.go:713 +0x8c7
github.com/containous/yaegi/interp.(*itype).frameType(0xc000292d00, 0x11d0daa, 0xc000265380)
        /Users/kaixia/.xiaket/go/src/github.com/containous/yaegi/interp/type.go:761 +0x13f
github.com/containous/yaegi/interp.(*scope).add(0xc000265380, 0xc000292d00, 0x0)
(more traceback omitted.)

As you can see, in line 1, I've downloaded the package to local GOPATH.
The interpreter is faster than I've expected in many cases, great work!

There is a bug in yaegi when parsing the aws package or one of its dependencies (if any). We will investigate. Otherwise, thank you for your nice comment about the interpreter speed.

One more question here, will yasgi support downloading dependencies defined in go.mod/Gopkg.toml?

@jiacai2050 I think it has already been raised in #259

As far as I can tell, the original bug is now fixed, but we now get this:
> import "github.com/aws/aws-sdk-go-v2/aws"
1:21: import "github.com/aws/aws-sdk-go-v2/aws" error: /Users/mpl/src/github.com/aws/aws-sdk-go-v2/aws/chain_provider.go:5:2: import "github.com/aws/aws-sdk-go-v2/aws/awserr" error: /Users/mpl/src/github.com/aws/aws-sdk-go-v2/aws/awserr/types.go:157:23: cannot use type github.com/aws/aws-sdk-go-v2/aws/awserr.awsError as type error

so we might as well keep the issue open for now.

As per v0.12.0, I have the following:
> import import "github.com/aws/aws-sdk-go-v2/aws"
: 0x14000112130

Not tested further, but at least the initial import works.


Memory and documentation Improvements

Added System.Memory to improve performance
Updated documentation for Exeptions

Unit test didn't worked


Workflows gallery

Create a workflow gallery in domino-docs repository.
This gallery can use as source of data files from the same repository as pieces gallery.
Our goal here is to provide the users a way to find nice workflows in our documentation and import them directly from there.

done : https://domino-workflows.io/gallery

@nathan-vm this is pointing to the Pieces gallery, do we a have a gallery for workflows already in place?


Support complex length_dims in data.

Summary
This PR starts to support complex length_dims in the preprocessing of data.

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

audyn/utils/data/dataset.py
97.14% <100.00%> (+65.71%)
:arrow_up:

audyn/utils/data/sampler.py
83.54% <66.66%> (+36.79%)
:arrow_up:

audyn/utils/data/postprocess.py
89.18% <87.09%> (+74.90%)
:arrow_up:

... and 15 files with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.


Windows 11 Detection

Debug info

OS: Windows 11 Pro
Uptime: 19 minutes and 36 seconds
Local IP: <IP_ADDRESS>
Motherboard: Apple Inc. (Mac-77F17D7DA9285301)
CPU: Intel(R) Core(TM) i5-7400 CPU @ 3.00GHz
GPU: Radeon Pro 555
Memory: 5.4GiB / 7.9GiB (68.17%)
Disk: C: 77.4GiB / 110.6GiB (69.99%)
D: 2.1TiB / 3.6TiB (58.28%)
[KeyError]: 'Font:'[KeyError]: 'Font:'----run_rpc----
uptime in epoch:<PHONE_NUMBER>.740357
Unknown distro, contact us on github to resolve this.
cpuid:<PHONE_NUMBER>43692821
cpuappid:<PHONE_NUMBER>98926920
----out.py----
----DE/WM----
----GPU INFO----
gpuinfo: Radeon Pro 555
gpuvendor: amd
----CPU INFO----
cpuvendor: intel
cpumodel: Intel i5
cpuinfo: Intel(R) Core(TM) i5-7400 CPU @ 3.00GHz
cpuline item 0: Intel(R) Core(TM) i5-7400 CPU @ 3.00GHz
memline: 5.4GiB / 7.9GiB (68.17%)
----OS INFO----
sysosline: Windows 11 Pro
Unknown distro, contact us on github to resolve this.
sysosid:<PHONE_NUMBER>43692821
diskline: C: 77.4GiB / 110.6GiB (69.99%)
----testing.py----
Unknown motherboard, contact us on github to resolve this.
moboid:<PHONE_NUMBER>79197462
moboline: Apple Inc. (Mac-77F17D7DA9285301)
----GPU INFO----
gpuvendor: amd
----CPU INFO----
cpumodel: Intel i5
Operating system & way of installation
Windows 11 Pro Build 22000.65 // pip
Error/issue
Fetchcord thinks Windows 11 is linux. Sorta. It shows it's name fine, but the first line is "linux" with the linux logo as the big image.

Thats a bit funny, ill be sure to add windows 11 support in the coming days

The funny thing he has a good cpu to run windows 11 i cant even run it

My CPU actually isn't in the compatibility list lmfao

Thats weird ive been hearing a lot of complaints on windows 11 because of the compatibility

Glad this is getting fixed <3


messages not going through

Since yesterday AMCDB has been sending this error whenever someone sends a message in minecraft chat. I think it is caused by something outside of the server as it is doing it on completely fresh servers.
I don't know if Issues is the right place for this but I can't find anywhere else for it.
[22:34:36] [Server thread/ERROR]: Chain link failed, continuing to next one
java.lang.NoSuchMethodError: 'net.minecraft.class_2561 net.minecraft.class_7471.method_44125()'
        at network.parthenon.amcdb.minecraft.InGameMessageHandler.handleChatMessage(InGameMessageHandler.java:45) ~[amcdb-1.1.7.jar:?]
        at net.fabricmc.fabric.api.message.v1.ServerMessageEvents.lambda$static$6(ServerMessageEvents.java:103) ~[fabric-message-api-v1-5.1.7+3265161977-c3ab6ca4ce6148ea.jar:?]
        at net.minecraft.class_3324.handler$zdd000$fabric-message-api-v1$onSendChatMessage(class_3324.java:2035) ~[server-intermediary.jar:?]
        at net.minecraft.class_3324.method_43673(class_3324.java) ~[server-intermediary.jar:?]
        at net.minecraft.class_3244.method_44155(class_3244.java:1429) ~[server-intermediary.jar:?]
        at net.minecraft.class_3244.method_45064(class_3244.java:1303) ~[server-intermediary.jar:?]
        at java.util.concurrent.CompletableFuture$UniAccept.tryFire(CompletableFuture.java:718) ~[?:?]
        at java.util.concurrent.CompletableFuture$Completion.run(CompletableFuture.java:482) ~[?:?]
        at net.minecraft.class_3738.run(class_3738.java:18) ~[server-intermediary.jar:?]
        at net.minecraft.class_1255.method_18859(class_1255.java:156) ~[server-intermediary.jar:?]
        at net.minecraft.class_4093.method_18859(class_4093.java:23) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_24306(MinecraftServer.java:782) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_18859(MinecraftServer.java:164) ~[server-intermediary.jar:?]
        at net.minecraft.class_1255.method_16075(class_1255.java:130) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_20415(MinecraftServer.java:764) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_16075(MinecraftServer.java:758) ~[server-intermediary.jar:?]
        at net.minecraft.class_1255.method_5383(class_1255.java:115) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_16208(MinecraftServer.java:742) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:675) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:265) ~[server-intermediary.jar:?]
        at java.lang.Thread.run(Thread.java:833) ~[?:?]

Issues is exactly the right place!
Hmm, what version of Minecraft are you running? Can you try grabbing the latest Fabric loader and Fabric API?

I'm running 1.20.1, and I used the latest versions of both.

okay, I was able to reproduce this. It seems to be some type of mapping incompatibility introduced when I attempted to set the required Minecraft version down to 1.19.2.
I'll work on this, but in the meantime release 1.1.6 doesn't seem to have this problem and is nearly identical.

ah okay, thank you


Make the code available as library code in a lib.rs

It would be useful to export some key functions, so it can be easily reused by users.
Also to publish the package on cargo.
Thank you

Thanks for your suggestion. I will do that for sure.


New setting: "Automatically open used words in shared results if I've already won a daily challenge"

Autoopen was added without a setting because, even if we don't want to spoil the results for others looking at our screen, the finished game is displayed in the background anyway.


feat: support legacy aggregator and sequence sender

Description

In #193, we introduced cdk-node as the default aggregator / sequence sender component for the CDK stack. In this effort, we removed support for the legacy zkevm aggregator and sequence sender unfortunately. However, this is something we still need to test.
This PR introduces a new flag aggregator_sequence_sender_type to specify the type of aggregator and sequence sender to deploy. There are two options:

cdk: Use the new cdk-node
zkevm: Use the legacy aggregator and sequence sender

The diagram on the left describes what kurtosis-cdk currently deploys in the central environment, and the diagram on the right represents what kurtosis-cdk will be able to deploy with this new change.

The zkevm job of the deploy workflows tests this change by using the zkevm-node for the aggregator and sequence sender of the first CDK L2 chain. The other jobs such as cdk use the cdk-node for the aggregator and sequence sender.
https://github.com/0xPolygon/kurtosis-cdk/actions/runs/10493040767/job/29066006689?pr=234#step:4:4
References (if applicable)

https://0xpolygon.slack.com/archives/C06M06XR5A4/p1724233695796189

Tested locally and lgtm. Might be worth noting somewhere that the legacy configs may not work with CDK-Erigon configs. e.g. aggregator_sequence_sender_type and sequencer_type had to be both zkevm to work on my test.

I think we should aim to cover scenarios 1, 2, 4, 7 and 8. I updated the CI jobs to test that.

This PR also cleans up some parts of the CDK central environment (e.g. how sequencer is deployed, how aggregator / sequence sender is deployed, etc.). The central RPC deployed will be of the same type as the sequencer. For example, if sequencer_type is equal to erigon, it will automatically deploy a cdk-erigon-node rpc. Same thing if sequencer_type is equal to zkevm,  it will deploy a zkevm-node rpc.

The legacy stack with new sequencer (cdk-erigon) does not work well right now. The deployment works but batches are not being virtualized nor verified. There is a problem with the zkevm-node sequence sender.

Update before I leave for a few days: every scenario work except the one with cdk-erigon as the sequence and the new zkevm components (zkevm-sequence-sender and zkevm-aggregator). Batches are being virtualized but not verified. I tried to bump the images but it looks like there is still an issue with the aggregator.
https://github.com/0xPolygon/kurtosis-cdk/actions/runs/10601202150/job/29380486728?pr=234#step:6:1654


Permission issue on OpenShift during sync using devspace dev

What happened?
We are migrating to OpenShift from a vanilla K8s where we are using DevSpace. We have tweaked the Dockerfile.devspace to use a non-root user. Build and deployment works perfectly but fails on initial sync as DevSpace tries to change the permission (chmod & chtimes).
Related issue: #1898
What did you expect to happen instead?
We should be able to remote and sync normally as on the vanilla K8s cluster.
How can we reproduce the bug? (as minimally and precisely as possible)
My devspace.yaml:
version: v1beta10
...

Using devspace dev we have this:
[0:sync] Downstream - Initial sync completed
[0:sync] Helper - Error touching Path:"/MyApp/foo-bar.cs" MtimeUnix:1647865271 Mode:511: chtimes /app/src/MyApp/foo-bar.cs: operation not permitted
[0:sync] Helper - Error chmod Path:"/MyApp/foo-bar.cs" MtimeUnix:1647865271 Mode:511: chmod /app/src/MyApp/foo-bar.cs: operation not permitted
... same for all files
[0:sync] Upstream - Upload 7 create change(s) (Uncompressed ~20.75 KB)
[0:sync] Upstream - Successfully processed 7 change(s)
[0:sync] Upstream - Restarting container
[0:sync] Upstream - Initial sync completed
[info]   Opening 'http://localhost:5000' as soon as application will be started (timeout: 4m0s)
[info]   Starting log streaming
[myapp-username:myapp] Start streaming logs for ns-devspace/myapp-username-7c7b9774b9-rjj5r/myapp
[myapp-username:myapp] ############### Restart container ###############
[myapp-username:myapp]
[myapp-username:myapp] Killed
[myapp-username:myapp] /usr/share/dotnet/sdk/6.0.202/Microsoft.Common.CurrentVersion.targets(4830,5): error MSB3374: The last access/last write time on file "/app/src/MyApp/obj/Debug/net6.0/MyProject.csproj.CopyComplete" cannot be set. Access to the path '/app/src/MyProject/obj/Debug/net6.0/MyProject.csproj.CopyComplete' is denied. [/app/src/MyProject/MyProject.csproj]
[myapp-username:myapp]
[myapp-username:myapp] The build failed. Fix the build errors and run again.
Local Environment:

DevSpace Version: 5.18.4
Operating System: WSL Ubuntu 18.04
ARCH of the OS: AMD64
Kubernetes Cluster:
Cloud Provider: OpenShift 4.9.23 on prem
Kubernetes Version: 1.22

Anything else we need to know?
Do you have any hints or guides to use it on RHOS? What user DevSpace is trying to perform the actions? We have a non-root user with permission 775 to the /app folder.

/kind bug

@tompatib thanks for creating this issue! DevSpace uses the same user as the container has specified for syncing files and folders. Its essentially taring the local files and folders, tries to keep the user and permissions locally and then applies them in the container. Usually we recommend during development to use root as a user, but I believe OpenShift is not permitting that. Doeskubectl cp work for those files, because thats essentially what DevSpace is doing behind the scenes.

Is there a way to get around this currently as OpenShift does not support root containers?

@FabianKramm, we could make this work. Checking on your suggestion the kubectl cp was working as expected, but it always failed when DevSpace tried to do the chmod and chtimes command.
As OpenShift doesn't allow root user on the containers we have defined an appuser but the default UID (1000) didn't match with the UID range of the namespace (project in RHOS), it was like 1000XX0000.
To solve this:

we have changed the UID range on the project (to support alpine images - not necessary otherwise)

explicitly defined the UID for the appuser in the DevSpace's dockerfile

gave the same permission for the source folder

When everything is right on the deployed pod you can see this:

This issue is good to close from my side.


add spatial controller (mixed reality) to primary button map

Adds support for Mixed Reality hand controller in PRIMARY_BUTTON_MAP, setting the trigger as the primary button.

Thanks for this, @jfcampos!


Duplicate package "util" (shared with connectors)

@dsyer you mentioned moving things into commons package. Could we move util out of cloud into cloud.client.

By my estimate it's not just client stuff though is it? We could move only the util package to commons?


calc function missing

Hi. I really like your work with these svg.js plugins! It seems that the version of svg.draw.js here is missing a calc prototype function. I can use the minified version from your demo page and it works fine.
Thanks.

Hey, are you using the version from the dist directory? The calc function is a dynamic function which is generated on runtime.
What is the error you are getting?

Sorry for the delay. I updated again from the dist package and am no longer seeing the calc error. I apologize if I used the wrong process here. I am new to GitHub.

I'm still not able to run the demo properly though. Getting 'rect.draw' is not a function. I have loaded svg.js and svg.draw.js from your zip in the demo page.
I have:  in my markup and on page load I run this:
var canvas = new SVG('rect').size('100%', '100%')
canvas.rect.draw();
I see the first line creates the SVG element within my div but the second line give the error.

Never mind. I got working. Thanks.


Expose the error/cause messages in case of error

I don't know if we need to open an issue just for this very small change

Merged, thanks

Thank you @tristantarrant


regarding iterations within each epoch and dropout setting

Hi Marko,
Regarding the the main optimization step model.fit(imgs_train, imgs_mask_train, batch_size=32, nb_epoch=20, verbose=1, shuffle=True,               callbacks=[model_checkpoint])
How many iterations within each epoch will it be included? Here, we have about 5635 pairs of training images and training mask images. If batch_size=32, should it be 5635/32 iterations?
I did not find a place where this parameter of training_iters are setup. Will it be handled automatically by model.fit?
Besides, how do you setup the dropout rate in this training process? Will it be helpful?
Thank you very much.

Hi,
You are correct, Keras model.fit() handles the number of iterations automatically (based on number of epochs, size of the training set and batch size).
Dropout can be added with Dropout layer (check https://keras.io/layers/core/#dropout), the use is the same like any other layer. Vanilla dropout after convolutional layers is not something that is often done in practice, some papers suggest using spatial dropout, available in Keras via SpatialDropout2D layer. I haven't used it for this particular problem though.
Hope this helps,
Marko

Hi Marko,
Thanks for the reply.


Rename file refactoring doesn't work in macOS

Does this issue occur when all extensions are disabled?: Yes, except the extension "C# for Visual Studio Code (powered by OmniSharp)" which enables functionality of "Rename file to".

VS Code Version: 1.63.2
OS Version: macOs Catalina 10.15.7
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 19.6.0
Project: .NET 6

Steps to Reproduce:

Create class with the same file name
Rename class name
Open refactoring offer -> "Rename file to" and get the error

/extC#


Erorr with intellj and android studio when i need to creat or run app ,firstly when i creat new app dart tool give me red

java.lang.Exception: pub activate of DevTools failed
	at io.flutter.run.daemon.DevToolsService.logExceptionAndComplete(DevToolsService.java:261)
	at io.flutter.run.daemon.DevToolsService.lambda$setUpWithPub$4(DevToolsService.java:177)
	at java.base/java.util.concurrent.CompletableFuture.uniAcceptNow(CompletableFuture.java:753)
	at java.base/java.util.concurrent.CompletableFuture.uniAcceptStage(CompletableFuture.java:731)
	at java.base/java.util.concurrent.CompletableFuture.thenAccept(CompletableFuture.java:2108)
	at io.flutter.run.daemon.DevToolsService.setUpWithPub(DevToolsService.java:173)
	at io.flutter.run.daemon.DevToolsService.lambda$startServer$2(DevToolsService.java:116)
	at com.intellij.util.RunnableCallable.call(RunnableCallable.java:20)
	at com.intellij.util.RunnableCallable.call(RunnableCallable.java:11)
	at com.intellij.openapi.application.impl.ApplicationImpl$1.call(ApplicationImpl.java:270)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1$1.run(Executors.java:668)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1$1.run(Executors.java:665)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1.run(Executors.java:665)
	at java.base/java.lang.Thread.run(Thread.java:834)

@AhmebashaAnter
Check this solution and see if it helps.

class jdk.internal.loader.ClassLoaders$PlatformClassLoader cannot be cast to class java.net.URLClassLoader (jdk.internal.loader.ClassLoaders$PlatformClassLoader and java.net.URLClassLoader are in module java.base of loader 'bootstrap')
class jdk.internal.loader.ClassLoaders$PlatformClassLoader cannot be cast to class java.net.URLClassLoader (jdk.internal.loader.ClassLoaders$PlatformClassLoader and java.net.URLClassLoader are in module java.base of loader 'bootstrap')
Gradle's dependency cache may be corrupt (this sometimes occurs after a network connection timeout.)
Re-download dependencies and sync project (requires network)
The state of a Gradle build process (daemon) may be corrupt. Stopping all Gradle daemons may solve this problem.
Stop Gradle build processes (requires restart)
Your project may be using a third-party plugin which is not compatible with the other plugins in the project or the version of Gradle requested by the project.
In the case of corrupt Gradle processes, you can also try closing the IDE and then killing all Java processes.

Did you try the solution link I shared earlier ?
Can you also try solutions mentioned in this SO link and see if it works ?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Facebook Video Embed

Hello, would be awesome to support Facebook video in the default sources in embed.
Thanks,
Kris

Hi @krishaamer
Hope this will solve your issue.

Don't forget to include SDK code in body
`

`
Demo