Doesnt import visions under senses.

I am using Foundry v11.313 tried to import a statblock, it worked perfect other than the vision under senses.
I tired your example "Glabrezu" aswell and it didnt have the truesight in the character sheet.

This should be fixed in the latest version (1.81.9) per #85, please update and test again

I have the same issue. I'm using v.1.82.00. It also puts entries that should be Features into the Actions section. Also, it doesn't populate the speed.
Orc Berserker
Orc, Lawful Evil
Armor Class 15 (Chain shirt)
Hit Points 90 (12d8 +36)
Speed 30ft
Proficiency +2
STR
DEX
CON
INT
WIS
CHA
20 (+5) 16 (+3) 17 (+3) 8 (-1) 15 (+2) 14 (+2)
Skills Intimidation +4 Athletics +7 Animal Handling +4 Survival +4 Nature +1
Saving Throws Strength +7 Constitution +5
Senses darkvision 60ft.
Languages Common, Orc
Challenge 4 (1100 XP)
Actions
Multiattack. The Berserker makes three weapon melee attacks or three ranged attacks.
Greataxe. Melee Weapon Attack +7 to hit, reach 5 ft, one target. Hit 11 (1d12 +5 ) slashing damage, Properties Heavy, Two-handed.
Javelin. Melee Weapon Attack +7 to hit, range 30/120ft, one target. Hit 8 (1d6 +5 ) piercing damage, Properties Thrown range 30/120ft.
Features
Aggressive. As a bonus action, the creature can move up to its speed toward a hostile creature that it can see.
Menacing. You have proficiency in the intimidate skill. (even if DMG didn't say so, all the orcs on MM got intimidation)
Indomitable. The berserker has advantage on saving throws against being frightened.
You are coming with me. When the berserker drops to 0 hit points he can try to make one melee weapon attack as a reaction to one enemy at 5ft of him.

@meta5by5 Just curious, what is the source of this statblock?
There's a lot of formatting that is out of the ordinary
There's no Size listed (the second line should start with Medium) and there should be a space in between the distance and units for the speed (e.g. 30 ft)
As for the individual spells, all of them look to be importing fine to me, except some of them in the provided statblock are inaccurate (Counterspell and Stoneskin are one word)
I do not see the spell slots being added for 4th level, the colon is needed after the parenthetical
There should not be a "Features" section, typically features are just placed after the creature data (e.g. after challenge)
The feature You are coming with me does get appended to the previous feature because the feature name is just too long, the max is 4 words otherwise it's assumed to be a sentence, it's hard to determine what the right cutoff is there, but most standard creatures dont have long feature names.
Aggressive should be in a Bonus Actions section
Weapon damages arent being set because of space prior to the closing parenthesis
I do see that spellcaster level is not being set, and it is trying to capture some of the xp text as a flee mortals role, but other than that once it's properly formatted it looks fine
Here's a cleaned up version of the statblock that imports well
Orc Berserker
Medium Humanoid (Orc), Lawful Evil
Armor Class 15 (Chain shirt)
Hit Points 90 (12d8 +36)
Speed 30 ft
Proficiency +2
STR
DEX
CON
INT
WIS
CHA
20 (+5) 16 (+3) 17 (+3) 8 (-1) 15 (+2) 14 (+2)
Skills Intimidation +4 Athletics +7 Animal Handling +4 Survival +4 Nature +1
Saving Throws STR +7 CON +5
Senses darkvision 60ft.
Languages Common, Orc
Challenge 4 (1100 XP)
Menacing. You have proficiency in the intimidate skill.
Indomitable. The berserker has advantage on saving throws against being frightened.
You are coming with. me. When the berserker drops to 0 hit points he can try to make one melee weapon attack as a reaction to one enemy at 5ft of him.
Spellcasting. The archmage is an 18th-level spellcaster. Its spellcasting ability is Intelligence (spell save DC 17, +9 to hit with spell attacks). The archmage can cast disguise self and invisibility at will and has the following wizard spells prepared:

Cantrips: fire bolt,light,mage hand,Prestidigitation,Shocking Grasp
1st level: detect magic, identify, mage armor, Magic Missile
2nd level: Detect Thoughts, Mirror Image, Misty Step
3rd level: counterspell, Fly, lightning bolt
4th level (3 slots): Banishment,Fire Shield,Stoneskin
Actions
Multiattack. The Berserker makes three weapon melee attacks or three ranged attacks.
Greataxe. Melee Weapon Attack +7 to hit, reach 5 ft, one target. Hit 11 (1d12 + 5) slashing damage, Properties Heavy, Two-handed.
Javelin. Melee Weapon Attack +7 to hit, range 30/120ft, one target. Hit 8 (1d6 + 5) piercing damage, Properties Thrown range 30/120ft.
Bonus Actions
Aggressive. As a bonus action, the creature can move up to its speed toward a hostile creature that it can see.

I sourced it from rpgTinker at first and then did some (very) heavy
editing. I rearranged a lot and tried variations (like your point about
Counterspell as two words at first, and then one word).
Thanks for the guidance. I'm trying to find a way to build custom
NPCs/monsters that calculate HP, damage, CR and such without building
everything from scratch.
On Sun, Feb 4, 2024 at 5:26 PM Max.Pat @.***> wrote:

@meta5by5 https://github.com/meta5by5 Just curious, what is the source
of this statblock?
There's a lot of formatting that is out of the ordinary
There's no Size listed (the second line should start with Medium) and
there should be a space in between the distance and units for the speed
(e.g. 30 ft)
As for the individual spells, all of them look to be importing fine to me,
except some of them in the provided statblock are inaccurate (Counterspell
and Stoneskin are one word)
I do not see the spell slots being added for 4th level, the colon is
needed after the parenthetical
There should not be a "Features" section, typically features are just
placed after the creature data (e.g. after challenge)
The feature You are coming with me does get appended to the previous
feature because the feature name is just too long, the max is 4 words
otherwise it's assumed to be a sentence, it's hard to determine what the
right cutoff is there, but most standard creatures dont have long feature
names.
Aggressive should be in a Bonus Actions section
Weapon damages arent being set because of space prior to the closing
parenthesis
I do see that spellcaster level is not being set, and it is trying to
capture some of the xp text as a flee mortals role, but other than that
once it's properly formatted it looks fine
Here's a cleaned up version of the statblock that imports well
Orc Berserker
Medium Humanoid (Orc), Lawful Evil
Armor Class 15 (Chain shirt)
Hit Points 90 (12d8 +36)
Speed 30 ft
Proficiency +2
STR
DEX
CON
INT
WIS
CHA
20 (+5) 16 (+3) 17 (+3) 8 (-1) 15 (+2) 14 (+2)
Skills Intimidation +4 Athletics +7 Animal Handling +4 Survival +4 Nature +1
Saving Throws STR +7 CON +5
Senses darkvision 60ft.
Languages Common, Orc
Challenge 4 (1100 XP)
Menacing. You have proficiency in the intimidate skill.
Indomitable. The berserker has advantage on saving throws against being frightened.
You are coming with. me. When the berserker drops to 0 hit points he can try to make one melee weapon attack as a reaction to one enemy at 5ft of him.
Spellcasting. The archmage is an 18th-level spellcaster. Its spellcasting ability is Intelligence (spell save DC 17, +9 to hit with spell attacks). The archmage can cast disguise self and invisibility at will and has the following wizard spells prepared:
Cantrips: fire bolt,light,mage hand,Prestidigitation,Shocking Grasp
1st level: detect magic, identify, mage armor, Magic Missile
2nd level: Detect Thoughts, Mirror Image, Misty Step
3rd level: counterspell, Fly, lightning bolt
4th level (3 slots): Banishment,Fire Shield,Stoneskin
Actions
Multiattack. The Berserker makes three weapon melee attacks or three ranged attacks.
Greataxe. Melee Weapon Attack +7 to hit, reach 5 ft, one target. Hit 11 (1d12 + 5) slashing damage, Properties Heavy, Two-handed.
Javelin. Melee Weapon Attack +7 to hit, range 30/120ft, one target. Hit 8 (1d6 + 5) piercing damage, Properties Thrown range 30/120ft.
Bonus Actions
Aggressive. As a bonus action, the creature can move up to its speed toward a hostile creature that it can see.
—
Reply to this email directly, view it on GitHub
https://github.com/jbhaywood/5e-statblock-importer/issues/86#issuecomment-1925973055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXGPHCLHA4T3GEAK7LSYA5LYSAKKNAVCNFSM6AAAAABBWFL2OCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRVHE3TGMBVGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Camera wiring harness should be able to use pin 11 OR 12 for SWCAN

Some variants of Equinox use pin 12 instead of 11 for the SWGMLAN.

Confirmed through service manuals that most global-A cameras have two unused pins; however Global-B seems to use all pins. This means we will still need 2 extra wires.
(Note that the serial data comm enable cannot be used for IGN and must be passed through)


Add Jenkinsfile.deploy for daily builds

I guess we should change the version of project too or not? as we release 0.x and it is 8.0.0-SNAPSHOT we should synchronize it. WDYT?

@mareknovotny good point. but will address that in another PR.

@mareknovotny @radtriste updated

@mareknovotny I started a thread on Kogito dev list to discuss the versioning. Also, do we need to do something for Jenkins to load this new job?

I don't think we need to load it, I saw already it is on there
great so I will read and respond if needed, thanks!


Normalize won't remove dotted outline from links

This behaviour differs for Chrome and Firefox;
Wherein clicking and dragging on Firefox will result in link (<a> tag) being surrounded by dotted box, on chrome said link will not be marked in any way
Please consider adding to normalize.css@288 another line, to match:
::moz-focus-inner,
button::-moz-focus-inner,
[type="button"]::-moz-focus-inner,
[type="reset"]::-moz-focus-inner,
[type="submit"]::-moz-focus-inner {
  border-style: none;
  padding: 0;
}

This is accessibility related. I would not mess around with anything accessibility related, and especially not by removing the styling. The same is true for default :focus styles which differ from browser to browser (and that is not necessarily a bad thing).

FTR #577 and probably a lot more

Understood


Facing problem while freezing graph for deployment

I tried freezing the checkpoints with "Reshape_1:0" as an output Tensor, but after converting into ProtoBuffer I am unable to generate an inference from it.
What should be right input and output nodes for the Frozen graph?

I have the same problem, did you find a solution?


Fix Latex format in mining article

@tuanhvdev can you review?

I missed to fix the link to the beginner pdf


FIX: Adding 2 adapters when target_modules is a str fails

Problem description
Adding two adapters (e.g. LoRA) when using a list for target_mdules works but passing a str fails. The issue is that for str, we do a re.fullmatch, whereas for list, we just check endswith. After adding the first adapter, though, the naming pattern of the modules changes. In the example above, the name for the linear layer changes from "lin0" to "base_model.model.lin0", which is why the fullmatch fails but the endswith still works.
Reproduction
from peft import LoraConfig, get_peft_model
from torch import nn

class MLP(nn.Module):
    def __init__(self, bias=True):
        super().__init__()
        self.lin0 = nn.Linear(10, 20, bias=bias)

def test_target_modules_list():
    config = LoraConfig(target_modules=["lin0"])
    test_it(config)
    print("Adding two adapters with target_module being a list works")

def test_target_modules_str():
    config = LoraConfig(target_modules="lin0")
    test_it(config)

def test_it(config):
    model = MLP()
    model = get_peft_model(model, config, "adapter0")
    model.add_adapter("adapter1", config)
    print("Adding two adapters with target_module being a str works")

if __name__ == "__main__":
    # works
    test_target_modules_list()
    # ValueError: Target modules lin0 not found in the base model
    test_target_modules_str()

I think that most users would be surprised that:

Adding the first adapter works but adding the second fails, even though they use the same config.
Using target_modules=["lin0"] works but target_modules="lin0" fails for the 2nd adapter.

Solution
We could change the logic of not using re.fullmatch for str, but I think that could be tricky to achieve without breaking BC. Instead, I chose to change the inject_adapter call in add_adapter to pass the base model, not the whole peft model. This way, the naming pattern is preserved.
Tests
I haven't added extra tests for this. The script above could serve as a test. However, it will be sufficient to remove the guard added in #1105 (here):
    if isinstance(config.target_str, modules):
        # TODO this should be doable
        self.skipTest("Multiple adapters cannot currently be added when target_modules is a string.")

as that will test exactly this behavior and was how the bug was originally uncovered. Depending on what PR lands first, the guard has to removed in this PR or in #1105.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Move cookies operations to metacom

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/impress/commit/508119e97f841e174b23ce60d24c1b0478f92870 and https://github.com/metarhia/impress/commit/247e199f6110432e34a8dca15052c9cc3eaba434


fix: fuzzer now generates ints in correct range

Resolves #458
This should prevent ints that have a value of type(intN).max + 1 from being generated, by inferring the sign from the overflowing subtraction result instead of having sign as a prop
This has not yet been thoroughly tested. @crisgarner can you forgeup to this branch and make sure it fixes your issue?

Still getting the same issue

This doesn't actually fix it because when we overflowing sub, it sets the upper bits to 0xffff..ff8 for example. This causes the I256 conversion with a negative number to convert all those set bits to turn off causes bad encoding. Masking the num prior to converting to I256 solves the issue. Will post code in a minute

let uint = U256::from(&x) % U256::from(2).pow(U256::from(y * 8));
let max_int_plus1 = U256::from(2).pow(U256::from(y * 8 - 1));
let num = I256::from_raw(uint.overflowing_sub(max_int_plus1).0);
num.into_token()

This should fix it

ah good find, thanks @brockelmore—implemented and tested locally and seems to be working as expected now, @crisgarner please try again 😁

ah good find, thanks @brockelmore—implemented and tested locally and seems to be working as expected now, @crisgarner please try again grin

It works, now doesn't matter how many tests I specify, passes


pip install and pip-compile fails for new package version 0.2.0

Hi, I tried updating sphinx-collections from 0.0.1 to 0.2.0 but get some errors.
Command pip-compile fails
I updated sphinx-collections version in my requirements.in file and tried to create the requirements.txt file using pip-compile.
pip-compile requirements.in
    WARNING: Generating metadata for package sphinx-collections produced metadata for project name sphinxcontrib-collections.
    Fix your #egg=sphinx-collections fragments.
  ERROR: Could not find a version that satisfies the requirement sphinx-collections==0.2.0 (from versions: 0.0.1a1, 0.0.1, 0.2.0)
Traceback (most recent call last):
  File "C:\...\.venv\Lib\site-packages\pip\_vendor\resolvelib\resolvers.py", line 348, in resolve
    self._add_to_criteria(self.state.criteria, r, parent=None)
  File "C:\...\.venv\Lib\site-packages\pip\_vendor\resolvelib\resolvers.py", line 173, in _add_to_criteria
    raise RequirementsConflicted(criterion)
pip._vendor.resolvelib.resolvers.RequirementsConflicted: Requirements conflict: SpecifierRequirement('sphinx-collections==0.2.0')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\...\.venv\Lib\site-packages\pip\_internal\resolution\resolvelib\resolver.py", line 92, in resolve
    result = self._result = resolver.resolve(
                            ^^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\pip\_vendor\resolvelib\resolvers.py", line 481, in resolve
    state = resolution.resolve(requirements, max_rounds=max_rounds)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\pip\_vendor\resolvelib\resolvers.py", line 350, in resolve
    raise ResolutionImpossible(e.criterion.information)
pip._vendor.resolvelib.resolvers.ResolutionImpossible: [RequirementInformation(requirement=SpecifierRequirement('sphinx-collections==0.2.0'), parent=None)]

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\...\.venv\Scripts\pip-compile.exe\__main__.py", line 7, in <module>
  File "C:\...\.venv\Lib\site-packages\click\core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\click\core.py", line 1055, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\click\core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\click\core.py", line 760, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\click\decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\piptools\scripts\compile.py", line 659, in cli
    results = resolver.resolve(max_rounds=max_rounds)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\piptools\resolver.py", line 604, in resolve
    is_resolved = self._do_resolve(
                  ^^^^^^^^^^^^^^^^^
  File "C:\...\.venv\Lib\site-packages\piptools\resolver.py", line 636, in _do_resolve
    resolver.resolve(
  File "C:\...\.venv\Lib\site-packages\pip\_internal\resolution\resolvelib\resolver.py", line 101, in resolve
    raise error from e
pip._internal.exceptions.DistributionNotFound: No matching distribution found for sphinx-collections==0.2.0

Command pip install fails
pip install sphinx-collections==0.2.0
Collecting sphinx-collections==0.2.0
  Downloading sphinx-collections-0.2.0.tar.gz (12 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
  WARNING: Generating metadata for package sphinx-collections produced metadata for project name sphinxcontrib-collections. Fix your #egg=sphinx-collections fragments.
Discarding https://files.pythonhosted.org/packages/f7/4d/6c94cf98457d43e4fed65a25e1cafc0a5250c8e9ce2838ae31c249912400/sphinx-collections-0.2.0.tar.gz (from https://pypi.org/simple/sphinx-collections/): Requested sphinxcontrib-collections from https://files.pythonhosted.org/packages/f7/4d/6c94cf98457d43e4fed65a25e1cafc0a5250c8e9ce2838ae31c249912400/sphinx-collections-0.2.0.tar.gz has inconsistent name: expected 'sphinx-collections', but metadata has 'sphinxcontrib-collections'
ERROR: Could not find a version that satisfies the requirement sphinx-collections==0.2.0 (from versions: 0.0.1a1, 0.0.1, 0.2.0)
ERROR: No matching distribution found for sphinx-collections==0.2.0

I noticed the version number in pyproject.toml is still 0.0.2. Should be updated too I guess.

Thanks for reporting this problem :+1:
Unfortunately, I can execute a pip install sphinx-collections successfully on my machine with a fresh Python venv.
However, I fixed the problem in pyproject.toml and pushed it to master
As I have no experience with pip-compile, can you do me a favour an test this?

Hi,
@danwos, could you check the installed sphinx-collections version ?
Mine is:
[dev]$ pip list | grep collections
sphinx-collections             0.0.1

Here is what happens when I try to install. It switches to 0.0.1 version:
[dev]$ pip install sphinx-collections
Defaulting to user installation because normal site-packages is not writeable
Collecting sphinx-collections
  Downloading sphinx-collections-0.2.0.tar.gz (12 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
  WARNING: Generating metadata for package sphinx-collections produced metadata for project name sphinxcontrib-collections. Fix your #egg=sphinx-collections fragments.
Discarding https://files.pythonhosted.org/packages/f7/4d/6c94cf98457d43e4fed65a25e1cafc0a5250c8e9ce2838ae31c249912400/sphinx-collections-0.2.0.tar.gz (from https://pypi.org/simple/sphinx-collections/): Requested sphinxcontrib-collections from https://files.pythonhosted.org/packages/f7/4d/6c94cf98457d43e4fed65a25e1cafc0a5250c8e9ce2838ae31c249912400/sphinx-collections-0.2.0.tar.gz has inconsistent name: expected 'sphinx-collections', but metadata has 'sphinxcontrib-collections'
  Downloading sphinx_collections-0.0.1-py3-none-any.whl (17 kB)
Requirement already satisfied: sphinx>2.0 in /usr/local/lib/python3.11/site-packages (from sphinx-collections) (6.1.3)
Requirement already satisfied: jinja2 in /usr/local/lib/python3.11/site-packages (from sphinx-collections) (3.1.2)
Requirement already satisfied: gitpython in /usr/local/lib/python3.11/site-packages (from sphinx-collections) (3.1.40)
Requirement already satisfied: sphinxcontrib-applehelp in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (1.0.7)
Requirement already satisfied: sphinxcontrib-devhelp in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (1.0.5)
Requirement already satisfied: sphinxcontrib-jsmath in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (1.0.1)
Requirement already satisfied: sphinxcontrib-htmlhelp>=2.0.0 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (2.0.4)
Requirement already satisfied: sphinxcontrib-serializinghtml>=1.1.5 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (1.1.9)
Requirement already satisfied: sphinxcontrib-qthelp in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (1.0.6)
Requirement already satisfied: Pygments>=2.13 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (2.17.2)
Requirement already satisfied: docutils<0.20,>=0.18 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (0.18.1)
Requirement already satisfied: snowballstemmer>=2.0 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (2.2.0)
Requirement already satisfied: babel>=2.9 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (2.14.0)
Requirement already satisfied: alabaster<0.8,>=0.7 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (0.7.13)
Requirement already satisfied: imagesize>=1.3 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (1.4.1)
Requirement already satisfied: requests>=2.25.0 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (2.31.0)
Requirement already satisfied: packaging>=21.0 in /usr/local/lib/python3.11/site-packages (from sphinx>2.0->sphinx-collections) (23.2)
Requirement already satisfied: MarkupSafe>=2.0 in /usr/local/lib/python3.11/site-packages (from jinja2->sphinx-collections) (2.1.3)
Requirement already satisfied: gitdb<5,>=4.0.1 in /usr/local/lib/python3.11/site-packages (from gitpython->sphinx-collections) (4.0.11)
Requirement already satisfied: smmap<6,>=3.0.1 in /usr/local/lib/python3.11/site-packages (from gitdb<5,>=4.0.1->gitpython->sphinx-collections) (5.0.1)
Requirement already satisfied: charset-normalizer<4,>=2 in /usr/local/lib/python3.11/site-packages (from requests>=2.25.0->sphinx>2.0->sphinx-collections) (3.3.2)
Requirement already satisfied: idna<4,>=2.5 in /usr/local/lib/python3.11/site-packages (from requests>=2.25.0->sphinx>2.0->sphinx-collections) (3.6)
Requirement already satisfied: urllib3<3,>=1.21.1 in /usr/local/lib/python3.11/site-packages (from requests>=2.25.0->sphinx>2.0->sphinx-collections) (2.1.0)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.11/site-packages (from requests>=2.25.0->sphinx>2.0->sphinx-collections) (2023.11.17)
Installing collected packages: sphinx-collections
Successfully installed sphinx-collections-0.0.1

@danwos Maybe you can try the command pip install sphinx-collections==0.2.0 ?
Then you'll probably get the same error No matching distribution found as described in my original post. This is also the main error cause why pip-compile fails.
I have the same output as @joaohf for command pip install sphinx-collections. Looks like the name and repository in pyproject.toml are wrong indeed (when I compare it to the repo version of sphinx-needs for example). I am not familiar with pyproject.toml, so I don't know if the definition of packages[] is okay/wrong as well?

Hi,
any updates on the above issue? I just tried to update my sphinx-collections and it seems this error still exists:
[dev]$ pip install --upgrade sphinx-collections
Defaulting to user installation because normal site-packages is not writeable
Requirement already satisfied: sphinx-collections in /home/dev/.local/lib/python3.10/site-packages (0.0.1)
Collecting sphinx-collections
  Using cached sphinx-collections-0.2.0.tar.gz (12 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
  WARNING: Generating metadata for package sphinx-collections produced metadata for project name sphinxcontrib-collections. Fix your #egg=sphinx-collections fragments.
Discarding https://files.pythonhosted.org/packages/f7/4d/6c94cf98457d43e4fed65a25e1cafc0a5250c8e9ce2838ae31c249912400/sphinx-collections-0.2.0.tar.gz#sha256=eb266811ca117524919a9f232dd27267a2838d9c0729604f8411ae7eab36f660 (from https://pypi.org/simple/sphinx-collections/): Requested sphinxcontrib-collections from https://files.pythonhosted.org/packages/f7/4d/6c94cf98457d43e4fed65a25e1cafc0a5250c8e9ce2838ae31c249912400/sphinx-collections-0.2.0.tar.gz#sha256=eb266811ca117524919a9f232dd27267a2838d9c0729604f8411ae7eab36f660 has inconsistent name: filename has 'sphinx-collections', but metadata has 'sphinxcontrib-collections'
Requirement already satisfied: jinja2 in /home/dev/.local/lib/python3.10/site-packages (from sphinx-collections) (3.1.2)
Requirement already satisfied: sphinx>2.0 in /home/dev/.local/lib/python3.10/site-packages (from sphinx-collections) (7.3.7)
Requirement already satisfied: gitpython in /home/dev/.local/lib/python3.10/site-packages (from sphinx-collections) (3.1.43)
Requirement already satisfied: sphinxcontrib-applehelp in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (1.0.8)
Requirement already satisfied: requests>=2.25.0 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (2.31.0)
Requirement already satisfied: Pygments>=2.14 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (2.17.2)
Requirement already satisfied: imagesize>=1.3 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (1.4.1)
Requirement already satisfied: sphinxcontrib-qthelp in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (1.0.7)
Requirement already satisfied: docutils<0.22,>=0.18.1 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (0.20.1)
Requirement already satisfied: sphinxcontrib-jsmath in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (1.0.1)
Requirement already satisfied: tomli>=2 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (2.0.1)
Requirement already satisfied: sphinxcontrib-devhelp in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (1.0.6)
Requirement already satisfied: alabaster~=0.7.14 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (0.7.16)
Requirement already satisfied: sphinxcontrib-serializinghtml>=1.1.9 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (1.1.10)
Requirement already satisfied: snowballstemmer>=2.0 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (2.2.0)
Requirement already satisfied: packaging>=21.0 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (24.0)
Requirement already satisfied: babel>=2.9 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (2.14.0)
Requirement already satisfied: sphinxcontrib-htmlhelp>=2.0.0 in /home/dev/.local/lib/python3.10/site-packages (from sphinx>2.0->sphinx-collections) (2.0.5)
Requirement already satisfied: MarkupSafe>=2.0 in /home/dev/.local/lib/python3.10/site-packages (from jinja2->sphinx-collections) (2.1.3)
Requirement already satisfied: gitdb<5,>=4.0.1 in /home/dev/.local/lib/python3.10/site-packages (from gitpython->sphinx-collections) (4.0.11)
Requirement already satisfied: smmap<6,>=3.0.1 in /home/dev/.local/lib/python3.10/site-packages (from gitdb<5,>=4.0.1->gitpython->sphinx-collections) (5.0.1)
Requirement already satisfied: idna<4,>=2.5 in /home/dev/.local/lib/python3.10/site-packages (from requests>=2.25.0->sphinx>2.0->sphinx-collections) (3.6)
Requirement already satisfied: urllib3<3,>=1.21.1 in /home/dev/.local/lib/python3.10/site-packages (from requests>=2.25.0->sphinx>2.0->sphinx-collections) (2.2.1)
Requirement already satisfied: certifi>=2017.4.17 in /home/dev/.local/lib/python3.10/site-packages (from requests>=2.25.0->sphinx>2.0->sphinx-collections) (2024.2.2)
Requirement already satisfied: charset-normalizer<4,>=2 in /home/dev/.local/lib/python3.10/site-packages (from requests>=2.25.0->sphinx>2.0->sphinx-collections) (3.3.2)

Yes, I would like an update as well :-)

I can confirm this issue. Installing the package with a requirments.txt via sphinxcontrib-collections@git+https://github.com/useblocks/sphinx-collections.git@37b54c0a78e1772aabe13634ea2e2211749ac865 works, but note that sphinx-collections needs to be replaced by sphinxcontrib-collections.
sphinxcontrib-collections==0.2.0 does not work.
As #26 (comment) states the naming is not consistent.
Is there any objection to changing sphinxcontrib-collections to sphinx-collections? And then package a version 0.2.1?

Hi @danwos,
how shall we proceed with this issue?
Shall I open a PR with the proposed fix?
I need this working, so I am fine helping to fix it.
Just let me know how you like it.


Support for Macbook keyboard media buttons

Please support media playback control using the media buttons of Mac keyboards.

So, I guess you tried it and it is not working? Do you see any errors in console in dev mode? /cc @KeitIG

Hello ! It has been done here: https://github.com/KeitIG/museeks/pull/177
It will be released as soon as we complete the 0.7.0 milestone :)

Apparently since I also installed Vox and Vox preferences, it's taking ownership of the buttons so that's why it wasn't working. @YurySolovyov I assumed it wasn't supported yet since it didn't work the first time hence me filing this issue.


Multiple backends issue

Right now, when using javacpp if two backends are specified in pom.xml, only the first one gets loaded and used. In both native cpu and cuda backends, target resource name is still jniNativeOps.
And since it would be nice to have different backends loaded at the same time, what workaround can we have there?
Here's how it works right now: https://github.com/deeplearning4j/nd4j/blob/0abcdf5d0493126506798c5ff057544495b7fd34/nd4j-backends/nd4j-api-parent/nd4j-native-api/src/main/java/org/nd4j/nativeblas/NativeOps.java#L15-L19
And javacpp Loader is scanning classpath for first target binary occurence: jniNativeOps.dll for example.
p.s. with maven-shade situation will become worse, if someone will have two backends at the same time, since two out of three backend binaries share name and they share common package name.
@saudet what would be the best solution here, from JavaCPP point of view?

Maybe make NativeOps abstract, and for each backend add it's own extender class? So, we'll have own namespace for each backend, while  relying on the same java code from parent abstract class? And since each of them will be located in its own namespace, we'll not have conflicts.

Basically, I guess we could copy NativeOps.java and Nd4jBlas.java to nd4j-native and nd4j-cuda-7.5, under different package names, and have them extend the original ones in nd4j-native-api, taking care to switch the methods from "native" to "abstract", and create instances of those two new classes in JCublasNDArrayFactory and CpuNDArrayFactory. How does that sound?

If you know of a better way to do it give it a  shot.
On Fri, Apr 1, 2016 at 12:28 AM, Samuel Audet<EMAIL_ADDRESS>wrote:

Basically, I guess we could copy NativeOps.java and Nd4jBlas.java to
nd4j-native and nd4j-cuda-7.5, under different package names, and have
them extend the original ones in nd4j-native-api, taking care to switch
the methods from "native" to "abstract", and create instances of those two
new classes in JCublasNDArrayFactory and CpuNDArrayFactory. How does that
sound?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/deeplearning4j/nd4j/issues/756#issuecomment-204244036

BTW, we could automatically generate at build time these proxy classes with JavaCPP's Parser. That's what it's for. We could also use something like Byte Buddy to create the proxy classes dynamically, but I have a feeling this won't play well with Android. Another option is to use virtual dispatch from native C++. This would allow us to have only one Java class, but it wouldn't work on Windows, for example, where we need to use MSVC for CUDA and GCC for OpenMP: Their C++ runtimes are incompatible. This is one reason why people came up with things like Java and .NET in the first place. :)

Any new thoughts here given recent changes?

In the end, do we need to be able to use two backends simultaneously?

@saudet how will the new JIT loader in javacpp affect this?

It won't, it should behave same, that is we can choose our backend once at runtime by putting what we need in the class path, and that's it. As for uber JARs, the library files are still going to cause a conflict, yes...

Fixed: https://github.com/deeplearning4j/nd4j/pull/1508


SyntaxError when trying to create website

When I execute hyde -s /path/to/your/site create it returns this error.
(.venv) joram@joram-mini MineNet Website % hyde -s minenet create
Traceback (most recent call last):
  File "/Users/joram/PyCharmProjects/MineNet Website/.venv/bin/hyde", line 7, in <module>
    from hyde.main import main
  File "/Users/joram/PycharmProjects/MineNet Website/.venv/lib/python3.9/site-packages/hyde/main.py", line 6, in <module>
    from hyde.engine import Engine
  File "/Users/joram/PycharmProjects/MineNet Website/.venv/lib/python3.9/site-packages/hyde/engine.py", line 5, in <module>
    from hyde.exceptions import HydeException
  File "/Users/joram/PycharmProjects/MineNet Website/.venv/lib/python3.9/site-packages/hyde/exceptions.py", line 10
    raise HydeException(message), None, tb
                                ^
SyntaxError: invalid syntax

Running Python 3.9.6
MacOS 14.0, M1 Chip

sad

Hi @JXSnack - Hyde is unmaintained and I have no plans to restart development.

If you are just looking for a good static website generator - use Astro JS
You can also try nuejs which is created by someone closely associated with hyde.
If you are particular about python - you can probably use pelican.

Hi @navilan, sad to hear it's unmaintained. Any chance you could archive this repository, update the README and/or update the website with this statement? Would've saved me (and probably others) some time and trouble if it was stated more prominently. Thanks for all your work on this project.


Subgraph/verified mint

All looks normal to me, we will need to test queries on rinkeby afterwards

All looks normal to me, we will need to test queries on rinkeby afterwards


Add support for password-protected certificate authority keys

Hey,
Generating server keys fails with a password protected CA.pem key. Right now you have to remove the key via openssl rsa -in key.pem -out keyless.pem and set MACHINE_AUTH_PRIVATE_KEY as the keyless version.
Would it be possible to prompt for the key password here?

Made a patch which allows us to decrypt a certificate key file to make a tls.Certificate.
I'm not sure how you'd prefer we test this — want me to add an encrypted CA & key pair for tests to the repo?
Also, I'm not such a fan of Godep as it continually bloody breaks with symlinks and git errors (I use GPM instead), so I modified the Godeps JSON manually. Honestly couldn't get the fucker to work, and ran into some consistent open issues.

Thanks!  Unit tests would be nice (perhaps a cert/key named insecure_test or something) but even instructions in the readme would help.
I haven't had too many issues with Godep.  You should be able to Godep update <repo> to update the depends and Godep save to vendor new ones.

@tonyhb any update on this?

Completely forgot about this and got back on internal projects, apologies. Rebasing master and adding documentation + tests now

sounds good thanks!
On Thu, Feb 26, 2015 at 12:58 PM, Tony Holdstock-Brown <
<EMAIL_ADDRESS>wrote:

Completely forgot about this and got back on internal projects, apologies.
Rebasing master and adding documentation + tests now
—
Reply to this email directly or view it on GitHub
https://github.com/docker/machine/issues/407#issuecomment-76229579.

@ehazlett just adding tests... I need to make a password encrypted CA key file for an x509 certificate to test accurately.
I think the best way is to add a new function to generate these, but I'm not sure if generating password protected CA certificates is out of scope.
An alternative is adding some premade CA files to a test directory, but that feels wrong. What do you think?

Maybe we could add a password string to GenerateCACertificate — if it's blank we create a non-password protected file.
That way, when a user creates a machine for the first time they can also password protect the CA files?

+1 for additional password option to GenerateCACertificate

@ehazlett awesome, just added and tested that in PR #408

I'm +1 on this concept.


Federated service IP out of sync between AWS LB and CloudDNS records

 Issue by juntagabor
Wednesday Oct 26, 2016 at 16:09 GMT
Originally opened as https://github.com/kubernetes/kubernetes/issues/35637

Is this a request for help? No
What keywords did you search in Kubernetes issues before filing this one?
federation dns
federation aws
federation aws cname

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
BUG REPORT
Kubernetes version (use kubectl version):
v1.4.4
Environment:

Cloud provider or hardware configuration:
GCE for federation control plane
AWS for cluster1 (federated)
GCE for cluster2 (federated)
OS (e.g. from /etc/os-release):
NAME="Debian GNU/Linux"
VERSION_ID="7"
VERSION="7 (wheezy)"
Kernel (e.g. uname -a):
3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt25-2 (2016-04-08) x86_64 GNU/Linux
Install tools:
Others:

What happened:
Federation controller creates the appropriate A entries for all federated services on Google CloudDNS, including resolving AWS load balancer DNS name to IPs. But as IP changes for AWS LBs, the A records get outdated, causing service disruption as they become out of sync.
What you expected to happen:
I expected the federation controller to either add a CNAME pointing to AWS LB, or to keep the IPs for AWS in sync overtime.
How to reproduce it (as minimally and precisely as possible):

Deploy a federation control plane at GCE
Create cluster at AWS
Add AWS cluster to federation
create a federated service
wait 24h (or until AWS Load Balancer IPs change)
Cloud DNS will be pointing to different IPs than AWS LB.

Anything else do we need to know:
AWS recommends:

Because the set of IP addresses associated with a LoadBalancer can change over time, you should never create an "A" record with any specific IP address.

 Comment by nikhiljindal
Friday Oct 28, 2016 at 00:57 GMT

cc @kubernetes/sig-cluster-federation @quinton-hoole

 Comment by quinton-hoole
Friday Oct 28, 2016 at 15:25 GMT

@jungtagabor thanks for the comprehensive bug report.  Yes, you're right, we should be publishing CNAME's for AWS LB's, not A records.
The bug is here:
https://github.com/kubernetes/kubernetes/blob/master/federation/pkg/federation-controller/service/dns.go#L185
in case someone wants to have a go at it before I get to it. The code is pretty self-explanatory, and the fix should be pretty trivial.

 Comment by juntagabor
Friday Oct 28, 2016 at 16:09 GMT

@quinton-hoole thanks for the response and pointing out where the issue is.
Looking more into this, it looks a little more complex. I think the decision to resolve AWS LB to IPs before creating the CloudDNS entry was because it is not possible to have A and CNAME records, or multiple CNAME records, for the same DNS entry (federated service in this case). So it is not possible to have

service.namespace.federation.svc.my.domain.com. (AWS) - CNAME
service.namespace.federation.svc.my.domain.com. (GCE) - A

Instead, Kubernetes resolves the CNAME and inserts the A records, but unfortunately that causes the out-of-sync situation.

 Comment by saturnism
Friday Nov 11, 2016 at 17:09 GMT

should it at least resync the IP?
and use CNAME for the cluster-specific entry

 Comment by nikhiljindal
Monday May 22, 2017 at 19:17 GMT

Yes, ideally we want A records for GCP and CNAME for AWS, but since we cant have both A and CNAME records for same DNS entry, we need to choose either one for hybrid.
cc @madhusudancs

 Comment by madhusudancs
Monday May 22, 2017 at 19:51 GMT

You cannot have CNAME of AWS-only clusters either. CNAMEs can point to a single target, however we need our records to load balance between multiple targets and that's an illegal CNAME configuration. An alternative on AWS clusters is to use ALIAS records, but that works for AWS-only clusters and when DNS provider is Route53. This leaves us with 2 options:

If the dnsprovider is Route53, configure an ALIAS record for the global DNS name and point that to a combination of ELB names and CNAME records. I haven't verified if it is legal to configure ALIAS records in that way, hopefully it is. In the targets directly point to AWS service shards using their ELB names and configure CNAMEs for non-AWS service shards.

Simply run an IP resync logic in the service controller for service shards whose service ingress status has a hostname instead of an IP. Resync period could be set to DNS TTLs and probably configured via a command line flag to federated service controller/controller manager.

Latter one is more generic and involves running the control loop, often unnecessarily.

 Comment by madhusudancs
Monday May 22, 2017 at 19:54 GMT

About running control loops unnecessarily, I wonder if AWS Cloud Watch is capable of delivering events/notifications when IP addresses behind an ELB change. That would help us optimize the loop a bit. But on the other hand, it would be specific to AWS I guess.

 Comment by nikhiljindal
Monday May 22, 2017 at 19:56 GMT

Yes the second one is better since it works for hybrid as well.

 Comment by baldeynz
Wednesday Aug 16, 2017 at 02:07 GMT

Hi - has there been any update on this issue? any idea when its likely to be fixed?

 Comment by quinton-hoole
Wednesday Aug 16, 2017 at 16:36 GMT

@baldeynz No updates that I'm aware of.  Code submissions to fix the bug would be well received.

 Comment by quinton-hoole
Friday Sep 08, 2017 at 01:07 GMT

Needs to be resolved for GA.

 Comment by nikox94
Thursday Oct 12, 2017 at 22:58 GMT

Hey, if nobody can start on the issue, I would like to take it. Seems pretty interesting...

 Comment by quinton-hoole
Thursday Oct 12, 2017 at 23:10 GMT

Sure, that would be great @nikox94 .  Welcome aboard!

 Comment by nikox94
Sunday Oct 15, 2017 at 07:40 GMT

/assign

 Comment by k8s-ci-robot
Sunday Oct 15, 2017 at 07:40 GMT

@nikox94: GitHub didn't allow me to assign the following users: nikox94.
Note that only kubernetes members can be assigned.

In response to this:

/assign

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

 Comment by nikox94
Friday Oct 20, 2017 at 07:02 GMT

Hey! :)
A small status update:
It seems that relevant code is here and here.
I have only glanced at the code and am also still trying to understand the use-cases as I am new to the project. Could anybody shed some light on what a federated service DNS is supposed to be doing? Why should CloudDNS be pointing to the same IPs that AWS LBs point to?
If I understand correctly, in the original bug report and reproduction steps, the person has created a GKE cluster and an AWS cluster. They have a federated service accross the two clusters. LBs in GCP should thus be pointing to both the GKE instances and the AWS ones, and also the AWS LB should be pointing to both the AWS service instances (inside the cluster) as well as the GCP ones? The issue is then that the AWS LB gets "desynched" and does not point to the correct IPs anymore. I am making this up completely so, is it sensible?
Also I will work towards setting up and reproducing the bug, so I can have a chance of observing and fixing it.
Thanks!
P.S. If it's not too much of a bother, @quinton-hoole or @shashidharatd , could you please validate the above is in the correct direction. Thank you!

 Comment by nikox94
Friday Oct 20, 2017 at 07:04 GMT

Also could you please assign the issue to me. Thank you!

 Comment by shashidharatd
Monday Oct 23, 2017 at 06:14 GMT

Hello nick,
Here is design for federated service. You will get all the scenarios documented there and also the implementations.
Coming to this issue, there is a technical limitation to the design vs reality in how the AWS load-balancers are implemented.

 Comment by shashidharatd
Monday Oct 23, 2017 at 14:09 GMT

missed the link https://github.com/kubernetes/community/blob/master/contributors/design-proposals/multicluster/federated-services.md :)

 Comment by nikox94
Sunday Oct 29, 2017 at 20:48 GMT

OK so I have my setup up and running. If you read more closely the guy is having the cluster based in GCP and using CloudDNS for the federation DNS.
I now have set up a cluster in GKE with federation running on it and then I have a DNS zone in CloudDNS that I am using for the federation dns.
I created a cluster in AWS and joined it, with a service exposed on a LB in AWS. Currently we have the scenario like this
AWS Load-balanced service:
 $ dig a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com
;; ANSWER SECTION:
a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com.	60 IN A<IP_ADDRESS>
a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com.	60 IN A<IP_ADDRESS>

CloudDNS from federation level:
serene_blog_7@macro-nuance-159207:~$ gcloud dns record-sets list --zone="the-dream-shack"
NAME                                                                     TYPE   TTL    DATA
thedreamshack.com.                                                       NS     21600  ns-cloud-e1.googledomains.com.,ns-cloud-e2.googledomains.com.,ns-cloud-e3.googledomains.com.,ns-cloud-e4.googledomains.com.
thedreamshack.com.                                                       SOA    21600  ns-cloud-e1.googledomains.com. cloud-dns-hostmaster.google.com. 1 21600 3600 259200 300
my-nginx.me.thedreamshack.com.                                           CNAME  1800   a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com.
my-nginx.default.fellowship.svc.thedreamshack.com.                       A      180    <IP_ADDRESS>,<IP_ADDRESS>
my-nginx.default.fellowship.svc.us-west-2.thedreamshack.com.             A      180    <IP_ADDRESS>,<IP_ADDRESS>
my-nginx.default.fellowship.svc.us-west-2a.us-west-2.thedreamshack.com.  A      180    <IP_ADDRESS>,<IP_ADDRESS>

 Comment by nikox94
Sunday Oct 29, 2017 at 20:50 GMT

Now I will wait some hours or days for the AWS LB to change the underlying IPs for the federated service.
Also please bear in mind that the 3rd record from the top above was created by me and is meant to be the "solution". I.e. there is no reason in this case to create A records and one should simply create CNAME records and that's it.
I am not sure why at all we are creating A records... will have to find the code that does that and check it.

 Comment by nikox94
Monday Oct 30, 2017 at 16:45 GMT

AWS ELB IPs have not yet changed. Will wait a bit more, if they don't change will look into a way of manually triggering a change.

 Comment by nikox94
Monday Oct 30, 2017 at 21:59 GMT

OK so "we are in the game" as we say in Bulgaria. Issue reproduced!
Now AWS changed LB IPs:
$ dig a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com +trace

; <<>> DiG 9.10.3-P4-Ubuntu <<>> a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com +trace
;; global options: +cmd
.			280244	IN	NS	a.root-servers.net.
.			280244	IN	NS	h.root-servers.net.
.			280244	IN	NS	k.root-servers.net.
.			280244	IN	NS	j.root-servers.net.
.			280244	IN	NS	d.root-servers.net.
.			280244	IN	NS	g.root-servers.net.
.			280244	IN	NS	i.root-servers.net.
.			280244	IN	NS	l.root-servers.net.
.			280244	IN	NS	f.root-servers.net.
.			280244	IN	NS	c.root-servers.net.
.			280244	IN	NS	e.root-servers.net.
.			280244	IN	NS	b.root-servers.net.
.			280244	IN	NS	m.root-servers.net.
;; Received 1652 bytes from <IP_ADDRESS>#53(<IP_ADDRESS>) in 0 ms

com.			172800	IN	NS	l.gtld-servers.net.
com.			172800	IN	NS	b.gtld-servers.net.
com.			172800	IN	NS	c.gtld-servers.net.
com.			172800	IN	NS	d.gtld-servers.net.
com.			172800	IN	NS	e.gtld-servers.net.
com.			172800	IN	NS	f.gtld-servers.net.
com.			172800	IN	NS	g.gtld-servers.net.
com.			172800	IN	NS	a.gtld-servers.net.
com.			172800	IN	NS	h.gtld-servers.net.
com.			172800	IN	NS	i.gtld-servers.net.
com.			172800	IN	NS	j.gtld-servers.net.
com.			172800	IN	NS	k.gtld-servers.net.
com.			172800	IN	NS	m.gtld-servers.net.
com.			86400	IN	DS	30909 8 2 E2D3C916F6DEEAC73294E8268FB5885044A833FC5459588F4A9184CF C41A5766
com.			86400	IN	RRSIG	DS 8 1 86400<PHONE_NUMBER>0000<PHONE_NUMBER>0000 46809 . uXJoWMQAgKQFAEvWCikbyKRvjkXEZ1PpKcouFSRMVyHW9GALDoR4s4hT RHTtrn/BfbL7jFzFSxsL9vLFs9ExkaRCB25r9Hzx271XT5ZJuOSQn5Rl ykQLJkIUfXXpVBt+dgzQzsNdtol9apQlfKb135a31qq4Z5KJh/a0zq7E 2Xgw6fhgOTkF5Hq8wxCiBL17+HvVYVQidYufLfpULu+SPUjpeSlPjh8X JhPZlzEeFP9CtYkH1ePUswMRWSG46sFE3bNo4lsQlhUXBktWcMmnuJeE 56RBKKVvEeGc06Mg2HGawlXNAf4R+YHjkyn4+zENpX/BMiYEU2zBaNXS CB6K6A==
;; Received 1230 bytes from <IP_ADDRESS>#53(f.root-servers.net) in 29 ms

amazonaws.com.		172800	IN	NS	u1.amazonaws.com.
amazonaws.com.		172800	IN	NS	u2.amazonaws.com.
amazonaws.com.		172800	IN	NS	r1.amazonaws.com.
amazonaws.com.		172800	IN	NS	r2.amazonaws.com.
CK0POJMG874LJREF7EFN8430QVIT8BSM.com. 86400 IN NSEC3 1 1 0 - CK0Q1GIN43N1ARRC9OSM6QPQR81H5M9A NS SOA RRSIG DNSKEY NSEC3PARAM
CK0POJMG874LJREF7EFN8430QVIT8BSM.com. 86400 IN RRSIG NSEC3 8 2 86400<PHONE_NUMBER>5018<PHONE_NUMBER>4018 11324 com. tUr5lk7lA/HTrs7fTVsG4IsUDY5vBVu8q8C+zac6NX/I6mIndhlw5uzR ap2fFLaEkhMV2D4Tf6Bonj4aqaFh5QxDMjVuVQwIGfuG0EDbNOSsH5Gq ud+ziOFfszPUpgqGFgMwvNHltTxtpJOVwSDMd81VXNMoIfUNbn0LTOOF imM=
F1RGCARTUOH1V344H263M3F4EHG6JI2L.com. 86400 IN NSEC3 1 1 0 - F1RNQU35LJSRBU32TLIABL5R3BQV3J82 NS DS RRSIG
F1RGCARTUOH1V344H263M3F4EHG6JI2L.com. 86400 IN RRSIG NSEC3 8 2 86400<PHONE_NUMBER>1824<PHONE_NUMBER>0824 11324 com. rFFjhgHbztB41+6iZ2BWGpdanWdDQfM+ATfJhWetsNsGOoCDLM8Dh1Fg 6R8j4IbVlhSQJskG/ZtlmUT349IQYVvN2hb4TiiZ2oQSr4ArgdFgiESx pZOuisbtffvVlzHzHaNhRNe/k672BRM+7atd6sMsuPnx8DsgjA1YKZNE DGY=
;; Received 716 bytes from <IP_ADDRESS>#53(h.gtld-servers.net) in 52 ms

us-west-2.elb.amazonaws.com. 300 IN	NS	ns-1475.awsdns-56.org.
us-west-2.elb.amazonaws.com. 300 IN	NS	ns-1769.awsdns-29.co.uk.
us-west-2.elb.amazonaws.com. 300 IN	NS	ns-332.awsdns-41.com.
us-west-2.elb.amazonaws.com. 300 IN	NS	ns-560.awsdns-06.net.
;; Received 236 bytes from <IP_ADDRESS>#53(r1.amazonaws.com) in 55 ms

a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com.	60 IN A<IP_ADDRESS>
a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com.	60 IN A<IP_ADDRESS>
us-west-2.elb.amazonaws.com. 1800 IN	NS	ns-1475.awsdns-56.org.
us-west-2.elb.amazonaws.com. 1800 IN	NS	ns-1769.awsdns-29.co.uk.
us-west-2.elb.amazonaws.com. 1800 IN	NS	ns-332.awsdns-41.com.
us-west-2.elb.amazonaws.com. 1800 IN	NS	ns-560.awsdns-06.net.
;; Received 268 bytes from <IP_ADDRESS>#53(ns-332.awsdns-41.com) in 54 ms

This means new LB IPs are now <IP_ADDRESS>, <IP_ADDRESS>.
However looking over at GCP we see that in CloudDNS:
my-nginx.default.fellowship.svc.us-west-2.thedreamshack.com.                A      180    <IP_ADDRESS>,<IP_ADDRESS>
my-nginx.default.fellowship.svc.us-west-2a.us-west-2.thedreamshack.com.     A      180    <IP_ADDRESS>,<IP_ADDRESS>
my-nginx.me.thedreamshack.com.                                              CNAME  1800   a9bec77a1bcdf11e7a6ca025c587eac3-248908204.us-west-2.elb.amazonaws.com.
my-nginx.default.fellowship.svc.thedreamshack.com.                          A      180    <IP_ADDRESS>,<IP_ADDRESS>

The A records created by Federation are still there, but now pointing to the wrong IP. We should instead have created CNAME records. The CNAME record I created for testing is still there and should redirect to the service correctly, but the A records should be broken.
Testing shows the CNAME is resolving correctly, however A records also are resolving correctly for the time being, but I think it's just because the IPs changed recently and AWS does not want to break any long-caching clients. Anyway the issue should be fixed. Starting to look for what code to change.
@shashidharatd and @quinton-hoole - how do we write a test case for this so that it's reproducible in the tests. E2E test? Is anybody writing those? What's our usual way of testing such scenarios?

 Comment by shashidharatd
Monday Nov 13, 2017 at 14:01 GMT

Hello @nikox94,
What you tested above is using one cluster. But if you add another cluster to federation, then the records for my-nginx.default.fellowship.svc.thedreamshack.com. should contain all the instances of the federated service. This is not possible if you change the dns record of my-nginx.default.fellowship.svc.thedreamshack.com. from A to CNAME.
CNAME record can just point to one more A Record or another CNAME records according to RFC. it cannot contain point to multiple CNAME records.

 Comment by nikox94
Wednesday Nov 15, 2017 at 07:46 GMT

So, after a short call with @shashidharatd, we reached the conclusion that implementing a full-blown DNS load balancing is too ambitious for now and that there are already plans for that. Currently we need to fix this bug, and the cleanest/simplest fix seems to be to make a go routine that will, every X minutes (this can be fairly large, on the order of 60 minutes or more), resolve all known LBs for all services managed by the cluster and if changed IPs have been found, will update the corresponding A records.
I was told that @quinton-hoole has designed the DNS and that I should run it by him first.
Related code can be found: here, method ensureDNSRrsets and others.

 Comment by quinton-hoole
Monday Nov 20, 2017 at 19:21 GMT

For the record, @nikox94 and I had a slack chat, and came to a slightly different conclusion.  I will leave it to him to post that update when he has time.  Thanks for the contributions @nikox94 - most appreciated.

 Comment by nikox94
Tuesday Nov 21, 2017 at 16:07 GMT

Hey @quinton-hoole, thank you for the welcoming atmosphere! :)
I have been a bit busy fighting some other fires this week, but have started running the tests and comprehending the code.
For the record, @quinton-hoole suggested that I instead integrate the DNS resync into the main resync loop that already exists. I did some research on AWS and it seems they will not offer notifications when the LBs change IPs. I have also read through this file and its tests. I am currently running the tests, and will want to do a bit of refactoring on them and add some tests that test the current bug if possible. Then I'll add the DNS refresh code to the control loop and submit the PR.
If anybody feels there is something to add - please go ahead. :)
It's my first K8s PR, so please @quinton-hoole, forgive me for any involuntary deficiencies.

 Comment by nikox94
Sunday Nov 26, 2017 at 09:52 GMT

My WIP merge request can be found here: https://github.com/kubernetes/federation/pull/155
It's not yet ready for final review. A lot of work left still.

cc @juntagabor

@nikox94, is #155 enough to resolve this issue?
cc @quinton-hoole

Nope, it was just a step towards resolving it. There is work left still.

/kind bug


revert allow_population_by_field_name

Revert "set allow_population_by_field_name pydantic config (#83)"
release version 0.6.3

FYI @fishi0x01
The pydantic mypy plugin doesn't support checking for aliases and field names :(

See https://github.com/python/mypy/issues/14850


pixbook2017 Firmware download checksum fail

I'm using the MrChromebox script to install Cloudready on my immaculate but 'end-of-life' pixelbook2017.
It starts downloading the Full ROM firmware but after a few minutes gives a red-text message:
Firmware download checksum fail; download corrupted, cannot flash.
i tried several type of network env but ran to the same error
Fw ver:google_eve.9584.201.0(11/18/2019)

then i found the script can't recognize my device,which caused checksum fail,may i force it to falshed?
some time ago,something was wrong with the pixelbook and i reinstalled os with some warning,but used it normal

then i found the script can't recognize my device,which caused checksum fail,may i force it to falshed?

what? post a pic of the script menu

Also, CloudReady/CrOS Flex don't support the Pixelbook. And it's not EOL yet.

yes, it was I had a wrong hwid,set one and resolved,tks for reply.


Added missing SAS permissions

This PR resolves https://github.com/Azure/azure-sdk-for-python/issues/8799 and resolves https://github.com/Azure/azure-sdk-for-python/issues/8784.

we should also cover the doc fix in this pr https://github.com/Azure/azure-sdk-for-python/pull/8793/files

Make sure the changelog is updated

Closing this due to age. Will create another PR to address the issue.


Failed downloads stuck in "connecting" state if you navigate away/back

The Issue
We recently made some improvements to download failure notifications (https://github.com/lbryio/lbry-desktop/pull/1768) but looks like we are running into a bug in the below scenario. You can still click "View" but it doesn't have the right color/mouseover.

Click play on lbry://oldmovie
Navigate away, wait 5 minutes
Go back to lbry://oldmovie
Notice the app shows connecting and the play button is greyed out.

The status should reset. If we want to be really creative, we should show some text on the file page that it failed to download previously, and to try again later.

System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

This still happens on the new video player (without floating player, or when you leave the page and start another video).


76 UI add units into time and mark allocations

I thought it would be better to let other person define it as they fit when designing the logics to them. I'll change them.


Feature/LASB-1832 Add endpoints to allow CRUD functionality on the EFORMS_STAGING table

What
Link to story
Added EForm Staging CRUD functionality to api. Adding 4 endpoints.
Checklist
Before you ask people to review this PR:

[ ] Tests should be passing: ./gradlew test
[ ] Github should not be reporting conflicts; you should have recently run git rebase main.
[ ] Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
[ ] You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

added a few comments for this PR. Ping me if you need to discuss them.

Hi @atharmajeed I think we have addressed all outstanding review feedback, please can you check once more and leave a comment for any we might have missed?
CC: @Kevin-Paxton


support Date conversions and not just DateTimes

we missing converting iso8601 date string. we should support them, too.

I'm happy to write this feature.

It appears to be already done.
I found the code that supports it here, and the accompanying test

This issue looks ready to be closed. Is there anything else that should happen with this feature?


Module structure exposes all methods to including base

In a scenario where the transaction logger is included into a module, the trapping of the logger doesn't work anymore:
require "transaction_logger"

module SomeModule
  include TransactionLogger

  def trace_me
    puts "trace_me"
  end

  add_transaction_log :trace_me
end

class Test
  include SomeModule
end

test = Test.new
test.trace_me # undefined method `trap_logger' for Test:Class

Well, TransactionLogger is a Module so isnt't supposed to be an extend?
module A
  def self.included(base)
    p "Included #{base}"
    base.extend ClassMethods
  end

  def self.extended(base)
    p "Extended #{base}"
    base.send :include, ClassMethods
  end

  module ClassMethods
    def gg
      p "hi"
    end
  end
end

module B
  extend A
end

class C
  include B
end

C.new.gg

Ah, true. You're right. But what I saw was the problem of the trap_logger method becoming a method on the including class.
Do you have some clues, how we could disguise it and not inject it?

What do you mean by disguise? The inheritance here of the ancestors makes sure that the method is in view and gets called.

After a short discussion at my table, we summarized the issue as follows:

The module definition shouldn't add any other method then .add_transaction_log to the base
Every other method required for the TransactionLogger's functionality needs to be encapsulated within the GEM

@dinks @JohnBDonner any objections?

:+1:

+1


sync OWNERS with containerd/project/MAINTAINERS

As containerd/cri is a core subproject of containerd, https://github.com/containerd/cri/blob/master/OWNERS should be synced with https://github.com/containerd/project/blob/master/MAINTAINERS

Closing, as the two repos were merged together recently


[PDF] Spec Title Hypehnates

In the PDF for v1.0.0-rc1, the spec title lists as:

Open Container Initiative Image Format Specifi-
cation

I suggest adding a line break so this may look like:

Open Container Initiative
Image Format Specification

Please set milestone to v1.0.0


Reset the consumer's position when an offset is out of range

If the committed offset for a given partition is invalid, there's not much to do except resetting to the default offset.
This could be a horrible idea. I need to do some more research in order to verify that commits can in fact become corrupted and it's not just ruby-kafka messing up right now.

Looks like I'm not the only one with this problem: https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/clients/consumer/OffsetOutOfRangeException.java

@tcrayford have you run into cases where fetch requests fail with OffsetOutOfRange in the JVM consumer? I'm unsure whether there's a race in the ruby-kafka code or something.

@dasch we haven't that much. They can happen during node failure under specific conditions but aren't part of normal operation in our experience.

I see a few scenarios in which a committed offset is no longer valid:

The message expires.
Log compaction is enabled and the message is removed.
The topic is deleted and re-created.

These would typically only be seen if the consumer had been paused for some time – but pausing a consumer group and starting it periodically is a valid use case, I think. There can also be times when a group has to go offline in order for some change to be made.
For these reasons, it could make sense to have automatic resetting of positions...

Okay, I'll try it out and evaluate again before making the API beta.


When using DuplicateRow, it is not duplicated correctly when there are multiple merged cells.

Description
When duplicating a row with multiple merged cells, the cells that are used correctly are not copied.
Perhaps this i++ skips the loop step.
https://github.com/360EntSecGroup-Skylar/excelize/blob/77978ac68d3808060e58df41ebede4b9f3631641/rows.go#L605
Steps to reproduce the issue:
1.
	f, err := excelize.OpenFile("../../.ignore/B.xlsx")
	if err != nil {
		panic(err)
	}
	sheet := f.GetSheetName(0)
	err = f.DuplicateRowTo(sheet, 1, 3)
	if err != nil {
		panic(err)
	}
	err = f.SaveAs("../../.ignore/B_.xlsx")
	if err != nil {
		panic(err)
	}

Describe the results you received:

Describe the results you expected:
Want the same thing as the 1 col to be the 3 col.
Output of go version:
go 1.15

Excelize version or commit ID:
77978ac68d3808060e58df41ebede4b9f3631641

Thanks for your issue. I have fixed it, please try to use the master branch code, and this patch will be released in the next version.


taskbar support issue

Describe the bug
So am current use windows10insider preview build 21364.So it have news&interest box on taskbar but now after use use taskbar x it have a little bug i mean it sometime appear and gone appear& gone it so very anoy
if you want to know abot my pc you can see from attachment and also how news&interrest bow actually look like

@Punrawee Just updated to 21370 and everything works fine here. I see you have a lot of icons in your taskbar. TaskbarX put's the icons back to the left when the icons start going behind the tray (clock, wheather..). You can disable that function inside the configurator under the Position tab. It's called "Revert to zero beyond tray".

okay

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


Update Dockerfiles

This PR updates the the Dockerfiles of Debian and Ubuntu based Owl container to catch up with current master branch.

Great! Thank you Jianxin for fixing it!


notification arriving as if app was in background when in foreground

When I send a notification it does not seems to matter whether the app is open, in background or closed, it just arrives as if it was from an external app.

It's kinda abnormal in a similar way for me too.
When the app is open,
nothing happens.
When the app is in the background it works.
I want it to work just like it's in the background.(when the user clicks the notification on the push alarm list)

I noticed this strange behaviour with an older phone running Android 4+

closing for now, reopen if still an issue with the latest version of the plugin


Feat: add CHM file loader

fix https://github.com/langchain-ai/langchain/issues/15469

a notebook for documentation in a follow up PR would be awesome!


Fix legacy slides

The issue is the slides link does not appear on the search page.
It's not maintained but the API should still accept it

@exussum12 Sorry for the delay in getting back you. Happy to merge this in, however before I do, phpcs has picked up an error in the change, could you please update the code to correct this? Thanks
<checkstyle version="2.3.4">
<file name="/vagrant/joindin-api/src/controllers/TalksController.php">
 <error line="361" column="9" severity="error" message="Expected 1 space after IF keyword; 0 found" source="Squiz.ControlStructures.ControlSignature.SpaceAfterKeyword"/>
</file>
</checkstyle>

Fixed that, Shocked that I missed it :scream:

No worries, happens to us all sometimes, thanks for the PR


Add system setting to automatically roll starship crit effects

When enabled, when a starship crosses at least one critical threshold, the critical damage table will automatically be rolled (multiple times if multiple CTs were crossed). Damage must be applied from the chat card in order for this to work.
(Will be updated to include #650 when that's merged)

Maybe instead of making this automatic, we can introduce a setting for it to be automatic or manual, and require the user to click a "Roll crit effect" button on the "Ship X took Y damage, crossing Z thresholds" card button? See #650 for this suggestion.

Consolidating changes into https://github.com/foundryvtt-starfinder/foundryvtt-starfinder/pull/1415.


feat(ui-results): add MatrixGridSynthesis

ANT-2416

moved to: https://github.com/AntaresSimulatorTeam/AntaREST/pull/2233


How to style checkbox and radio from outside

Here: https://mariomka.github.io/vue-checkbox-radio/ you are giving examples of how to style checkbox and radio components, but when I try to do that, my style is overridden by internal component style and thus does not have any effect.
How to solve or workaround this?

I don't know why (each project, build system is different...) but a workaround can be using class-nameto add a custom class.

Yeah, thank you. I've used !important as a workaround although this is not very beautiful.


Build CLI: Configurations on User Level

This PR closes #2119:
It provides a way for the users to configure their environment and preferences in a file in Chipmunk home directory to be loaded and used by the build CLI tool
The following configuration are provided currently:

The Shell to run the processes on (Bash, Zsh, Fish...)
Default UI preference to use with the tool (bars, report, print, immediate)

Roadmap:

[x] Implement user configurations as singleton with their load method
[x] Provide an enum to represent the shell running by users with methods to provide command to run process in the selected shell.
[x] Replace the implementation global commands function with the one from the user configurations.
[x] Add CLI commands to print the path of the configuration file and dump the default configurations to stdout and to the configuration file.
[x] Validate given shell before running any further commands.
[x] Consider preferred UI mode form the configuration.
[x] Use default values for not defined configurations
[x] Extend integration tests.
[x] Extend README.
[x] Increase tool version.
[x] Test on Windows.
[ ] Test on MacOS.

@DmitryAstafyev The default shell will be retrieved from SHELL environment variable.
I'll test this PR on Linux and Windows, but could you please test it on MacOS

Test passes on Windows


Template Detail: Added template info section

This PR adds the template info section (i.e. the right side of the screen with the template title, author, description, metadata) to the Template Detail page:
Things that will come in a follow up PR:

Update the header nav bar.
Add in the template controls.

This is looking so great. Well done!!
When you say:

Update the header nav bar.
I am assuming you mean also adding the spacing between the header and the body at that point yeah?

Few minor things - mostly just adding in that localization for strings.

Oh, the Template Detail page actually has a new header (or at least has different controls in the header):

This is looking so great. Well done!!
When you say:

Update the header nav bar.
I am assuming you mean also adding the spacing between the header and the body at that point yeah?

Few minor things - mostly just adding in that localization for strings.

Oh, the Template Detail page actually has a new header (or at least has different controls in the header):

🙃 yes I did notice that. there just wasn't any space between the body and the existing header structure and i wanted to make sure that that was part of the header update. All good!

This is looking so great. Well done!!
When you say:

Update the header nav bar.
I am assuming you mean also adding the spacing between the header and the body at that point yeah?

Few minor things - mostly just adding in that localization for strings.

Oh, the Template Detail page actually has a new header (or at least has different controls in the header):

🙃 yes I did notice that. there just wasn't any space between the body and the existing header structure and i wanted to make sure that that was part of the header update. All good!

Ha, then yes, the header update will also fix the space :D


Nick: Watch streamsgeek.com

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

streamsgeek\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


DBaaS Getting Started Updates Nov24Release

Added KB article on Replication and Failover feature in MySQL DBaaS

@christineparr11 did you mean to delete this?  Your comment says you are adding a file.

@jim-phillips - I am trying to add the file.  I actually thought it should have gone with the previous update I made, but I don't see it published, so I was attempting to push it again.

@christineparr11 It might not have been published yet, but it is in a different commit.  This PR will delete it.  I am closing this PR.  We will have a kb sync in staging.ctl.io before noon (CST).  Check that.

Thanks @jim-phillips .  The other changes from my previous pull request were made which is why I thought there was a problem with this specific file.  I will check out staging.


modeld.cc: always use extra client

@adeebshihadeh how much of this kind of stuff do we want to remove now?

Thanks for maintaining comma zero :)


Test Issue Mig

PHP Version
Test Issue Mig
Shopware Version
Test Issue Mig
Expected behaviour
Test Issue Mig
Actual behaviour
Test Issue Mig
How to reproduce
Test Issue Mig

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/MIG-1043.
[created from MIG-1043, comment 483933]

[public] This is just a test issue. We will close it
[created from MIG-1043, comment 483956]


Breaks on 3x3 with more than one solution

If I try to rref a matrix like
1 0 0
0 0 1
0 0 0

the rref function returns undefined. Try running rref([[1,0,0],[0,0,1],[0,0,0]]) to see what I mean
Tested on version 0.0.1

It also breaks on
1 0 0 0
0 0 1 0
0 0 0 1
0 0 0 1

and
1 0 0 0
0 0 1 0
0 0 0 0

Just get rid of rows with all zeros.  It will work after that.


AWS Parameter Store (aws+smp) support

@hairyhenderson are you working on this? Any ETA?

@iamralch Thanks for the reminder, I'd actually forgotten that I hadn't added this yet 😅
I don't have a lot of spare bandwidth right now, but will attempt to spend time on this over the next couple weeks.

Thank you for your quick reply. Appreciate!


--post-renderer does not support cross-platform bash/sh scripts

Output of helm version: version.BuildInfo{Version:"v3.6.3", GitCommit:"d506314abfb5d21419df8c7e7e68012379db2354", GitTreeState:"clean", GoVersion:"go1.16.5"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.2", GitCommit:"8b5a19147530eaac9476b0ab82980b4088bbc1b2", GitTreeState:"clean", BuildDate:"2021-09-15T21:38:50Z", GoVersion:"go1.16.8",
Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.1", GitCommit:"632ed300f2c34f6d6d15ca4cef3d3c7073412212", GitTreeState:"clean", BuildDate:"2021-08-19T15:39:34Z", GoVersion:"go1.16.7",
Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.): minikube
I am using Helm on Windows 10, Git bash is my shell of choice. I also use Ubuntu on WSL2and have aliased the helm command so that I can share the executable between the two platforms while it's installed in Windows.
I have create an sh script to use envsubst as a --post-renderer, but when I invoke the command (regardless of whether this is in WSL2 or Git Bash under Windows) Helm throws the following error:
$ helm template mytemplate ./mytemplate/ --post-renderer envsubst/envsubst.sh
Error: error while running post render on files: error while running command \\wsl$\Ubuntu\home\esune\git-repos\myproject\helm\envsubst\envsubst.sh. error output:
: fork/exec \\wsl$\Ubuntu\home\esune\git-repos\myproject\helm\envsubst\envsubst.sh: %1 is not a valid Win32 application.

It appears that Helm is expecting the file to be a Win32 file (it will, in fact, run bat files) and it is therefore impossible to use the script even if the correct interpreter (Git bash) is installed on the host machine. This seems like a big issue as it prevents charts from being usable cross-platform.

From my understanding Helm is using the standard os/exec library (https://github.com/helm/helm/blob/main/pkg/postrender/exec.go#L46). And within the standard library

The Windows version specified by internally-linked Windows binaries is now Windows 7 rather than NT 4.0.
Reference: https://golang.org/doc/go1.13#windows

Can you explicitly call the interpreter, Git Bash in this case (something like --post-renderer "/path/to/bash -c envsubst/envsubst.sh" ) and see if it works?

Error: invalid argument "/bin/bash -c envsubst/envsubst.sh" for "--post-renderer" flag: unable to find binary at /bin/bash -c envsubst/envsubst.sh: exec: "/bin/bash -c envsubst/envsubst.sh": stat /bin/bash -c envsubst/envsubst.sh: no such file or directory

I also tried using $(pwd)/envsubst/envsubst.sh to have a fully qualified path for the script, same result.
I think the issue might be with Helm using Go's os/exec to spawn the new process and how it might expect a "compatible" architecture for the target binary. I am not familiar enough with Go to provide further input, but I think this doesn't really scale well when you potentially have developers using different host architectures executing the same Helm command.

Still looking for clarifications/suggestions.

Still looking for clarifications/suggestions.

Still looking for clarifications/suggestions.

Your error was because you used the whole string as the name of the binary. It looked for the command, spaces and all, as the name of the command. You need to split that between --post-renderer and --post-renderer-args. See https://helm.sh/docs/helm/helm_install/#options

@joejulian I am not sure I follow you: there are no arguments in my call, just the path to the executable to be used as renderer, which is consistent with the documentation.
The reported issue is not that the command doesn't work, but that it ONLY works under Linux and not in other bash-compatible shells such as Git bash due to Go relying on the architecture of the underlying OS to spawn the process.

Error: invalid argument "/bin/bash -c envsubst/envsubst.sh" for "--post-renderer" flag: unable to find binary at /bin/bash -c envsubst/envsubst.sh: exec: "/bin/bash -c envsubst/envsubst.sh": stat /bin/bash -c envsubst/envsubst.sh: no such file or directory

is caused by --post-renderer "/bin/bash -c envsubst/envsubst.sh" as you've told it the name of the file you're trying to run is that entire string, spaces and all.
Try --post-renderer /bin/bash --post-renderer-args -c --post-renderer-args envsubst/envsubst.sh

Try --post-renderer /bin/bash --post-renderer-args -c --post-renderer-args envsubst/envsubst.sh

I will give a try your suggestion, however to be 100% clear my initial command 9see opening post) was:
$ helm template mytemplate ./mytemplate/ --post-renderer envsubst/envsubst.sh
Error: error while running post render on files: error while running command \\wsl$\Ubuntu\home\esune\git-repos\myproject\helm\envsubst\envsubst.sh. error output:
: fork/exec \\wsl$\Ubuntu\home\esune\git-repos\myproject\helm\envsubst\envsubst.sh: %1 is not a valid Win32 application.

This works on Linux, will not work in Windows under a bash-compatible shell. No spaces in the post-renderer argument in this case.

Does envsubst.sh have a shebang to inform the library function what program to execute the script with? Clearly without it windows is trying to interpret text as a binary.

Does envsubst.sh have a shebang to inform the library function what program to execute the script with? Clearly without it windows is trying to interpret text as a binary.

Yes, it does have a shebang identifying bash as the interpreter/executor for the script.

Well I'm stumped. Someone with a similar question asked here and it worked for them. Maybe there's a nugget there that I'm just missing.

@joejulian in the other question they are using a ps1 script (Powershell) in Powershell on Windows. Powershell is part of the OS architecture in windows.
Since I would like my script to be portable, I used bash and am using Git Bash as interpreter in Windows. Bash is not part of the OS architecture in windows.
Quoting one of my previous replies:

I think the issue might be with Helm using Go's os/exec to spawn the new process and how it might expect a "compatible" architecture for the target binary.

Again, I can only give you hints to what worked for someone else. Ultimately, this is for helm support and what you seem to need is operating system support. If you think it's Go's os/exec, perhaps you need to visit Gopher's slack. Maybe write a small go program and see if you can run your script from there. If it works, maybe that'll shine some light on this issue.

fyi, I did double-check the source. The command you give helm is run from here: https://github.com/helm/helm/blob/main/pkg/postrender/exec.go#L47
There's nothing in there that would tell windows to run it any other way. I suppose, if the shebang isn't being interpreted by windows, the other option would be to run bash as your post-renderer and specify your script as the post-renderer-option.

Thank you @joejulian, I'll try to follow-up with the Go team if/when I get a chance. This issue may come in handy in the future if someone else is having a similar problem.


New Version

Hello, @mihaeu! Thanks for the cool project :)
Are there any plans to release a new version?
We are trying to use your product, but there are some issues:

https://github.com/mihaeu/cosmere/issues/4
https://github.com/mihaeu/cosmere/issues/32
https://github.com/mihaeu/cosmere/issues/34
https://github.com/mihaeu/cosmere/issues/47

Yeah, I've been meaning to release a new version. Thanks for the push.
#4 might not make it in there, because there's a lot of design decisions to be considered. What I instead would like to do is provide a plugin system so that it's easy to add your own macros/transformers and just tweak it the way you need it.
#32 is high up on my list and I'll try to get it fixed soon. Unfortunately this requires reverse-engineering how Confluence builds their links, as there is no documentation on how the slugs are created.
#34 I can have another look at, but this is one of those nasty examples where I would have to touch the internals of the markdown converter.

Oh, great news! Waiting! Thanks!


[BUG] ThetaForecaster fails to give prediction intevals for pd.DataFrames

Describe the bug
@fkiraly , yesterday you mentioned that pd.Series should not appear in the code. I feel that this is somehow connected.
I noticed that ThetaForecaster is unable to generate the prediction intervals when it's inside a ForecastingPipeline. It turns out that the problem is not with the pipeline but with the input being forced to be a pd.DataFrame.
The error i'm getting is:
ValueError: Unable to coerce to Series, length must be 1: given 36

The problem is connected with the addition of y_pred with sem in  _predict_quantiles, in theta.py.
I'd offer a work around but I'm sure a temporal change of data type is not the best solution;-).
Btw, is it expected that sem is a pd.Float64Index instance?
Here's a toy example:
pd.DataFrame([1,2])+pd.Float64Index([.1,.2])

To Reproduce

import pandas as pd
from sktime.forecasting.theta import ThetaForecaster
from sktime.forecasting.model_selection import temporal_train_test_split
from sktime.datasets import load_airline
y = load_airline()
y_train, y_test = temporal_train_test_split(y)

fp = ThetaForecaster(sp=12)
fp.fit(pd.DataFrame(y_train))
fp.predict_quantiles(fh=y_test.index)

Expected behavior

import pandas as pd
from sktime.forecasting.theta import ThetaForecaster
from sktime.forecasting.model_selection import temporal_train_test_split
from sktime.datasets import load_airline
y = load_airline()
y_train, y_test = temporal_train_test_split(y)

fp = ThetaForecaster(sp=12)
fp.fit(y_train)
fp.predict_quantiles(fh=y_test.index)

Additional context

Versions

System:
python: 3.8.13 | packaged by conda-forge | (default, Mar 25 2022, 05:59:45) [MSC v.1929 64 bit (AMD64)]
executable: C:\Users\Mi\miniconda3\envs\sktime-dev\python.exe
machine: Windows-10-10.0.22000-SP0
Python dependencies:
pip: 22.0.4
setuptools: 62.1.0
sklearn: 1.0.2
sktime: 0.11.2
statsmodels: 0.13.2
numpy: 1.22.3
scipy: 1.8.0
pandas: 1.4.2
matplotlib: 3.5.1
joblib: 1.1.0
numba: 0.53.0
pmdarima: 1.8.5
tsfresh: None

Bug confirmed, and fixed here: #2529.
The explanation is, likely a typo, a single missing underscore.
A test that would catch this is passing different input types - we need to discuss in the dev meeting how we trade off the increased compute with increased case coverage.

Btw, is it expected that sem is a pd.Float64Index instance?

That shouldn´t matter, should it?

Btw, is it expected that sem is a pd.Float64Index instance?

That shouldn´t matter, should it?

Nah, was just a bit surprised to see something like this.

Nah, was just a bit surprised to see something like this.

I guess float would be cleaner, and may be less risky in case addition for exotic types changes or gets deprecated.
But, in the spirit of "keeping PR scopes confined", I haven't changed it.
Maybe @guzalbulatova can shed light on why it's an index?

I frankly have no idea, it seems like a bug to me, somewhere defaulting to that class perhaps? There shouldn't be anything in Theta itself to require a Float64Index anywhere really. 🤔

well, it still computes the right thing seemingly, due to dunder magic...

Let's see how this would work in new version of Theta, I don't fully understand this current implementation. I'll keep this index issue in mind.

Let's see how this would work in new version of Theta, I don't fully understand this current implementation.

I thought that line was coming from you?

No, the new modular version is to substitute this current one and enable enhanced 'autotheta'. This implementation is by big-o if I'm not mistaken


Meteor to support local npm package which is not published in npm registry

Hi, I developed a local NPM package which didnt publish to npm yet, and I installed the package by cd into the package folder locally and run "npm install -g", I can see it is installed into global node_modules folder, and can be used by other node scripts with require('...').
however, when I added the package name in packages.json like
{
"packageName": "1.0.0"
}
and start meteor using mrt, it complains
....
=> Started MongoDB.
npm: updating npm dependencies -- packageName...
=> Errors prevented startup:
While building package npm:
error: there is no npm package named 'packageName'
I digged into the meteor code and it seems that the meteor-npm.js is doing below
npm install --force package@version and parse its output to check if npm returned 404 package cannot be found, if so, it will give above error.
so it sounds to me that local package which is not available in NPM yet is always getting this error which is very inconvenient, and also it requires to maintain the connection to npm registry to be able to start the application which is not a good idea.
could you make it support local installed packages and maybe even without dependency of querying global npm registry on application startup ?

Same issue - I'd like to test an NPM module locally before I publish it. @Kingwzb - did you submit an issue with Meteor? Could you cross ref please?

As a workaround I symlinked packages/npm-container/.npm/package/node_modules\my-package to my dev directory, then added the module to app.browserify.js as normal.


Proof steps should not be nested

See proof in https://de.wikibooks.org/wiki/Mathe_für_Nicht-Freaks:_Supremum_und_Infimum_bestimmen_und_beweisen#Anker:Menge_von_Folgengliedern
@kulla Suggest to use list instead or are nested proof steps actually allowed?

How would a proof step list template solve nesting?

It could be included. However, I think nested proof steps are the better solution since sometimes one needs additional text after and before proof steps...

All right, nested proof steps will be allowed then.


Download video?

Is there any example of download video?

@swapnilnandgave Have you wrked on download video? I need some help.

I am working on it. I will send PR once done


Would you accept a 1.3.0+1.19.2 PR?

I don't expect you to keep updating it past this PR, because the codebases for dynamic registries diverged a lot ever since, but there's a few vital features missing from where you left off with 1.19.2.
I'm close to finishing a 1.3.0 1.19.2 port for a friend, but I was just wondering if you would be willing to accept it as a one off version bump, mainly so this port is more accessible for those that need it.
The fork can be found here

totally! I don't have the time to backport those features but i know a lot of people use .2

Alright! I'll send the PR over in a bit.


dart analyze can't complete, and memory keeps increasing

Flutter SDK Version 1.22.4
Dart SDK Version 2.10.4
MacOSX 11.5.2

I use 'dart analyze' to analyze flutter project, but it hungs：

and the memory of dart is increasing:

However, use 'dart analyze lib' works.

If dart analyze hangs, but dart analyze lib completes normally, that suggests that there is something outside of lib that is exposing the problem. Is this code that you can share with us so that we can attempt to reproduce the problem you're seeing? If not, would you be able to try to narrow down the problem by analyzing other directories and files until you find one that causes this behavior, then look to see if there is some obvious difference between that file and others for which the tool completes?
Unless we can reproduce the behavior we are unlikely to be able to fix it.

Assigning p3 until there's a reproduction case.

@bwilkerson I have known the reason. It works after I deleted the ios folder.  I used 'fvm' to manager the different versions of flutter. I develop the project with different flutter version on the different branches of the project, so I need to switch the version of flutter occasionally. Maybe, in ios folder, the dependencies is based on flutter 2.5.3, but the whole project depends on 1.22.4, so it hangs while using dart('2.10.3') to analyze the code in ios folder which based on dart('2.14.4').


created_at undefined

#347
This error is occurring today, I can't really tell which line  in my code tho, it's not showing in the stack but but looks like the search method and garrytan username is what I'm experiencing the error with
error TypeError: Cannot read properties of undefined (reading 'created_at')
at new Tweet (/Users/iandeeq/dev/purpleai/node_modules/rettiwt-api/src/models/Tweet.ts:76:33)
at new CursoredData (/Users/iandeeq/dev/purpleai/node_modules/rettiwt-api/src/models/CursoredData.ts:34:20)
at TweetService.FetcherService.extractData (/Users/iandeeq/dev/purpleai/node_modules/rettiwt-api/src/services/FetcherService.ts:185:10)
at TweetService. (/Users/iandeeq/dev/purpleai/node_modules/rettiwt-api/src/services/FetcherService.ts:207:21)
at step (/Users/iandeeq/dev/purpleai/node_modules/rettiwt-api/dist/services/FetcherService.js:33:23)
at Object.next (/Users/iandeeq/dev/purpleai/node_modules/rettiwt-api/dist/services/FetcherService.js:14:53)
at fulfilled (/Users/iandeeq/dev/purpleai/node_modules/rettiwt-api/dist/services/FetcherService.js:5:58)

What where the search parameters?

Working on it, expect patch on monday.

@jsonkcli Can you specify all the parameters that you used for searching which thew this error?

@Rishikant181 I was just taking a proper look at the API, did you update the search method?
My initial params
const favUserTweets = await twitter.tweet.search({
fromUsers: [favUser.userName],
sinceId: mostRecentTweet?.tweetId || undefined
endDate: oneDayFromNow
links: true
});

It occurs when it get's to next cursor
next: Cursor {
value: 'DAADDAABCgABF7c_Ql1W8VUKAAIXtJprLBqxXQAIAAIAAAACCAADAAAAAAgABAAAAAAKAAUXt6_WVgAnEAoABhe3r9ZV_9jwAAA'
}

Still not able to reproduce the error, unfortunately.

const favUserTweets = await twitter.tweet.search({
fromUsers: [favUser.userName],
sinceId: mostRecentTweet?.tweetId || undefined
endDate: oneDayFromNow
links: true
});

I need the exact values of these params when the error occurs.