Git-upstream fails when doing a keptn update project and changing files in the repo

The following procedure results in a bug:

Create a project (without Git upstream)
Create a Git repo
Update the project by setting the git-remote-url and git credentials: keptn update project PROJECTNAME --git-user=GIT_USER --git-token=GIT_TOKEN --git-remote-url=GIT_REMOTE_URL
Change a file in the repo (e.g. on GitHub)
Send a new-artifact event does not work since git commit does not work

Logs of the configuration-service:
{"timestamp":"2020-07-27T08:59:42.774391415Z","logLevel":"DEBUG","message":"Staging Changes"}
{"timestamp":"2020-07-27T08:59:43.31659198Z","logLevel":"ERROR","message":"Could not commit to staging branch of project keptn07project"}
{"timestamp":"2020-07-27T08:59:43.316619254Z","logLevel":"ERROR","message":"Error executing command git push<EMAIL_ADDRESS>exit status 1\nTo https://github.com/grabnerandi/keptn07__simplenode.git\n ! [rejected]        staging -\u003e staging (fetch first)\nerror: failed to push some refs to<EMAIL_ADDRESS>Updates were rejected because the remote contains work that you do\nhint: not have locally. This is usually caused by another repository pushing\nhint: to the same ref. You may want to first integrate the remote changes\nhint: (e.g., 'git pull ...') before pushing again.\nhint: See the 'Note about fast-forwards' in 'git push --help' for details.\n"}

Problem:

The branches in the local repository are not set to the branches on the upstream repo.

Fix:

Set remote branch using: git branch --set-upstream-to=origin/dev dev or git push -u origin HEAD

The solution to this is explained in the following stackoverflow post: https://stackoverflow.com/questions/32056324/there-is-no-tracking-information-for-the-current-branch
Here is the important piece:

This means that when doing a "keptn update project" the configuration-service additionally needs to iterate through all branches and setup branch trecking with:
git branch --set-upstream-to=origin/BRANCHNAME BRANCHNAME

I validated this manually and it fixed my installation

@checkelmann was also able to reproduce this behavior in a local test

I will prepare a fix for this issue.
Need to extend the Addorigin function to loop over all bracnhes, check them out and add the remote to them.

Hi @checkelmann,
I already implemented the fix in #2129


Format the JSON response for HTML examples.

Use JSON.pretty_generate to improve format of JSON responses for HTML examples.

This seems like a great idea! Do you have a photo you can post of the updated output.

I just attached a screen shot of the formatted JSON output.

Feel free to add this PR to my fork.
https://github.com/jramos/rspec_api_documentation


macOS 10.15 visual issue: default avatar now switches to white, so text needs to switch foreground and shadow color


LUCENE-9963 Fix issue with FlattenGraphFilter throwing exceptions from holes

Description
There are various situations were the flatten graph filter may throw an exception or drop tokens due to mishandling of holes in the analysis graph. The primary causes were

Output nodes that were hole destinations and valid sources
Freeing nodes before they had been fully resolved
Putting edges at incorrect output nodes after a hole

Solution
1)Iterate through all output.InputNodes since dest gaps can exist
2)Freebefore the minimum input node not the first input node(which is usually, but not always, the minimum)
3)Recovery nodes that have end offset less than their preceeding node gaplessly follow their preceeding node
4)Ensure nodes  positionInc >1 are arriving at the end of the graph.
Tests
same tests as https://github.com/apache/lucene/pull/146
plus tests for shingled tokens and gaps at the start of a token graph.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Lucene maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.

I love all the added test cases!
Maybe in the new random test we could use tokenstreamToAutomaton and then confirm no dead states in that, using existing APIs?

Maybe another way to improve the checking for correctness in the randomized test (or maybe in a new randomized test) would be to randomly generate a set of strings from a limited alphabet, create the minimal automaton matching only those strings (we have a nice API to do that, efficiently, already), call flatten, and the confirm that the resulting output graph still accepts all the original strings?
I.e., flatten should only ever "generalize" -- accepting strings that the original machine did not -- and never "remove" previously accepted strings?
But I think one missing part for such a test would be an "Automaton to TokenStream" converter, i.e. a "serializer" from (acyclic) Automaton to TokenStream.  I think such a thing would not be too difficult to build, basically just topo sort the input graph (and throw exception if it has cycles), then emit the transitions as tokens.  The posInc attribute is guaranteed to never go negative because of the topo sort.  This would (separately) be a nice utility API to convert between these two things that are really nearly the same ;)

Thank you for the ascii-art explanation of neighbor versus consumed holes!
Do we also handle the neighbor-hole case when there is more than one long path?
E.g. if the document has the wizard of oz, and we overlay the_wizard (pos len 2) and also the_wizard_of_oz (pos len 4) shingles on top of that, then run through StopFilter which deletes the (hmm and also of, but ignore that consumed hole for now), then we have a neighbor hole with three paths (the original tokens plus the two shingles).  Do we handle that / have a test case for it?

Maybe another way to improve the checking for correctness in the randomized test (or maybe in a new randomized test) would be to randomly generate a set of strings from a limited alphabet, create the minimal automaton matching only those strings (we have a nice API to do that, efficiently, already), call flatten, and the confirm that the resulting output graph still accepts all the original strings?

Are you referring to the Automoton.Builder API here?

But I think one missing part for such a test would be an "Automaton to TokenStream" converter, i.e. a "serializer" from (acyclic) Automaton to TokenStream. I think such a thing would not be too difficult to build, basically just topo sort the input graph (and throw exception if it has cycles), then emit the transitions as tokens. The posInc attribute is guaranteed to never go negative because of the topo sort. This would (separately) be a nice utility API to convert between these two things that are really nearly the same ;)

I'm having trouble with the transition from Automoton back to TokenStream. posInc is easy to calculate because after a toposort it's either 1 or 1+ number of holes since last non-hole. poslen is giving me some trouble because it needs to know how many positions were in alternate paths. I should be able to look at each path from a node that has multiple edges, follow those paths until they converge counting token separators along the way, then take the max of those as the length to be covered. However I'm not sure how to distribute that length across paths.
For example
 --->O--->O--->O---
|                  |
|                  V
(Start)           (End)
|                  ^
----->O------>O-----

There are 3 positions on the longest path so the total positions to be covered is 3, but on the short 2 node path I don't think it's possible to tell what should be poslen=1 and which should be poslen=2. For creating a valid token stream that can match the same as the automaton can accept it might not matter. However to reverse tokenstreamToAutomaton we would need to get those poslens right.
We would also need to set Offsets. Offsets are completely lost in the conversion from TokenStream to Automaton.  We should be able to generate offsets that are valid for a TokenStream but may not reflect what the text originally looked like. For creating a general AutomotonToTokenStream I think this would be a problem, as tokenstreamToAutomaton -> AutomotonToTokenStream would end in a different TokenStream then one started with.
For the generalize test I might have to create an Automaton from a TokenStream with and one without FlattenGraphFilter, then create the acceptable paths from the one without FGF and check if the Automaton with FGF can accept all of them. It's less elegant but would still confirm that the flattened graph was a generalization.

While writing the generalization test I've realized there are some strange interactions with it and holes. For example the FlattenGraphFilter will squash continuous sequences of holes down to one. Also if all the edges in a path get deleted then FlattenGraphFilter will squash the remaining edges down to remove the deleted path.
I think the first aspect of squashing down holes may be a bug. I noticed the behavior earlier and added a test for it, but looking at the code it may come from hole recovery using outputNodes.getMaxPos() + 1 instead of outputNodes.getMaxPos() + positionIncrement

For the generalize test I might have to create an Automaton from a TokenStream with and one without FlattenGraphFilter, then create the acceptable paths from the one without FGF and check if the Automaton with FGF can accept all of them. It's less elegant but would still confirm that the flattened graph was a generalization.

Ooh +1 this is a good test too!  But you would need some way to randomly creating "interesting" TokenStreams.  Hmm do we have a RandomGraphTokenStream yet?

However I'm not sure how to distribute that length across paths

I think you are making it harder than it needs to be!
First, topo sort the Automaton, which results in integer numbers assigned to each node (different integers than the original states, in general).
Then you will output each transition from the Automaton as a token in the TokenStream, one by one.  Start with the state numbered 0 (by the toposort).  Output all transitions leaving it.  For posLength of each of those, just look at the to topo-sort state and subtract those two integers.  E.g. if topo-sort state 0 has a transition to topo-sort state 7 then that token's posLength is 7.
Give a non-zero posInc each time you jump to the next top-sort state, and pretend you jumped from state=-1 for the first state=0.
Note that this algorithm requires a finite Automaton (we should call Operations.isFinite to ensure, and throw an exception if it is infinite).  Also, it cannot handle fully disconnected graphs ("isolated subgraphs").  All nodes must be reachable from any other node.  If the randomized test simple creates the DFA accepting a set of randomly generated strings both of these properties will hold.

Thanks for the explanation. I think I understand how to do the Automaton to TokenStream now.

Changed 3 parts of flatten graph filter

don't call freeBefore when we finish a hole src node. Hole src nodes can mean something hasn't been accounted for correctly so calling freeBefore may free nodes that still have info we need.
Change how RecoverFromHole picks the outSrc node. Instead of always always adding to the end of the OutputNodes list it scales down its position increment to match the output graph and uses that to pick the OutputNode. This doesn't require the distinction between neighbor and consumed holes.
Change the condition for fixing gaps after an alternate path hole. Now it checks if the shortest edge from the position the current edge is incrementing from comes to the current start node. if it doesn't then we've crossed a gap.

Added a few more tests testHeavilyConnectedGraphWithGap and testShingleFromGap I believe cover the cases @mikemccand mentioned earlier. I also added the Automaton -> TokenStream -> FlattenedTokenStream -> Automaton no paths lost test as well as a validation test on the existing random graph test. The validation gets weird with handling holes, but I think that's necessary as a hole in an accept sequence could be anything and hole sizes can change after flattening.

For additional testing I've built a few indexes and played some queries against them. I didn't find any changes to latency or memory usage. Depending on the data set used there was impact on results. The English data sets didn't show any impact. Using it with our Japanese analyzer had a larger impact. In one index 10% of queries were impacted by the change with those queries matching more documents. Previously those documents had lost tokens from FlattenGraphFilter. The primary cause of these dropped tokens was graphs with gaps in them. The JapaneseTokenizer would create graphs that FilteringTokenFilters would remove tokens from and then FlattenGraphFilter would incorrectly flatten them.

Simplified AutomatonToTokenStream and added license comment.
I beasted the random tests a few hundred times and found 2 issues.
1)When completely separated edges get connected it can introduce gaps and cause book keeping errors.
2)The checkGeneralization test can miss paths where a hole in the unflattened graph would cross multiple nodes.
The logic to navigate around issue 2 slows down checkGeneralization to the point I don't think it is useful so I commented that portion out. The test that calls it still checks if asserts are getting hit, and testPathsNotLost is checking if strings to make an automaton are accepted after the automaton is flattened.

Failure is from random testing. It looks like I'm overthinking how to handle calculating the new output node in recoverHole. Since multiple graphs could look the same after deletions, but would have flattened differently without the deletes, calculating future nodes in the middle of flattening isn't going to work. Instead the code will have to place at the end of the output graph adjusting for known holes.

We have been using this change internally for a few weeks now. We no longer encounter the ArrayIndexOutOfBounds exceptions that we were previously experiencing. Depending on the dataset/analyzer combination we have seen up to a 1% increase in the average number of tokens per field. This comes from the tokens that had previously been dropped now being correctly indexed.

We have been using this change internally for a few weeks now. We no longer encounter the ArrayIndexOutOfBounds exceptions that we were previously experiencing. Depending on the dataset/analyzer combination we have seen up to a 1% increase in the average number of tokens per field. This comes from the tokens that had previously been dropped now being correctly indexed.

Thanks for the update @glawson0!  This is great news.

+1, thanks @glawson0 -- I'll try to merge this soon.
I think this is also safe to backport?  No API breaks, and it is "just" fixing bugs in this hairy TokenFilter?

Thanks for the merge! You are correct, there are no API changes and it should be safe to backport.


Simplify the Random extern for uniform distributions only

The text briefly mentions how a user's P4 program can use this to
generate pseudo-random values with non-uniform distributions.

As per decision discussed during 2017-Sep-07 PSA working group meeting: https://github.com/p4lang/p4-spec/wiki/PSA-Meeting-Minutes-Sep-7,-2017

@cc10512 This is a pretty small change, one that ends up with fewer options and a simpler design than the original, so should be fairly quick for someone to review and merge, if you have a few minutes.


Pydantic warning on run

Hi,
Whenever I invoke any Elevenlabs related code using this python package, Python spits out the following to my console:
.virtualenvs/btf/lib/python3.11/site-packages/pydantic/_internal/_fields.py:127: UserWarning: Field "model_id" has conflict with protected namespace "model_".

You may be able to resolve this warning by setting `model_config['protected_namespaces'] = ()`.
  warnings.warn(

I think this is coming from this package. For ease of reproduction, I can reproduce this with Python 3.11.3, and the following code:
class Gender(Enum):
    MALE = "male"
    FEMALE = "female"

class BaseVoiceBackend(ABC):
    default_voice = None
    @abstractmethod
    def get_voices(self, gender: Gender):
        pass

    @abstractmethod
    def generate_audio(self, text, voice=None):
        pass

    @abstractmethod
    def save_audio(self, audio, path):
        pass

class ElevenLabsVoiceBackend(BaseVoiceBackend):
    voices = None
    default_voice = "Bella"

    def __init__(self):
        set_api_key(os.environ["ELEVENLABS_API_KEY"])

    def get_voices(self, gender):
        if not self.voices:
            self.voices = voices()
        return [voice.voice_id for voice in self.voices if 'gender' in voice.labels and voice.labels['gender'].lower() == gender.value]

    def generate_audio(self, text, voice=None):
        return generate(
            text=text,
            voice=voice if voice is not None else self.default_voice,
            model="eleven_multilingual_v2"
        )

    def save_audio(self, audio, path):
        save(audio, path)
        song = AudioSegment.from_mp3(path)
        song.speedup(playback_speed=1.18).export(path, format="mp3")

Hi. Can you update to v0.2.16? This should be fixed

Thanks!


stuck in loop?

Running on binary v1.1.1 on Windows 7 x64 (tried both x86 and x64 versions). After entering username and password and 2 step auth, the program loops, trying to connect or sign in. I do connect through a proxy, not sure if this is causing the problem. If I try to open any menu items a I get this error.

I changed the login mechanism to be more robust. I will publish version 1.1.3 soon. Please reopen if the bug is still there


.cdsapirc not found in build context

Hello,
the command "docker-compose --file docker-compose-db.yml up --build" fails with the following error:
Step 8/16 : COPY .cdsapirc /root/.cdsapirc
COPY failed: file not found in build context or excluded by .dockerignore: stat .cdsapirc: file does not exist
ERROR: Service 'data-integration' failed to build

thank you in advance for your help
Ernest.

Sorry for the late reply.
The .cdsapirc contains the API key associated to one's personal account in Copernicus Climate Store. As such it is private file and cannot be included in this public repository.
You can find more information here:
https://cds.climate.copernicus.eu/api-how-to


quotemark config option ordering matters?

Bug Report

TSLint version: 5.5.0
TypeScript version: 2.4.2
Running TSLint via: tslint-language-service in VSCode

TypeScript code being linted
let k = "blah";

with tslint.json configuration:
{
  "rules": {
    "quotemark": [
      true,
      "avoid-template",
      "single"
    ]
  }
}

Actual behavior
no error
Expected behavior
" should be ' (quotemark) error
If I put "single" after true in the config array, then it works. I tested rearranging the other possible config options like "jsx-double", and those all seem to work regardless of order, but as soon as "single" is not the first config option, the entire rule stops working.
Is this by design or a bug? I couldn't see any documentation on the ordering of config options mattering. Could this be affecting other rules too? I ordered all of my config options alphabetically because I assumed order wouldn't matter.

Welp found the place in the code is explicitly checking the first value of the config options: https://github.com/palantir/tslint/blob/master/src/rules/quotemarkRule.ts#L78-L85
I would have seen the error too if I was not using tslint-language-service, but Typescript doesn't yet support logging from plugins for end users. Although, this should probably be mentioned in the rule documentation.

You're right, this rule expects "single" or "double" to be the first option.
That should at least be documented.
#889 also tracks validating the config to show an error in such cases.

Would it be better if we searched through argument list to find OPTION_SINGLE or OPTION_DOUBLE and throw error if no such argument provided?


Udpated to latest stable versions of Drools and Spring Boot.

... also added more IntelliJ patterns to gitignore.

Coverage remained the same at 100.0% when pulling c59a4f297bd4df085114f6eccd0fdf59cc96bcdd on nikos:master into 22a192fc81c8f39daa3772ba5c20f975f5ca5eb3 on jonashackt:master.

Coverage remained the same at 100.0% when pulling cd51dd910e3d389f96028c696055accd56e75c2d on nikos:master into 22a192fc81c8f39daa3772ba5c20f975f5ca5eb3 on jonashackt:master.


Assign locked resource names with extra keyword

I would like to run the following block before a states-block:
            options {
                lock(extra:
                    [
                        [label:'test-instance', variable: 'TEST_INSTANCE', quantity: 1, resource: null],
                        [label:'remote-dongle', variable: 'HOSTNAME', quantity: 1, resource: null],
                        [label:'local-dongle', variable: 'LOCALNAME', quantity: 1, resource: null]
                    ],
                    inversePrecedence: false,
                    variable: 'LOCKED_RESOURCES'
                )
            }

I get a comma-separated list of the names of the 3 locked resources in ${LOCKED_RESOURCES}. However, the order of the resource names is random and varies every time I run the pipeline. That is why I need to assign variable on a per-locked-resource base, which however currently does not work (the variables ${TEST_INSTANCE},  ${HOSTNAME},  ${LOCALNAME} are simply null).

Probably #294 does not address this, but worth linking to that PR as a recent one addressing a "nearby" matter.

I am proposing PR #307 which I think would resolve this. For example, the following pipeline works with this change:
lock(extra:[[label:'client', quantity:1],[label:'server',quantity:1]]) {
    theClient = findLocks(label:'client')
    theServer = findLocks(label:'server')
    echo "connecting ${theClient.name} to ${theServer.name}"
}

will be handled in #428


Request to update documentation for maven slave image regarding MAVEN_MIRROR_URL environment variable

From the source code of bootstrap script of the maven slave image, I saw the support of setting a maven repository mirror server URL using MAVEN_MIRROR_URL environment variable.
But this is not documented on the OpenShift documentation website. I just tested it this afternoon, and found it worked like a charm.
Thanks for this wonderful feature.
Can you also update the documentation to add some instructions regarding this feature?

Hello @jiajliu ,
Glad you liked the feature.
We will update the documentation on this.
/assign @waveywaves

I don't suppose there's a mechanism to separately declare a MAVEN_PLUGIN_MIRROR_URL ?
A Maven pom has two artifact resolution sections, one for dependencies and one for plugins. This is explained in Plugin_Repositories
Notwithstanding the "I have yet to hear a convincing argument for doing so", we have this implemented in our Nexus repository. "The pluginRepository elements each specify a remote location of where Maven can find new plugins."
We can't seem to work around this. Should a separate issue be created to implement such?

@ianwilliams1 Please do open a separate issue for this feature, It would be easier for us to track and work on that way.


feat: update to remix auth v4

this commit replaces remix session with cookies

Looks good @CyrusVorwald, and I appreciate the efforts.
A question I have: why have all the JSDoc comments been removed? Also, have the refactors been done with some AI? As I know, those usually remove the JSDoc comments sometimes.
In any case, I need to give it a test, but overall, it looks good, especially if you can confirm that the tests successfully pass.

I used ai to help me understand the code and it removed the js doc. I can add them back in

It's okay, don't worry. I suggested that, as AI usually tends to remove the JSDocs. Sadly, those were a preference we had decided on some time ago for this repository/module.
Don't worry; I will need to check the implementation and test it myself too.

IMO is good to keep the comments, trying the latest version of rr and remix-auth pkg noticed this pkg is outdated, but the comments helped me out to understand what I need to do. ALso, great to see this is gonna be working with the latest rr version soon. Great Job!

@dev-xo I tested the pkg in local, using rr7 and the magic links seems to be broken. the sendEmail fn works, but the url and the verification is not working, always receiving error=Missing+email+to+verify.+Check+that+same+browser+used+for+verification. in the totp cookie.

[!NOTE]
I'm using the same browser and implementing your basic example
May be u wonder how I tested, I just forked the @CyrusVorwald repo and add an extra gh workflow adding pkg.pr.new, you can check it out here

I haven't had the chance to implement it in my application yet, but it might be because the success and failure redirects are set in the strategy directly in the latest version of remix-auth. For this reason, I instantiated two separate strategies in the unit tests in cases there were multiple success or failure redirects.

IMO, the success or error handling function should be something that user should write by his own, like in the github auth helper. Of course an example of basic handler for this auth would be great.

Gonna start by saying that, I'm not a big fan of having the whole Strategy refactored it with AI, and I'm also unsure how much of the work has been handled by it (as there were no comments anywhere, so I guess it handled the whole file).

With that said, I checked the code myself, cleaned it up, added the JSDocs, and overall, it looks good.

About the possible implementation:
As far as I understand, session will not be a thing anymore, and the user storage (previously Remix session) may be handled by the client (storing it in a cookie, based on the data the authentication returns).
I haven't been able to test the current implementation on a real app, and I'm also unsure how that should be set up either, although I guess it will be more or less the same as what we had before.
Will look a bit more into it, do some tests, and share some feedback.
Please, if you have the time and want to do the same, and share some insights, I would appreciate it, as getting this out would not only be great for us but probably for some other folks!

I marked this as draft because it has not been tested in a real app

Suggest leaving the implementation as it is, @CyrusVorwald, as it looks already good, and I want to continue it myself based on your feedback or @diecodev's one.
So please, leave it as it is, do not add new commits, and allow me to continue from here.

@dev-xo JIC, I was trying to implement something based on what you got here but only for rr7, may be that could help you in something. check it out here.

[!NOTE]
I'm not trying to create a npm pkg or something, I just used the sergiodxa/remix-auth-strategy-template to create a example to you. :D

Looks good, @diecodev! Does it already work? I haven't had the chance to give it a try yet. Hopefully, you can help us get this out based on your implementation and the current one!
Also, if you want to create a package from your implementation, feel free to do so! 👍🏻

@dev-xo The README.md has the example of usage. I'm currently using that implementation in my SaaS (WIP) just bc I need server-side verification.

Ah sorry, I only wanted to push that last commit to my account's version for my own purposes

All good @CyrusVorwald.
I will use my last commit on here to continue the implementation (I'm a bit busy right now). If you get a working one, based on your own criteria or @diecodev implementation, let me know.
I will also share some feedback on here the moment I am able to look a bit into it.

The last commit I force pushed works. I just added a function to update passed in cookies so that I could update the session cookie. I think it could be improved by removing the success and failure redirects but this works fine for me.


Pylance not using the correct conda/mamba environment in multi-root workspace

 Environment data

Language Server version: 2023.12.1
OS and version: linux x64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: off

 Code Snippet

my_root/test.code-workspace
{
	"folders": [
		{
			"name": "cfg",
			"path": "."
		}
		{
			"path": "./dir1"
		},
		{
			"path": "./dir2"
		}
	],
	"settings": {}
}

my_root/dir1/test1py
import sys
import pandas as pd # pylance says that pandas is installed (which is wrong)

print(sys.version)

# This file is using the mamba environment py310

my_root/dir2/test2.py
# print python version
import sys
import scipy  # pylance says that scipy is installed (which is wrong)

print(sys.version)

# This file is using the mamba environment py311

 Repro Steps

Create a multi-root workspace. I created a workspace with a structure like:

root
|____ test.code-workspace
|____  dir1
\____ dir2

Create two clean mamba or conda environments. In my case I created a py310 and a py311 environment.
Set the workspace conda environment for each.

 Expected
Pylance should show that scipy and pandas are not installed, as they are clean environments.
 Actual
Pylance shows that both are installed (I have them installed in my base py39 environment - I know this isn't ideal).
Additionally, if I try to use py310 features, such as type union syntax:

It reports that it is the wrong python version, even though the interpreter is set to the correct one:

If I then run the program using the play button, it works as expected. This suggests to me that it is an issue with pylance not recognising the conda environment being set.
 Logs

2024-01-12 13:45:01.562 [info] Experiment 'pythonaacf' is active
2024-01-12 13:45:01.562 [info] Experiment 'pythonRecommendTensorboardExtcf' is active
2024-01-12 13:45:01.562 [info] Experiment 'pythonREPLSmartSend' is active
2024-01-12 13:45:01.562 [info] Experiment 'pythonTerminalEnvVarActivation' is active
2024-01-12 13:45:01.562 [info] Experiment 'pythonTestAdapter' is active
2024-01-12 13:45:01.562 [info] Default formatter is set to ms-python.black-formatter for workspace /home/oliver/dev/multiroot_workspace_conda_issue
2024-01-12 13:45:01.562 [info] Default formatter is set to ms-python.black-formatter for workspace /home/oliver/dev/multiroot_workspace_conda_issue/dir1
2024-01-12 13:45:01.562 [info] Default formatter is set to ms-python.black-formatter for workspace /home/oliver/dev/multiroot_workspace_conda_issue/dir2
2024-01-12 13:45:01.562 [error] Following setting is deprecated: "python.linting.pylintEnabled"
2024-01-12 13:45:01.562 [error] All settings starting with "python.linting." are deprecated and can be removed from settings.
2024-01-12 13:45:01.562 [error] Linting features have been moved to separate linter extensions.
2024-01-12 13:45:01.562 [error] See here for more information: https://code.visualstudio.com/docs/python/linting
2024-01-12 13:45:01.562 [error] Please install "pylint" extension: https://marketplace.visualstudio.com/items?itemName=ms-python.pylint
2024-01-12 13:45:01.562 [error] Following setting is deprecated: "python.linting.pylintEnabled"
2024-01-12 13:45:01.562 [error] All settings starting with "python.linting." are deprecated and can be removed from settings.
2024-01-12 13:45:01.562 [error] Linting features have been moved to separate linter extensions.
2024-01-12 13:45:01.562 [error] See here for more information: https://code.visualstudio.com/docs/python/linting
2024-01-12 13:45:01.562 [error] Please install "pylint" extension: https://marketplace.visualstudio.com/items?itemName=ms-python.pylint
2024-01-12 13:45:01.562 [error] Following setting is deprecated: "python.linting.pylintEnabled"
2024-01-12 13:45:01.562 [error] All settings starting with "python.linting." are deprecated and can be removed from settings.
2024-01-12 13:45:01.562 [error] Linting features have been moved to separate linter extensions.
2024-01-12 13:45:01.562 [error] See here for more information: https://code.visualstudio.com/docs/python/linting
2024-01-12 13:45:01.562 [error] Please install "pylint" extension: https://marketplace.visualstudio.com/items?itemName=ms-python.pylint
2024-01-12 13:45:01.562 [info] Not enabling terminal env var experiment in multiroot remote workspaces
2024-01-12 13:45:01.562 [info] Test server listening.
2024-01-12 13:45:01.562 [info] Python interpreter path: ~/miniforge3/envs/py39/bin/python
2024-01-12 13:45:04.280 [info] Not enabling terminal env var experiment in multiroot remote workspaces
2024-01-12 13:45:06.144 [info] Starting Pylance language server.
2024-01-12 13:45:06.155 [info] > conda info --json
2024-01-12 13:45:12.230 [info] Python interpreter path: ~/miniforge3/envs/py310/bin/python
2024-01-12 13:45:12.797 [info] > conda run -n py39 --no-capture-output python ~/.vscode-server/extensions/ms-python.python-2023.22.1/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2023.22.1/pythonFiles/printEnvVariables.py
2024-01-12 13:45:12.798 [info] shell: bash
2024-01-12 13:45:13.077 [info] Not enabling terminal env var experiment in multiroot remote workspaces
2024-01-12 13:45:13.078 [info] Starting Pylance language server.
2024-01-12 13:45:17.296 [info] Python interpreter path: ~/miniforge3/envs/py311/bin/python
2024-01-12 13:45:17.424 [info] Not enabling terminal env var experiment in multiroot remote workspaces
2024-01-12 13:45:17.424 [info] Starting Pylance language server.
2024-01-12 13:45:22.902 [info] Python interpreter path: ~/miniforge3/envs/py39/bin/python

I have set the logging to trace, but don't see any DEBUG or TRACE statements. I set it for the user, workspace, and remote.
I have tested this on both a remote linux box, and WSL2 Ubuntu.

Thanks for the issue. Can you post the logs for this window too:

The logs above are from the Python extension. The Python Language Server logs are the pylance output.

This doesn't seem to reproduce for me with a virtual environment.

I also can't reproduce it with conda environments. The log should should show us what python interpreter we're getting though.
In my conda example, I have entries like so in the log:
2024-01-12 09:37:43.002 [info] [Info  - 9:37:43 AM] (7700) Setting pythonPath for service "root1": "C:\Users\rchiodo\AppData\Local\anaconda3\envs\py310\python.exe"
2024-01-12 09:37:43.002 [info] [Info  - 9:37:43 AM] (7700) Setting environmentName for service "root1": "3.10.0 (py310 conda)"
2024-01-12 09:37:43.087 [info] [Info  - 9:37:43 AM] (7700) Assuming Python version 3.10

And then another one later one when I set the other root:
2024-01-12 09:37:43.443 [info] [Info  - 9:37:43 AM] (7700) Setting pythonPath for service "root2": "C:\Users\rchiodo\AppData\Local\anaconda3\envs\py39\python.exe"
2024-01-12 09:37:43.443 [info] [Info  - 9:37:43 AM] (7700) Setting environmentName for service "root2": "3.9.0 (py39 conda)"
2024-01-12 09:37:43.539 [info] [Info  - 9:37:43 AM] (7700) Assuming Python version 3.9

What does it say for Setting pythonPath in your log?


Treat TL_messageEntityTextUrl and TL_messageEntityUrl the same way as TL_messageEntityTextURL and TL_messageEntityURL

Hi!
Browsing through a dump made with _TL_LAYER: 143 I encountered entities spelled as TL_messageEntityTextUrl and TL_messageEntityUrl instead of TL_messageEntityTextURL and TL_messageEntityURL.

Merged in https://github.com/3bl3gamer/tg_history_dumper/commit/5e220f3c71ca795447895f4fc680e6db5b8c5a31


Request: web UI showing promptfoo's version

It would be useful for debugging versioning if the web UI showed what version of promptfoo created the result
A useful spot could be under the Eval Actions dropdown, or in the title bar somewhere

This was added by @mldangelo in the about dialog


only truncate the original log file for rotation

other rotation log files will be overwrite, only the original log file need to truncate after copy.

Can you describe the issue it's fixing? How to reproduce it?

It avoid the unnecessary truncate when do log file rotation.
Existing rotation code work like this, let's say the max log files is 5:

delete app.log.5
copy app.log.4 to app.log.5, truncate app.log.4
copy app.log.3 to app.log.4, truncate app.log.3
copy app.log.2 to app.log.3, truncate app.log.2
copy app.log.1 to app.log.2, truncate app.log.1
copy app.log to app.log.1, truncate app.log

After this fix, it will only do truncate for the 6th step, only truncate the original log file:

delete app.log.5
copy app.log.4 to app.log.5
copy app.log.3 to app.log.4
copy app.log.2 to app.log.3
copy app.log.1 to app.log.2
copy app.log to app.log.1, truncate app.log

It seems we already have tests for that. It doesn't make sense to truncate rotated files because they will be overwritten anyway. Meanwhile, we need to truncate the original log file in case that some other process has a handle to this file and is writing something.

@samdark, I can't edit files in this PR, should I create a new PR to refactor current rotateFiles() method to make codeclimate happy? This PR doesn't make the method worse. I can make PR with a refactored code after this PR was merged.

@developeruz yes. Please do.

Replaced by #15173


failed to load python extension for lz4 support. lz4 compression will not be available

After installation, when import pyrobolearn, get the following error:
 Failed to load python extension for lz4 support. lz4 compression will not be available. pybullet build time: Oct 8 2020 00:10:04 Dart could not be found on this system... SKipping prl.simulators.Dart... ERROR: Expired activation key

Hi,
May you please be more specific about the OS you are using, how you are creating the virtual environment and any other steps you followed when facing this problem? That would be useful to try to reproduce the problem you are currently experiencing.
Best,
Leonel

Hi,
I have a similar problem. I want to know how to run examples/robots/xxx.py is the correct command.
`(env) wq@wq:~/pyrobolearn/examples/robots$ python load_robot.py
Failed to load Python extension for LZ4 support. LZ4 compression will not be available.
Some ROS packages were not found... Skipping prl.simulators.middlewares.ROS...
pybullet build time: Nov 26 2020 23:07:47
Dart could not be found on this system... Skipping prl.simulators.Dart...
MuJoCo could not be found on this system... Skipping prl.simulators.Mujoco...
Raisim could not be found on this system... Skipping prl.simulators.Raisim...
Traceback (most recent call last):
File "load_robot.py", line 18, in
import pyrobolearn as prl
File "/home/wq/pyrobolearn/pyrobolearn/init.py", line 31, in
from . import worlds
File "/home/wq/pyrobolearn/pyrobolearn/worlds/init.py", line 7, in
from .world import World, BasicWorld
File "/home/wq/pyrobolearn/pyrobolearn/worlds/world.py", line 15, in
import cv2
ImportError: /opt/ros/kinetic/lib/python2.7/dist-packages/cv2.so: undefined symbol: PyCObject_Type
`
Experimental environment: Ubuntu1604 python3.6

The examples in pyrobolearn/examples/robots/ are currently running using Ubuntu 16.04 and python 3.6. I am afraid you may have some issues regarding your virtual env. configuration, as import cv2 seems to be using python2.7 and not python3.6, as the error suggests:
import cv2 ImportError: /opt/ros/kinetic/lib/python2.7/dist-packages/cv2.so: undefined symbol: PyCObject_Type


Error with One Tap Login

``
version<EMAIL_ADDRESS>
But if i use this link's code(https://stackblitz.com/edit/github-7tvhqt?file=nuxt.config.ts,app.vue), I can get access_token

did you follow all the instructions correctly

did you follow all the instructions correctly

Yes

me too

@yaowangmx how do you fixed？

@yaowangmx how do you fixed？

I gave up this plugin and switched to using it instead<EMAIL_ADDRESS>

I gave up this plugin and switched to using it instead<EMAIL_ADDRESS>
can you give me a link about<EMAIL_ADDRESS>i'm from china,thank you

vue3-google-login

咦，遇到老乡了

vue3-google-login

咦，遇到老乡了

我就说你咋这会在线，看着像老乡 我vx: yangyao8 来真有缘 哈哈

vue3-google-login

咦，遇到老乡了：https://www.npmjs.com/package/vue3-google-login

这个怎么用到nuxt啊，直接引入模块报错

vue3-google-login

咦，遇到老乡了：https://www.npmjs.com/package/vue3-google-login

这个怎么用到nuxt啊，直接引入模块报错

解决了，非常感谢老乡，中国人帮助中国人 哈哈

Hi @jsony @yaowangmx
https://stackblitz.com/edit/github-erar5n?file=app.vue
in here i've tried exact same code as in the documentation to setup OneTap login
it seems to work for me

+1 Error with One Tap Login

Popup not displaying

this seems to be related to fedcm issue https://github.com/wavezync/vue3-google-signin/issues/74
tracking it in main package


Clarify the meaning of fields in Timezone

The documentation for Timezone is very terse and doesn't really explain what any of the fields mean. I presume these fields are taken from the Natural Earth dataset, but I couldn't find the relevant documentation for the dataset online.
It would be useful if either the meaning of these fields was clarified or if the docs of this crate linked to relevant documentation from Natural Earth that explains their meaning.
Some of the specific questions I had:

Is friendly_name the IANA TZ identifier?
Why is friendly_name an Option? What does it mean when it is None?
What is zone_str? Is it just zone_num as a string and if so why does it's example value differ from zone_num?
Why is zone_num an Option but offset_str isn't? What does it mean when zone_num is None?

Is friendly_name the IANA TZ identifier?

Yes, I can add this to the docs.

Why is friendly_name an Option? What does it mean when it is None?

There are multiple places that are considered a timezone that do not have an identifier; e.g., surrounding bases in Antarctica.

What is zone_str? Is it just zone_num as a string and if so why does it's example value differ from zone_num?

Honestly, for now, I am just passing this through from the underlying dataset.  I'm not sure why they differ.  Doing more research.

Why is zone_num an Option but offset_str isn't? What does it mean when zone_num is None?

This is a bug: fixing.

Fixed the bug, and clarified fields.


Use System.Buffers.ArrayPool

When profiling xamarin-android builds with the Mono profiler, I noticed
libZipSharp is one of the biggest allocators of byte[]:
Allocation summary
    Bytes      Count  Average Type name
251089192      57759     4347 System.Byte[]
    94852584 bytes from:
        Xamarin.Tools.Zip.ZipArchive:Close ()
        (wrapper managed-to-native) Xamarin.Tools.Zip.Native:zip_close (intptr)
        (wrapper native-to-managed) Xamarin.Tools.Zip.ZipArchive:stream_callback (intptr,intptr,ulong,Xamarin.Tools.Zip.SourceCommand)
        Xamarin.Tools.Zip.ZipArchive:stream_callback (intptr,intptr,ulong,Xamarin.Tools.Zip.SourceCommand)
        (wrapper alloc) object:ProfilerAllocVector (intptr,intptr)
        (wrapper managed-to-native) object:__icall_wrapper_mono_gc_alloc_vector (intptr,intptr,intptr)
    25673176 bytes from:
        Xamarin.Android.Tasks.ResolveLibraryProjectImports:Extract (System.Collections.Generic.IDictionary`2<string, Microsoft.Build.Framework.ITaskItem>,System.Collections.Generic.ICollection`1<Microsoft.Build.Framework.ITaskItem>,System.Collections.Generic.ICollection`1<Microsoft.Build.Framework.ITaskItem>,System.Collections.Generic.ICollection`1<Microsoft.Build.Framework.ITaskItem>)
        Xamarin.Android.Tools.Files:ExtractAll (Xamarin.Tools.Zip.ZipArchive,string,System.Action`2<int, int>,System.Func`2<string, string>,System.Func`2<string, bool>)
        Xamarin.Tools.Zip.ZipEntry:Extract (System.IO.Stream)
        Xamarin.Tools.Zip.ZipEntry:DoExtract (intptr,System.IO.Stream,Xamarin.Tools.Zip.EntryExtractEventArgs)
        (wrapper alloc) object:ProfilerAllocVector (intptr,intptr)
        (wrapper managed-to-native) object:__icall_wrapper_mono_gc_alloc_vector (intptr,intptr,intptr)
    3780312 bytes from:
        Xamarin.Tools.Zip.ZipEntry:DoExtract (intptr,System.IO.Stream,Xamarin.Tools.Zip.EntryExtractEventArgs)
        (wrapper managed-to-native) Xamarin.Tools.Zip.Native:zip_fread (intptr,byte[],ulong)
        (wrapper native-to-managed) Xamarin.Tools.Zip.ZipArchive:stream_callback (intptr,intptr,ulong,Xamarin.Tools.Zip.SourceCommand)
        Xamarin.Tools.Zip.ZipArchive:stream_callback (intptr,intptr,ulong,Xamarin.Tools.Zip.SourceCommand)
        (wrapper alloc) object:ProfilerAllocVector (intptr,intptr)
        (wrapper managed-to-native) object:__icall_wrapper_mono_gc_alloc_vector (intptr,intptr,intptr)

This seems like a lot of byte[] allocations.
To improve this, I added a <PackageReference/> to System.Buffers
and could make use of ArrayPool.
libZipSharp will likely still allocate some large byte[], but things
should improve because many will be reused.
The changes appear to have saved ~122,523,744 bytes of allocations:
Allocation summary
    Bytes      Count  Average Type name
Before:
251089192      57759     4347 System.Byte[]
After:
128565448      31764     4047 System.Byte[]

If I compare stream_callback, it has massively less allocations:
Before:
94852584 bytes from:
    Xamarin.Tools.Zip.ZipArchive:Close ()
    (wrapper managed-to-native) Xamarin.Tools.Zip.Native:zip_close (intptr)
    (wrapper native-to-managed) Xamarin.Tools.Zip.ZipArchive:stream_callback (intptr,intptr,ulong,Xamarin.Tools.Zip.SourceCommand)
    Xamarin.Tools.Zip.ZipArchive:stream_callback (intptr,intptr,ulong,Xamarin.Tools.Zip.SourceCommand)
    (wrapper alloc) object:ProfilerAllocVector (intptr,intptr)
    (wrapper managed-to-native) object:__icall_wrapper_mono_gc_alloc_vector (intptr,intptr,intptr)
After:
243168 bytes from:
    (wrapper native-to-managed) Xamarin.Tools.Zip.ZipArchive:stream_callback (intptr,intptr,ulong,Xamarin.Tools.Zip.SourceCommand)
    Xamarin.Tools.Zip.ZipArchive:stream_callback (intptr,intptr,ulong,Xamarin.Tools.Zip.SourceCommand)
    System.Buffers.DefaultArrayPool`1<byte>:Rent (int)
    System.Buffers.DefaultArrayPool`1/Bucket<byte>:Rent ()
    (wrapper alloc) object:ProfilerAllocVector (intptr,intptr)
    (wrapper managed-to-native) object:__icall_wrapper_mono_gc_alloc_vector (intptr,intptr,intptr)

I saw build performance improvements for the Xamarin.Forms integration
project, the two tasks heavily using libZipSharp:
Before:
1881 ms  ResolveLibraryProjectImports               1 calls
3406 ms  BuildApk                                   1 calls
After:
1795 ms  ResolveLibraryProjectImports               1 calls
3150 ms  BuildApk                                   1 calls

I would guess this saves ~350ms on an initial build. Incremental builds
won't be allocating byte[] as heavily, but should see some improvement.
The only drawback here would be the dependency on System.Buffers.

This looks good. I am beginning to wonder if those Utf8String marshalling methods might be more performant using Span<byte> to process the native memory directly, rather than copying it over.
That said the Span.ToArray seems to do a managed creation of an array anyway. So it won't buy us anything.
Or perhaps even better?
public unsafe static string Utf8StringPtrToString (IntPtr utf8Str)
        {
            if (utf8Str == IntPtr.Zero)
				return null;

			byte* ptr = (byte*)utf8Str.ToPointer ();
			byte* p = ptr;
			while (*p != 0)
				p++;
			var len = (int)(p - ptr);
			if (len == 0)
				return String.Empty;

            return Encoding.UTF8.GetString ((byte*)utf8Str.ToPointer (), len);
        }

Just use the UTF8.GetString (byte* ptr, int len) override instead and do away with the buffers completely?

@jonathanpeppers  I've been playing with System.Buffers on another app, and I'm getting a bunch of dependency issues with netcore 3.1. It keeps trying to load System.Buffers <IP_ADDRESS> instead of the 4.5.0 one I've specified. Also I can't seem to get it to include the dll in the local build. Are we sure this is not going to break anything?

When I tested with xamarin-android, I did this:
$ msbuild libZipSharp.csproj /p:Configuration=Release
$ cp bin/Release/netstandard2.0/libZipSharp.dll ../xamarin-android/bin/Release/lib/xamarin.android/xbuild/Xamarin/Android/

I could see my changes working, because I got different stack traces in the profiler.
Is there an easy way to try out the nupkg on a xamarin-android PR build? I can do it locally, but it won't be exactly the same as doing a full PR build that tests everything.


dart:io stdin is a non-broadcast stream

How exactly is this supposed to work? A global stream that only supports a single listener is not very scale-able. I'm trying to run a process interactively from within dart and this makes it very hard to do that reliably.
What am I missing? It seems that the current dart:io implementation does not make sense.
Thanks,
Andreas

There is only one stdin in the process, and when you consume input, it's gone.
If you want multiple concurrent listeners on input, you can use Stream.asBroadcastStream().
If you want to look at the input and do things depending on the input, you can look at StreamQueue in package:async which makes it easier to act on a stream one event at a time.

tl;dr: Having a global non-broadcast stream is still stupid and your design is inconsistent. Your answer is an obvious workaround but it does not fix the underlying conceptual issue at hand
Maybe you have misunderstood me. Having a global stdin variable means that is is impossible for a library to use it by itself because that code cannot know if stdin has already been listened to or not.
Moreever, readLineSync and readByteSync don't have this issue at all. You can call them from anywhere. This is inconsistent.
How is anyone supposed to use the "Stream" stdin at all in a library/shared code? I guess the answer is not at all? Is this part of Dart a priority at all or should we look at other scripting languages in general for console scripting?

You can't really use stdin from a library, for the reasons you state.
I think that's OK. The library should not access stdin directly, it should take a stream as a parameter, and then the user can choose to use stdin - or some other stream.
Making the stream broadcast isn't going to make anything better. Instead you'll likely find that some of the input has been consumed before you even get to listen to it. A library can still not assume that it can listen to stdin - it has no idea where it is in the stdin stream.
The stdin stream is a "generated" stream. It can't be replayed, it can't be rewound, when you use it it is consumed in the process. the stream's events form a consecutive whole, and dropping events leaves you in an uncertain state (maybe in the middle of a line).
Streams like that are always single-subscription streams.

Thanks for the explanation! I'll try to do that then and use StreamQueue.
On Wed 11 Nov 2015 at 16:31 Lasse R.H. Nielsen<EMAIL_ADDRESS>wrote:

You can't really use stdin from a library, for the reasons you state.
I think that's OK. The library should not access stdin directly, it should
take a stream as a parameter, and then the user can choose to use stdin -
or some other stream.
Making the stream broadcast isn't going to make anything better.
Instead you'll likely find that some of the input has been consumed before
you even get to listen to it. A library can still not assume that it can
listen to stdin - it has no idea where it is in the stdin stream.
The stdin stream is a "generated" stream. It can't be replayed, it can't
be rewound, when you use it it is consumed in the process. the stream's
events form a consecutive whole, and dropping events leaves you in an
uncertain state (maybe in the middle of a line).
Streams like that are always single-subscription streams.
—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/24881#issuecomment-155817309.


Warnings for watchman sensors in HA 2023.03

Warning: The unit of sensor.watchman_missing_entities is changing, got multiple (None, 'items'), generation of long term statistics will be suspended unless the unit is stable and matches the unit of already compiled entities (Items).

Warning: State attributes for sensor.watchman_missing_entities exceed maximum size of 16384 bytes. This can cause database performance issues. Attributes will not be stored.

I think this is the same as #87 https://github.com/dummylabs/thewatchman/issues/87 but I have the same issue.
Not noticed it in the past.
Is Watchman the problem or did HA pull something that's made our missing entities explode...

I think this is the same as #87 #87 but I have the same issue. Not noticed it in the past.

Part is a duplicate of #87, part is not.
I suggest to rename the issue to something like "Warning caused by non-stable unit {None, 'items'}" to address the other warning.
My log on 2023.7.1:
2023-07-11 17:50:10.265 WARNING (Recorder) [homeassistant.components.sensor.recorder] The unit of sensor.watchman_missing_entities is changing, got multiple {None, 'items'}, generation of long term statistics will be suppressed unless the unit is stable and matches the unit of already compiled statistics (items). Go to https://my.home-assistant.io/redirect/developer_statistics to fix this
Simone


Error Internal Service Error

Ok I think this is more user error than anything.  However, as a suggestion the errors should be more descriptive.  You need to have stake to send tokens and one more other error is if you use uppercase letters in the account name you will get an error.  Something to check for as well if you want this wallet to be more user friendly.
Thanks,
~S
I get this error when trying to send EOS from one account to another.  Can you help?

I think I found the issue.  Here is what I get when I try to vote.

However I can't stake my coins

I'm in a catch 22?

I’ll also address the other issues mentioned tomorrow, I think we have most of them working already. I didn’t see the full extent of your comments in the initial email from GitHub!

No worries.  I love the wallet.  A few suggestions:

This is more than a voter wallet so you might think about changing the name eos-voter?
As you can see from some of the errors above I didn't realize I needed to stake enough token in order to send.  So,  I'm not sure if you can check for the user before they try any of the functions like voting and sending EOS a head of time and pop up a warning that the need to stake for CPU and Bandwidth
Lastly it would be good if you can check for upper-case letters in account names so user knows that only lower-case works.  or force a routine to auto lower-case?

Thanks again.
~S

A few answers for ya, based on the previous comments.

We are in the process of rebranding/renaming now, and having some art assets created.
It's a hard thing to nail down, because it depends on the usage on the entire network. We do see it as an issue though and are looking for ways to help notify users if their staked balance is too low (0.2.0 has warnings now if you try to unstake too much).
Good suggestion, we'll get that tied in to everywhere we allow username inputs.

If you ended up in a state with 0 staked in CPU/NET, you won't even be able to increase your stake, and will need another account to delegate some resources to fix the problem. It's a weird situation and some BPs have instructed people to set their stake to 0 (which we wouldn't recommend, since you can't do anything), so that's why we're now adding warnings on any actions that set the staked amount below 1.

Awesome.  Look forward to the version update.  Keep up the great work!
U rock!

The account name inputs are all lower case now - and I've opened issue #231 as a "catch all" for adding some sort of warnings to help guide users on resources within their account.
Closing this one out for housekeeping purposes!

U Rock!


Getting RIghtArm is not listed when selecting new motion

Hey,
I am getting the following error when running new motions on already annotated characters:
joints_idxs = [self.bvh_joint_names.index(joint_name) for joint_name in joint_names]

ValueError: 'RightArm' is not in list
This was not an issue with the Dab bvh, why is it causing an error with the other selected motion and how can I solve this error?
thank you for the support and this cool tool!

Hello TadewosBell,
See #133.
I'm closing this issue, but feel free to reopen if those comments don't solve your problem.


In iOS 15, Navigation bar not hiding in tab screen

@iosYash Have you tried .uipNavigationBarIsHidden(forIos15Only: true) this or .uipNavigationBarHidden(true) It will work .

It's fixed.


[improve][broker] Reduce unnecessary persistence of markdelete

Motivation
Reduce unnecessary persistence of markdelete：

When mdEntry.newPosition is equal to persistentMarkDeletePosition, this persistence should be skipped：
https://github.com/apache/pulsar/blob/36806853b5a445299d8f9a6ba89905c1915345a3/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java#L1971-L1980

It should be modified to:
if (persistentMarkDeletePosition != null
          && mdEntry.newPosition.compareTo(persistentMarkDeletePosition) <= 0)

When mdEntry.newPosition is equal to INPROGRESS_MARKDELETE_PERSIST_POSITION_UPDATER, this persistence should also be skipped：
https://github.com/apache/pulsar/blob/36806853b5a445299d8f9a6ba89905c1915345a3/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java#L1982-L1999

It should be modified to:
if (current != null && current.compareTo(mdEntry.newPosition) >= 0)

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@lhotari PTAL,thanks!

The change looks good me.
It's better to add a test since this looks like a boundary problem. We should have a test to enforce correct behavior

OK , I will fixed

persistentMarkDeletePosition

I will add test

Close this PR.
I submitted this change to https://github.com/apache/pulsar/pull/18042 because they are the same issue.


Dragging stopped working since 0.13.6 update

Since we updated library from 0.13.5 to 0.13.6 dragging for all widgets stopped working

Can you check the most recent version?  I think there was an issue with the drag if the click event occurred on a child element of a elements matching the handle selector.

I tried with the latest.  It does let you drag the "handle" dom node, but children of the handle prevent dragging, which is different from 0.13.5.

@danomatic Should you close this issue since it has been resolved?


Can't use config variables in aliases

The config variables are working fine elsewhere, so I know they're loading properly, but with the below setup, I'm getting an error.
Gruntfile.js
var mrPig = grunt.file.readYAML('mr_pig/settings.yaml');

require('load-grunt-config')(grunt, {
  configPath: path.join(process.cwd(), 'mr_pig/core/tasks'),
  overridePath: 'mr_pig/tasks',
  config: {
    m: mrPig
  }
});

mr_pig/settings.yaml
default:
  - serve

mr_pig/core/tasks/aliases.yaml
default:
  tasks: '<%= m.default %>'

Error message from running grunt
Running "default" task
Warning: Task "<%= m.default %>" not found. Use --force to continue.

Aborted due to warnings.

Should I be able to use the config variables in aliases.yaml?

Hi @curtisblackwell, as far as I know templates are not supported in Grunt registerTask, only in initConfig. So unfortunately you are not able to use templates in aliases because it uses Grunt registerTask inside. :(

Ah, got it. Thanks!


Allow @ in Instagram

Is it possible to allow Instagram user with @?

Why?
Instagram links don't work with @
Check this example:
https://www.instagram.com/bitcoin
https://www.instagram.com/@bitcoin

That's true, however, as user mentions on Instagram uses @, some users refers to profiles with @.
It could be valid and the sanitized url redirect to the valid form:
Profile: @test
isValid: True
sanitizedUrl: https://www.instagram.com/test

I get it. You mean profileId alone.
I fixed it in v1.3.1 patch:
https://www.npmjs.com/package/social-links/v/1.3.1


import issue tensorflow v2 and v1

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

Windows Server 2016 Standard:
CPU Xeon (R) CPU E5-2698 v4
AVX2
TensorFlow installed from (source or binary): binary
TensorFlow version: 2.0.0-alpha0
Python version: 3.6.8
Installed using: Tried with: virtualenv, pip and  conda

Describe the problem
Installation works fine but  import tensorlfow does not work
Provide the exact sequence of commands / steps that you executed before running into the problem
import tensorflow as tf
Any other info / logs
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\site-packages\tensorflow\__init__.py", line 27, in <module>
    from tensorflow._api.v2 import audio
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\site-packages\tensorflow\_api\v2\audio\__init__.py", line 8, in <module>
    from tensorflow.python.ops.gen_audio_ops import decode_wav
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\site-packages\tensorflow\python\__init__.py", line 49, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 74, in <module>
    raise ImportError(msg)
ImportError: Traceback (most recent call last):
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\imp.py", line 243, in load_module
    return load_dynamic(name, filename, file)
  File "C:\ProgramData\Anaconda3\envs\testtf\lib\imp.py", line 343, in load_dynamic
    return _load(spec)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.

Failed to load the native TensorFlow runtime.

See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

@ivanjacobs Did you Download and install the Microsoft Visual C++ 2015 Redistributable Update 3. Also have look https://www.tensorflow.org/install/pip (select windows). Thanks!

Installed Microsoft Visual C++ 2015 Redistributable Update 3  and did all the steps under  https://www.tensorflow.org/install/pip (select windows).
reintsalled python,pip,virtualenv nothing helps. The only version that runs is the tensorflow==1.5
Log:
C:\code\tensorflowtest\tensorflowtest\Scripts\python.exe C:/code/tensorflowtest/test.py
Traceback (most recent call last):
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Program Files\Python\Python36\lib\imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "C:\Program Files\Python\Python36\lib\imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:/code/tensorflowtest/test.py", line 2, in
import tensorflow as tf
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow_init_.py", line 27, in
from tensorflow.api.v2 import audio
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow_api\v2\audio_init.py", line 8, in
from tensorflow.python.ops.gen_audio_ops import decode_wav
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow\python_init_.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\code\tensorflowtest\tensorflowtest\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Program Files\Python\Python36\lib\imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "C:\Program Files\Python\Python36\lib\imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

The only other reason I can think of is, if you installed the GPU version of TF without having a GPU. Other than that, CPU not having AVX and MSVC redistributable are the only cases I have seen this failure.

@ivanjacobs  Is this still an issue?

@gadagashwini
This is still an issue.
@gunan I am confident that I didn't install gpu version.
the only version of tf I was able to install is the one compiled by intel
https://software.intel.com/en-us/articles/intel-optimization-for-tensorflow-installation-guide

Your cpu does not support AVX instruction sets.
See https://ark.intel.com/content/www/us/en/ark/products/91753/intel-xeon-processor-e5-2698-v4-50m-cache-2-20-ghz.html
TF 1.6 and above binaries require AVX instruction sets.
See https://www.tensorflow.org/install/pip#hardware-requirements


526 mybabb get ogimage tag to work

Netlify is not refiguring it.   I get the same results after I change it in the docusaurus config,   If i use the old #528 preview then it still works fine.   It  is still deployed.   The LoremPicsum generates a new image every single time on the Linked in test site.  But now when I change it, it  re compiles it the same way every time that does not use the image source that I put in but tries to insert one that it will not recognize.   I am going to remove the config meta date now, I will just try inserting the meta data into the specific page of Markdown with Front Matter.


Steps to Reproduce:

Open a TTY (on Linux, press Ctrl+Alt+F1)
git clone https://github.com/microsoft/vscode-extension-samples
cd vscode-extension-samples/helloworld-test-sample
npm install
xvfb-run -a npm test

Expected behavior
The tests succeed
Actual behavior
The tests fail
Additional information
The xvfb-run command was taken from the docs.
Install xvfb with sudo apt install xvfb on Ubuntu
Seems like extension tests cannot be run in an environment without graphic interface
Seems related to #103060 and #103113 but I don't use snap and those issues are closed
Complete trace
Found .vscode-test/vscode-1.51.1. Skipping download.
Warning: 'sandbox' is not in the list of known options, but still passed to Electron/Chromium.

[main 2020-12-04T13:01:47.387Z] update#setState idle

[main 2020-12-04T13:01:47.388Z] update#ctor - manual checks only; automatic updates are disabled by user preference

/dev/fd/3: Aucun fichier ou dossier de ce type

Server response:

/dev/fd/3: Aucun fichier ou dossier de ce type

Server response:

/dev/fd/3: Aucun fichier ou dossier de ce type

Server response:

/dev/fd/3: Aucun fichier ou dossier de ce type

Server response:

/dev/fd/3: Aucun fichier ou dossier de ce type

Server response:

[26655:1204/140147.684597:FATAL:gpu_data_manager_impl_private.cc(439)] GPU process isn't usable. Goodbye.

/dev/fd/3: Aucun fichier ou dossier de ce type

Server response:

/dev/fd/3: Aucun fichier ou dossier de ce type

Server response:

/dev/fd/3: Aucun fichier ou dossier de ce type

Server response:

Exit code:   null
Done

Failed to run tests

Version: 1.52.0-insider
Commit: 92192baf760315ac47fb2caaf3ce6d0778962352
Date: 2020-12-03T05:44:34.000Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 4.15.0-123-generic

Does this issue occur when all extensions are disabled?: Yes

I think we have https://github.com/microsoft/vscode-test/blob/master/sample/azure-pipelines.yml as an example for how to configure Azure pipelines to run extension tests.

Thank you, it works
Seems like xvfb-run doesn't work in a TTY, but starting an Xvfb server does.
Some notes

It makes some extensions crash, so I launch test with extensions disabled (launchArgs: ["--disable-extensions"])
You might want to add a TTY section in this document if I'm not the only one doing this

Future developers might want to try this quickly-written script
#!/bin/bash
set -eux

# Start server
/usr/bin/Xvfb :99 -screen 0 1024x768x24 &
sleep 1

# Make sure it's running
ps aux | grep Xvfb --color=always | grep -v grep

# Run tests
DISPLAY=:99.0 node ./dist/test/runTest.js

# Kill server
killall Xvfb


Error response from daemon: could not select device driver "nvidia" with capabilities: [[gpu]]

Hey,
installation went fine, no problems whatsoever. However, when trying to run launch.sh I get following error:
➜ ./launch.sh
[+] Running 1/0
 ⠿ Container fauxpilot-copilot_proxy-1  Running
Attaching to fauxpilot-copilot_proxy-1, fauxpilot-triton-1
Error response from daemon: could not select device driver "nvidia" with capabilities: [[gpu]]

I have all dependencies installed:

docker (v. 20.10.17, build 100c70180f)
docker-compose (v. 2.6.1)
nvidia driver (v. 515.57, CUDA v. 11.7)
nvidia-docker (v. 2.11.0-1)

Running on Arch Linux, kernel: 5.15.55-1-lts

Looks like the same issue as this: https://github.com/NVIDIA/nvidia-docker/issues/1034
Could you try the steps mentioned there?

Looks like the same issue as this: NVIDIA/nvidia-docker#1034
Could you try the steps mentioned there?

Awesome it worked, thanks.


The commands stop working if one of the scripts has been removed

Take the following picture for example, I removed HUD Color Editor and none of the commands work:

I'm using master/1695 from AppVeyor.

Even if I abort the script, SHVDN keeps complaining about it:

Looks like is related to aborting a single script, because if I abort all of them GGOV is not listed:


explorer is not working for subsocial in localnet

Overview
explorer is not working for subsocial
Steps to Reproduce
Steps to reproduce the behavior:

run dive chain polkadot -p subsocial --explorer --metrics
Try to open explorer with subsocial endpoint

Expected Behavior
EXplorer to give details

verified and closing this issue as fixed


Changes to allow setting of MTU size and jumbo frames.

With this change the environment variable "want_mtu_size" can be used to set desired MTU size for Admin and Storage nodes. Setting want_mtu_size value to "jumbo" makes it to take the maximum MTU size feasible (8900) for these nodes. The qa_crowbarsetup.sh is modified to change the 'network' proposal to set the MTU value.

:+1: lgtm

+1 (not sure if we need the --raw attribute but i think this is minor)

please rebase to current master and use the new onadmin_help function


[Feature]: Forbid adding/updating users for deleted organization

Is there an existing issue for this?

[X] I have searched the existing issues

Description
If the organization is deleted, then it should not allow adding users to it. Updating existing users should also be forbidden.

If the organization is deleted, we are deleting all its users using database trigger, so there would be no user left who would invite, add, or update users for the org.
So closing this issue as invalid.


[Proposal] Set GitHub website url

Click the edit wheel on the right of "About" here and add:
https://neelnanda-io.github.io/TransformerLens/
Maybe add a one-liner description as well (e.g. "Mechanistic interpretability library for language models like GPT).
One for Neel I think (I don't have permission to change this).

Thanks, fixed! (For future reference, feel free to slack me about things like this)


Notebooks 1 and 4-7 for the LSST workshop.

Please make the following changes, then commit and push again:

do not git-track any *pyc files (i.e. git rm them)
remove all pycache directories and everything in them from tracking
please be sure that all other tracked file are there intentionally (I see several *png, *jpg, *jpeg, *gif, and *pdf files; are they required for the notebooks?)
Can you standardize the supplementary image files (if they are indeed needed) to be just *png, or just *jpg? If not, please at least use only *jpg and not both *jpg and *jpeg
Please standardize imports; at DL we use (after looong discussions ;-)) import numpy as np for instance (i.e. not import numpy as N)
The TeenAstronomyCafe/README.md files speaks of "public notebook server", which is obsolete information; please update the README.md file.

Closing without merging, since it this PR was superseded by PR #14


nested jdbc calls can leave you in deadlock state

because there are nested jdbc calls in JdbcOAuth2AuthorizationConsentService and JdbcOAuth2AuthorizationService you can easily wind up in a deadlock state where no progress can be made if your pool has all connections in outer call.
easily reproducible if you make your max-pool size = 1
hikari:
maximum-pool-size: 1
you can never finish.
solution is to not nest jdbc calls, but make it sequential always, see attached potential fix.
JdbcOAuth2AuthorizationService.java.txt
JdbcOAuth2AuthorizationConsentService.java.txt

@jgrandja ok. I was just trying to illustrate the point by making the pool 1 so it's very obvious.
How do you fine tune this away? Even If you 100 connections in the pool and you spike traffic and each connection goes out and gets used in stage1 stuck waiting for the nested connection, the only way forward is for 1 or more to fail?
I do think people will wind up using this in production and it will bite them, so at least this will serve as reference to those who come later trying to figure it out.


Modal has a bug when triggered with javascript

Uncaught TypeError: BACKDROP: Option "rootElement" provided type "null" but expected type "element".
at index.js:130
at Array.forEach ()
at l (index.js:124)
at xe._getConfig (backdrop.js:92)
at new xe (backdrop.js:33)
at Ne._initializeBackDrop (modal.js:218)
at new Ne (modal.js:84)
at S (main.js:1)
at HTMLInputElement.onSelect (main.js:1)
at b.onSelect (jquery.autocomplete.min.js:8)

Duplicate of #33840


Use redux-electron-store

https://github.com/samiskin/redux-electron-store

This would insta-fix https://github.com/mozilla/tofino/issues/1354


[Tracking] Cron job for updating the registry

https://github.com/paritytech/asset-transfer-api-registry/pull/98 sets the cron job to every 24 hours, and once https://github.com/paritytech/asset-transfer-api/pull/327 goes in we can revert the cron jobs to every hour

For now I think 24 hours seems to be working well. Closing this for now as we don't need to set it to every hour.


Unknown command "migrate"

Hi @aronasorman & @MCGallaspy this fixes #61
I add the KALITE_DIR to windows environment variables and update the name of pip dependency zip file.

@aronasorman I have difficulty to use glob in pip dependency zip file name during the installation, Any other idea on this?

I believe glob should be included by default in python. If glob doesn't work, try using fnmatch instead.

I'm not convinced this is necessary. I have not had this issue, albeit I have only built and tested the installers on Windows 8. I would like someone to reproduce this issue on XP (assuming that is the problem) using a known good installer.

@aronasorman You commented on my old PR that the zip file name is hardcoded. So I find a way to use glob as your suggestion but its hard for me to implement that on pascal. But I observe that there's MyVersion variable so I used it on the zip file so  is not hardcoded.

@MCGallaspy I asked @djallado to reproduce this issue on Windows XP tomorrow. Hope he might give us feedback.

@amodia Good find! :+1:
Minor request: Can you please change getKALiteVerion() to getKALiteVersion() and push your commit on this same PR?  It's wrong and confusing.  Here's the source.

@amodia the hardcoded version number is a separate issue. Can you open a PR with that fix?

Oops, sorry, silly me! Disregard the previous comment!

Stale PR, let's re-create if issue persists.


Implement initial support for automated testing

This should not impact normal usage of the app

Just a couple minor comments, otherwise looks good. We can defer dealing with how exactly to destroy the audio engine to later, after we implement repeats for the mic tests.

PTAL

Looks good, merging.

+1


Stricter regex for collapsing images in text tutorial hints

fixes https://github.com/microsoft/pxt-arcade/issues/2723

Also fixes https://github.com/microsoft/pxt-arcade/issues/2220

Also fixes https://github.com/microsoft/pxt-arcade/issues/2264


Favicon

When the Favicon image changes, it is new at the top of the browser but too new at the bottom of the page footer - which is a pity..
The original Bludit Favicon is nice and should stay in the footer.
Steps to reproduce the problem
Complete here.
Bludit version
Complete here.
PHP version
If you do not know delete this line.
PHP logs
If you do not know delete this line.
The default settings of the PHP Error Log file varies from OS to OS. The location of the error log file itself can be set manually in the php.ini file. On a Windows server, in IIS, it may be something like error_log = C:\log_files\php_errors.log in Linux it may be a value of /var/log/php_errors.log.

Let me know which theme, because is more of an issue with the theme.


Bug when tries to make changes to published API

Test Scenario:
After API is Published, ensure a user with publish permission can do modifications to:
Transports
Subscription tiers
Subscription availability
Description:
When an user with internal/Publisher permissions are unable to change to Transports of the API when published

Addressed in https://github.com/wso2/product-apim/issues/6032.
And publisher is not allowed to update transport info.


New keyboard shortcut to commit without pushing

Added a keyboard shortcut for commit only (now set to alt+shift+s by default), in addition to the already existing options. For those cases when Logseq is used with local git.
This partially takes care of #40, albeit manually.

An additional checkStatus() is needed

Done

Thank you all for your efforts, sorry for the late reply.


Move es6-promise from devDependencies to dependencies

Using the wrap method with a Promise breaks when no global Promise is defined. Although since you have es6-promise in your devDependencies all your tests pass.

How are you using Promises?  I was assuming people would use a poly-fill to set globals.Promise.  e.g.,
require('es6-promise').polyfill();

We explicitly require the promise library, in our case it is bluebird, and don't rely on the Promise global.    I think it would make sense to include a promise library as dependency and use it in case global.Promise is not found.

Another option would be to allow an optional Promise polyfil to be passed into the constructor. That way, this should work with any A+ compliant Promise library without an additional dependency being installed. So in our case, we wouldn't need es6-promise as a dependency if we already have bluebird. h/t @siddharthkchatterjee for this suggestion.

Good idea.  I like the idea of passing in the Promise dependency.

@BryanDonovan, take a look at #55 please.


local-storage

Thanks for sharing. It is a great repo.
I would like to recommend you using ngrx-store-localstorage instead.

That's a good point, will look into that if i find some time. Feel free to submit a PR if you wanna look into that 😉.

local-storage loads in the initial states saved, but on @ngrx/effects/init for root action event it removes feature states that are yet to be loaded in and only the root state gets hydrated. my question does it support lazy-loaded feature states. thanks for sharing


Autoreconnect bug

It doesn't work. Again.

Related to (but different from) #917?

It works at first disconnect, but fails on second.

So, problem is in irc-framework, line this.socket.end();
I'll make a PR later, after some tests.

Alright, if it's an issue with irc-framework, I'll close this. We'll pull in the changes whenever irc-fw releases.

@bews could you make an irc-fw issue?

https://github.com/kiwiirc/irc-framework/issues/96


validBuildingPlacement method does not account for the distance rule

👍 it does now

Yeah this is not working anymore...


Test framework

Create an automated testing framework for all nodes and implement tests across the the nodes. This breaks down into:

[ ] Automated tests for redioactive.
[ ] Automated tests for nodes.
[ ] Automated tests for classes without coverage.

Current plan is to add black box testing based around FIMS-like jobs that create and destroy infrastructure.


"User is already active!"  Error - Line 232

OctoberCMS Build: 455
PHP Version: 7.1.27
Database Engine: mysql
Plugins Installed: RainLab.User

Description:
Reactivating a user from the List View throws:
"User is already active!"
error on line 232 of vendor/october/rain/src/Auth/Models/User.php
Steps To Reproduce:

Create User
Activate User
Deactivate User
Activate User

Yup I can confirm that this is happening to me as well.

Fixed as of https://github.com/rainlab/user-plugin/commit/9edc9556b0e14aa1d44072b3f9ecd7a3ef6fb8ab. Will be released shortly in edge updates with v1.5.0 of the User plugin.

This message is a bit too much for the display
"User is already active!" on line 232 of /home/xxx/public/vendor/october/rain/src/Auth/Models/User.php how can i get rid of the full path from the message popup

@skynetweb Do you have debug enabled in your config?

I have the same problem. Debug mode is disabled.

@XIOLog Are you saying the same problem as OP? Or the same issue with the full message being shown on the message popup?


It is expected to occur with other types of deaths as well.

I'm not familiar with English or issues, so if there's anything else I should say, please let me know.

I cannot reproduce this.

Minecraft 1.16.5
Forge 36.28
Crock Pot 1.6.2-release
Patchoiuil 1.16.4-53.2
https://user-images.githubusercontent.com/41111170/145147554-d7efa477-17fe-42b4-84fe-c7f1480b5225.mp4
The same applies to English as well as Japanese.

I don't know which version of Forge 36.28 is, so I used the latest version of Forge (36.2.19) that applies to 1.16.5. I use the same version of the mods as yours. I still can't reproduce the bug, even in a Japanese environment.

I do wonder why, maybe it's your version of Forge? What about trying to update Forge to the latest version?

We think it needs to be right after the attack.
It still happens in 36.2.19


fix(1548): correct import of escaped curl string

Description
Fixes #1548

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Merged!
Thank you for taking care of this @carlwilk32 !


Possible bug in ecto.migrate command when used with --prefix options

When I run mix ecto.migrate --prefix private, the command fails if no schema_migrations table exists in the public prefix.
This bug description as well as a complete repo to reproduce the bug are available here: https://github.com/maxmarcon/ecto_migration. The "Steps to reproduce" section refers to that repo.
Environment

Elixir: 1.12.2
Erlang/OTP: 24
Postgres: 13-alpine running in docker container
OS: MacOS 11.4
Dependencies:

ecto: 3.7.0
ecto_sql: 3.7.0
postgrex: 0.15.10
Others: see mix.lock file

Steps to reproduce:

mix deps.get
docker compose up -d
mix ecto.create
Create schema private. You can use your favorite DB client or psql with:

psql -h localhost -U postgres ecto_migration
Enter postgres as password
CREATE SCHEMA private;

mix ecto.migrate --prefix private

Expected behavior

schema_migrations and sample_table are created in the private schema
schema_migrations contains the timestamp of the migration that created sample_table

Actual behavior

schema_migrations is created in the private schema
sample_table is not being created
schema_migrations is empty
mix ecto.migrate fails with the error:

** (MatchError) no match of right hand side value: {:error, %Postgrex.Error{connection_id: 5373, message: nil, postgres: %{code: :undefined_table, file: "namespace.c", line: "431", message: "relation \"schema_migrations\" does not exist", pg_code: "42P01", routine: "RangeVarGetRelidExtended", severity: "ERROR", unknown: "ERROR"}, query: nil}}
(ecto_sql 3.7.0) lib/ecto/adapters/postgres.ex:220: anonymous fn/3 in Ecto.Adapters.Postgres.lock_for_migrations/3
(ecto_sql 3.7.0) lib/ecto/adapters/sql.ex:1013: anonymous fn/3 in Ecto.Adapters.SQL.checkout_or_transaction/4
(db_connection 2.4.0) lib/db_connection.ex:1512: DBConnection.run_transaction/4
(ecto_sql 3.7.0) lib/ecto/adapters/postgres.ex:215: Ecto.Adapters.Postgres.lock_for_migrations/3
(ecto_sql 3.7.0) lib/ecto/migrator.ex:493: Ecto.Migrator.lock_for_migrations/4
(ecto_sql 3.7.0) lib/ecto/migrator.ex:388: Ecto.Migrator.run/4
(ecto_sql 3.7.0) lib/ecto/migrator.ex:146: Ecto.Migrator.with_repo/3
(ecto_sql 3.7.0) lib/mix/tasks/ecto.migrate.ex:133: anonymous fn/5 in Mix.Tasks.Ecto.Migrate.run/2

Observations
mix ecto.migrate --prefix private only works if there already is a schema_migrations table in the public prefix.
The error suggests that Ecto is trying to lock the schema_migrations table in the public prefix. Since there is none there, Ecto is raises an error.
In fact, If a schema_migrations table exists in the public schema, the migration command with prefix succeeds.
Note that the structure or content of this table is irrelevant, it can just be a dummy table created with create table schema_migrations ();
This suggests that ecto is simply trying to lock the schema_migrations table in the wrong schema (public) before running the migration.

Please give master a try and let me know how it goes :)

Works! Thank you so much. Calling this fix "quick" would be an understatement 😀

@josevalim I think a similar problem arises with the "mix ecto.migrations" command. The command only looks for migrations defined in the public schema, and can't see migrations that were run in a different schema (they are always listed as "down"). There is currently no way to pass a prefix option to that command, as far as I can see.

A pull request for prefixes on Ecto.migrations is welcome!

Gotcha. I won't have time to work on it for a few weeks. However, as soon as I find the time, I'll gladly get to it. I just wanted to check that the problem is real :)


Body battery support

Adds support for Garmin Body Battery data.

@shaulbd, thanks for your contribution.
Could you fix indent style to spaces in all your changes so we can merge it?

done


Test added for 0 value problem this was solved in #226

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/metasql/commit/ad5a10b864ce7778e36f00ce35d68c196adff160


feat: add ability to enable toc on certain posts ignore global toc settings

What problem does this PR solve?
Add ability to enable toc on certain posts ignore global ToC settings.
Is this PR adding a new feature?
Added higher priority for the post ToC parameter instead of the global ToC parameter.
Is this PR related to any issue or discussion?
No
PR Checklist

[x] I have verified that the code works as described/as intended.
[ ] This change adds a social icon which has a permissive license to use it.
[x] This change does not include any external library/resources.
[x] This change does not include any unrelated scripts (e.g. bash and python scripts).
[ ] I have enabled maintainer edits for this PR.

Hi @hugo-sid! Can your review this PR?


[spi_device] Remove Generic (a.k.a. FwMode) from SPI_DEVICE HWIP

This is a topic brought up multiple times. As TPM can behave almost identical but limited frame size (64B in TPM vs 1kB or more), it is OK to remove Generic mode. TPM protocol, however, has better flow control compared to Generic mode.
The benefit of removing Generic mode is to clean up the complex CDC path on SRAM B port in SPI_DEVICE. It raises many CDC warnings and errors, so that we waived many of them. By removing the Generic mode, SRAM b datapath sits in SPI_CLK always, which simplifies the clock domain and makes easier to verify CDC.
CC: @tjaychen @weicaiyang @jeoongp

The benefit of removing Generic mode is to clean up the complex CDC path on SRAM B port in SPI_DEVICE. It raises many CDC warnings and errors, so that we waived many of them. By removing the Generic mode, SRAM b datapath sits in SPI_CLK always, which simplifies the clock domain and makes easier to verify CDC.

I'll point out that adjusting the generic mode logic so it doesn't use asynchronous FIFOs in front of the SRAM would also take care of that issue. Removing generic mode altogether may be the best choice, but it's not the only way to get rid of the complex CDC.

Yep. you are right. We can fix the issue as DPRPAM now supports byte enable (it wasn't when GenericMode was designed. That's why DPSRAM was in SYS_CLK and async fifo was there). But as I mentioned, it is not worth to have two almost identical modules in SPI_DEVICE.

if we remove generic mode, does that mean spi device would have no way of constructing a full duplex protocol if someone wanted to use it that way?

i've honestly only ever seen 2 use cases of someone doing that.
@a-will may have seen others.

Triaged for spi_device. Labeling https://github.com/lowRISC/opentitan/labels/Type%3AFutureRelease because removing Generic / FW mode seems out of scope for this release. Verification of the mentioned CDC path is tracked in #15457.

Indeed, removing / changing generic mode is out of scope for M2.5.
CDC will be closed separately as noted above.


Property auto-detect used at configuration time when using toolchain with auto-download=false

Expected Behavior
Build fails because it cannot find a suitable toolchain.
Current Behavior
Build fails with:
FAILURE: Build failed with an exception.

* What went wrong:
Could not determine the dependencies of task ':compileJava'.
> Could not resolve all dependencies for configuration ':compileClasspath'.
   > Failed to calculate the value of task ':compileJava' property 'javaCompiler'.
      > Cannot obtain value from provider of Gradle property 'org.gradle.java.installations.auto-detect' at configuration time.
        Use a provider returned by 'forUseAtConfigurationTime()' instead.

Context
Gradle 6.7-rc-1
Using
java { toolchain { languageVersion.set(JavaLanguageVersion.of(15)) } }

with org.gradle.java.installations.auto-download=false
Steps to Reproduce
Should be reproducible with any Java (or Kotlin/Groovy/Scala) project.

Configure a toolchain you do not have installed.
Execute any JavaCompile task (even one that would be skipped because it has no source) with org.gradle.java.installations.auto-download=false

Your Environment
Build scan URL: https://scans.gradle.com/s/hi4zunolp7x6i
This scan is from gradle-errorprone-plugin, with some local modifications (bumping to Gradle 6.7-rc-1 to begin with). I originally noticed the issue within a TestKit test while adding toolchains support to the plugin.

Thanks for the report. I had fixed a bunch of these before the RC1. Looks like I missed one!

Indeed, I enhanced an error message to show the state of these properties ... and forgot to mark them for use at configuration time 🤦


Bug: createActorContext is not starting the machine

XState version
XState version 5
Description
Hi, it is my first time using XState and everything looks really nice. I started using useMachine etc but then I decided to switch to createActorContext and this is when the issues started.
I checked all examples and I can not see anything weird from my setup but basically when I wrapped my entire app with
const TheContext = createActorContext(machine);

...

<TheContext.Provider>
  {children}
</TheContext.Provider>

and when I try to call
const theActorRef = TheContext.useActorRef();
const { send, getSnapshot } = theActorRef;

console.log(getSnapshot());

I'm seeing the following issue:

I don't understand whats going on at all.
Expected result
I should get the current snapshot fro the state instead of an error
Actual result
The app is crashing because nothing got initialized.

Reproduction
none
Additional context
No response

Hi, @davidkpiano, I will close this issue because I found what the problem was... when I was testing the library, I forgot to remove a createActor that was pointing to the same machine, and maybe there is a singleton happening in the background or something like that, but it was making some mess...
Thank you so much for your quick reply and awesome library!


[BUG] fails with modelator trace --model-checker tlc spec.tla spec.cfg

modelator 0.4.1
Description
modelator trace --model-checker tlc spec.tla spec.cfg
thread 'main' panicked at '[modelator] full TLA state should have been parsed', modelator/src/model/language/tla/json/mod.rs:10:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I need time to create a minimal example.
System information

OS: OSX Big Sur M1

Additional context
Mirrors https://github.com/informalsystems/mbt/issues/45

The parser throws error because it does not support ranged sets i.e. 1..10. Let me create a PR to add support.


Getting Value Conversion error when pass "kubernetes" schema from the caller side

Hi. I would like to define kubernetes schema on the caller (live) side and pass to the module. However, I got Value Conversion error as shown below:
Error: Value Conversion Error
│
│   with module.flux.provider["registry.terraform.io/fluxcd/flux"],
│   on ../../../modules/flux/main.tf line 19, in provider "flux":
│   19:   kubernetes = var.flux_kubernetes
│
│ An unexpected error was encountered trying to build a value. This is always an error in the provider. Please report the following to the provider developer:
│
│ Received unknown value, however the target type cannot handle unknown values. Use the corresponding `types` package type or a custom type that handles unknown values.
│
│ Path: kubernetes
│ Target Type: *provider.Kubernetes
│ Suggested Type: basetypes.ObjectValue

On the caller side, I declare:
flux_kubernetes = {
  config_path = "~/.kube/config"
  config_context = "docker-desktop"
}

On the module, I do:
provider "flux" {
  kubernetes = var.flux_kubernetes
  ...
}

This is variables.tf
variable "flux_kubernetes" {
  type = any
}

Not sure does this suppose to work or I might miss something?

Which version of the provider are you using?

Hello @smeeklai ,
I hope you're doing well! I'm the newest contributor to this repository, and I'm currently in the process of issue grooming to ensure that all concerns are addressed promptly and efficiently.
I noticed this issue you reported and wanted to check in with you to see if it's still affecting your work. Your feedback is invaluable to us, and any additional insights or updates you can share would be greatly appreciated to help us understand and solve the problem more effectively.
If this issue has been resolved, could you please share how it was fixed? This information could be incredibly helpful to others in the community facing similar problems. It would also allow us to close this issue with a clear resolution.
In case the issue is still open and troubling you, let's work together to find a solution. Your satisfaction and the smooth functioning of our project are our top priorities.
Thank you for your time and contributions to our community. Looking forward to your response!
Best regards,
Steve


fix(fatfs): _FFCONF_DEFINED added to ffconf.h

I don’t know if this affects compilation, but preprocessed .i files parsing is broken because of this

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Denis Feklushkin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

cla-assistant isn't works for me

Hi, thank you for contributing!
Could you elaborate a bit about what the issue was?

Could you elaborate a bit further about what the issue was?

Sure!
(This is only experiment)
To create D language language bindings I use preprocessed .i files which can be obtained by adding such code to the project root CMakeLists.txt:
     # Preserve *.i files
     set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -save-temps")

After this, files are obtained, in some of which (in diskio_rawflash.c.i, for example) I found (because my .i parser failed on them) that the definitions like
void* ff_memalloc (unsigned msize);

are scattered twice. These duplicated definitions was sourced only from ffconf.h.

LGTM!