"mlir-opt: not found" while executing scripts

Hi. I'm trying to run your project on my machine using the Docker image you've provided. I've managed to get the basic setup down, and have gotten around the directory structure of the project.
However, when I'm trying to run the script.sh from the baseline in demo, it shows me the following errors:
script.sh: 2: mlir-opt: not found
script.sh: 5: mlir-opt: not found
script.sh: 8: soda-opt: not found
script.sh: 11: soda-opt: not found
script.sh: 14: soda-opt: not found
script.sh: 17: soda-opt: not found
script.sh: 20: soda-opt: not found
script.sh: 21: soda-opt: not found
script.sh: 24: mlir-translate: not found
script.sh: 25: mlir-translate: not found
script.sh: 28: llc: not found
script.sh: 29: llc: not found

Similar errors occur when I try to run the script4x4.sh file from the MiniLM's baseline folder.
script4x4.sh: 2: mlir-opt: not found
script4x4.sh: 5: mlir-opt: not found
script4x4.sh: 8: soda-opt: not found
script4x4.sh: 11: soda-opt: not found
script4x4.sh: 14: soda-opt: not found
script4x4.sh: 17: soda-opt: not found
script4x4.sh: 20: soda-opt: not found
script4x4.sh: 21: soda-opt: not found
script4x4.sh: 27: mlir-translate: not found
script4x4.sh: 28: mlir-translate: not found
script4x4.sh: 31: llc: not found
script4x4.sh: 32: llc: not found
clang: error: no such file or directory: '11-model-4.o'
clang: error: no such file or directory: '12-accel-4.o'

I understand that mlir-opt and soda-opt are supposed to be in the PATH variable (you've mentioned so in the README.md), which I've tried doing with the following commands:
export PATH=$PATH:/llvm-project/mlir/tools/mlir-opt
export PATH=$PATH:/llvm-project/mlir/test/mlir-opt
export PATH=$PATH:/llvm-project/mlir/lib/Tools/mlir-opt
export PATH=$PATH:/llvm-project/build/bin/mlir-opt
export PATH=$PATH:/llvm-project/mlir/include/mlir/Tools/mlir-opt
export PATH=$PATH:/llvm-project/build/tools/mlir/tools/mlir-opt
export PATH=$PATH:/llvm-project/build/tools/mlir/test/mlir-opt
export PATH=$PATH:/llvm-project/build/tools/mlir/lib/Tools/mlir-opt

export PATH=$PATH:/mlir-cgra/tools/soda-opt
export PATH=$PATH:/mlir-cgra/test/soda-opt
export PATH=$PATH:/mlir-cgra/build/bin/soda-opt
export PATH=$PATH:/mlir-cgra/build/tools/soda-opt
export PATH=$PATH:/mlir-cgra/build/test/soda-opt

Using the echo $PATH, I get this as output:
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/llvm-project/build/bin/mlir-opt:/mlir-cgra/build/bin/soda-opt:/llvm-project/mlir/tools/mlir-opt:/llvm-project/mlir/tools/mlir-opt:/llvm-project/mlir/test/mlir-opt:/llvm-project/mlir/lib/Tools/mlir-opt:/llvm-project/mlir/include/mlir/Tools/mlir-opt:/llvm-project/build/tools/mlir/tools/mlir-opt:/llvm-project/build/tools/mlir/test/mlir-opt:/llvm-project/build/tools/mlir/lib/Tools/mlir-opt:/mlir-cgra/tools/soda-opt:/mlir-cgra/test/soda-opt:/mlir-cgra/build/tools/soda-opt:/mlir-cgra/build/test/soda-opt

And yet, the project refuses to run. I've also tried installing llvm-12-tools within Docker as an added attempt, but it's already there. What am I doing wrong here?

Hi, please do:
source /setup.sh

The script would setup the environment for you, in my case, echo $PATH shows:
/llvm-project/build/bin:/mlir-cgra/build/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin

You mean MiniLM baseline script stop execution due to that warning? Can you cleanup your $PATH and retry?

I'm not sure what you mean by cleaning up the $PATH variable. It looks like this currently (after executing setup.sh once):
# Separated by line break for reading convenience.
/llvm-project/build/bin
:/mlir-cgra/build/bin
:/usr/local/sbin
:/usr/local/bin
:/usr/sbin
:/usr/bin
:/sbin
:/bin

I was trying to run the files in the demo folder when I encountered the errors. Both scripts in baseline and cgra were showing the same warning before halting execution.
As for running the model in MiniLM folder, I get a similar warning message about 04-finalized.mlir. This happens after I run the MiniLM.py file in the model folder, move the 02-linalg.mlir file to the cgra folder, and try to run the script4x4.sh script in the cgra folder. To be specific, this is the error I get:
04-finalized.mlir:129:3: warning:
  func.func @forward(%arg0: memref<1x12xi64>, %arg1: memref<1x2xf32>) {
  ^
04-finalized.mlir:129:3: note: see current operation: func.func @forward(%arg0: memref<1x12xi64>, %arg1: memref<1x2xf32>)
{...}}

, followed by a very long block of (I'm guessing) object code within the curly braces.

Mine did pass 04 stage. I guess sth wrong with your re-install stuff or the path. Plz make sure ur path is same as mine:
/llvm-project/build/bin:/mlir-cgra/build/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin

Try again or re-download the docker.

That's odd. My PATH's the same as yours, as I've posted above. I've also tried reinstalling Docker and pulling the image again, but it gave me the same result.

That's odd. My PATH's the same as yours, as I've posted above. I've also tried reinstalling Docker and pulling the image again, but it gave me the same result.

Tried a couple random things after this, but still the same issue. Any suggestions?

Hi @Yiran-ASU, please help reproduce and triage this issue, thanks!

Hello, Dr. Tan @tancheng , I also got the 04-finalized.mlir:3:3: warning by running sh script.sh in baseline.
Then I run ./simulate, and got check result and total cycles:
Check result:  1	2	3	4	5	6	7	8	9	10	11	12	13	14	15	16	 33	34	35	36	37	38	39	40	41	42	43	44	45	46	47	48	 65	66	67	68	69	70	71	72	73	74	75	76	77	78	79	80	 97	98	99	100	101	102	103	104	105	106	107	108	109	110	111	112	 129	130	131	132	133	134	135	136	137	138	139	140	141	142	143	144	 161	162	163	164	165	166	167	168	169	170	171	172	173	174	175	176	 193	194	195	196	197	198	199	200	201	202	203	204	205	206	207	208	 225	226	227	228	229	230	231	232	233	234	235	236	237	238	239	240	 257	258	259	260	261	262	263	264	265	266	267	268	269	270	271	272	 289	290	291	292	293	294	295	296	297	298	299	300	301	302	303	304	 321	322	323	324	325	326	327	328	329	330	331	332	333	334	335	336	 353	354	355	356	357	358	359	360	361	362	363	364	365	366	367	368	 385	386	387	388	389	390	391	392	393	394	395	396	397	398	399	400	 417	418	419	420	421	422	423	424	425	426	427	428	429	430	431	432	 449	450	451	452	453	454	455	456	457	458	459	460	461	462	463	464	 481	482	483	484	485	486	487	488	489	490	491	492	493	494	495	496	 Total cycles: 3296
Also, my $PATH is the same: /llvm-project/build/bin:/mlir-cgra/build/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
Then, I found in /mlir-cgra/experiments/MiniLM/model directory, there is only bert.py, not MiniLM.py. After I run python 3.9 bert.py, I got bert.milr. So I guess the script4x4.sh in cgra folder need to be modified.

So you are able to generate the simulate binary and run it, though there is warnings, right?

yes, I think so

Hello. I haven't been able to solve this issue, and I had kinda given up on it for the time being.
However, I had noticed something which I'd like to bring your attention towards. When I run any of the scripts in baseline or cgra, I don't get to see an end to the execution of that script. Since all I get is a warning, the script should finish its execution and return to the terminal for letting the user input other commands. In my case however, I never get to that finishing point. I have to manually stop the execution with Ctrl+C, which basically interrupts the script. That's why nothing happens for me whenever I execute the ./simulate command, I think.
Are there high system resource requirements for executing this project? I'm currently on a laptop which has 16 GB of RAM and 4 GB of Nvidia RTX 3050 GPU. Is this configuration enough for running this project?

Hi @Vedgon, I think 16GB is enough, just make sure the docker get enough memory via --memory.


Adds acceptance test for rollback feature.

Fix for #1736
Added two scenarios for testing rollback functionality.

@mattbrictson I have made the suggested changes.

Great, thanks! Can you also fix the RuboCop issues that are causing CI to fail? Once CI passes I can merge this PR.

Hi, I have fixed the RuboCop issues.

Thanks very much! 🙌


finite loops don't work

When modifying N in the infinite square wave example, only N=float('inf') works.  PB doesn't output anything when N is finite.

Ooops-- this was working fine.  I just didn't have the scope setup well.


tentative GINE

This is a tentative to implement GINE (adding edges)
Current code needs to define the edges size maybe this can be avoid.
Proposed changes
tentative GINE example to address #141.
Checklist
Put an x in the boxes that apply.

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have updated the necessary documentation

@thegodone it would be great to have this implemented within mlx-graphs rather than in a notebook.

Of course, yes, but I am not sure of the result yet. can you first check my
code ?
Le dim. 10 mars 2024 à 10:36, Francesco Farina @.***> a
écrit :

@thegodone https://github.com/thegodone it would be great to have this
implemented within mlx-graphs rather than in a notebook.
—
Reply to this email directly, view it on GitHub
https://github.com/mlx-graphs/mlx-graphs/pull/145#issuecomment-1987160524,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJBWYSHPMKCEAQPEKOR2OTYXQSQXAVCNFSM6AAAAABEN3BT72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBXGE3DANJSGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

For a next PR, it would be easier if you can write your code in real .py files instead of notebooks, as it's complicated to leave comments on a notebook.
Here are

Can you add bias=True in the Linear layer you instantiate? We generally follow PyG's implementation and they use bias.
You use edge_projection instead of edge_lin_proj...
and not edge_features.shape[1] == node_features.shape[1] -> you should apply the edge projection even if the node and edge shapes match.
hasattr(self, 'edge_projection') -> you can use 'edge_projection' in self instead
your tests do not test GINE but only GINConv

I update the python file gin_conv.py

Thanks Tristan, for the next time I will follow PyG implementations. I would like to add the DMPNNconv what is the best way cause basically this is just MPNN but with a directed graph.

I plan to merge this PR @thegodone, however you pushed a notebook related to the Esol dataset, can you remove it and cherry pick your work to a new branch for another PR?

I plan to merge this PR @thegodone, however you pushed a notebook related to the Esol dataset, can you remove it and cherry pick your work to a new branch for another PR?

@TristanBilot I'll merge after your last review


Develop

Thanks for recognising that you were out of date and doing this. However, this is not the typical way of merging changes from the apache marmotta repository into this repository, and it causes a few minor issues because each pull request creates a new merge commit that is not necessary in this case. The typical method is to do the following commands.
You need to do the following command once only for each computer (run it inside of the marmotta directory that you have checked out already):
git remote add upstream https://github.com/apache/marmotta.git

Then, when there are changes, you can merge them in with the following commands:
git fetch upstream
git checkout develop
git merge --ff-only upstream/develop

However, because you merged a pull request here, the last command will not work though, so you will need to run it without "--ff-only" the first time, but after that you should always do it with "--ff-only".
To merge develop into your branch you would do the following two commands to pull in all of the changes into the RDF/HDT branch (after doing the git fetch above):
git checkout MARMOTTA-593
git merge --no-ff upstream/develop

Thank you! I'll have a try following your instructions!


fix Optimize memory gc frequency

Proposed changes
Issue Number: close #xxx
Problem summary

Ensure that the memory status is refreshed in time
Avoid frequent GC

Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.28 seconds
stream load tsv:          476 seconds loaded<PHONE_NUMBER>9 Bytes, about 149 MB/s
stream load json:         40 seconds loaded<PHONE_NUMBER> Bytes, about 56 MB/s
stream load orc:          69 seconds loaded<PHONE_NUMBER> Bytes, about 15 MB/s
stream load parquet:          27 seconds loaded 861443392 Bytes, about 30 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230228132640_clickbench_pr_105745.html


Fix erring parsing filter from JXplorer & Apache Directory Studio

Fixes an issue decompiling filters sent across from both JXplorer and Apache Directory Studio.
Unit test included. Example failure:
--- FAIL: TestDecompileFilter (0.00s)
filter_test.go:65: Problem decompiling hwtvYmplY3RjbGFzcw== - LDAP Result Code 202 "": Error decompiling filter
filter_test.go:68: hwtvYmplY3RjbGFzcw== Expected (objectclass=*) got (
FAIL

@nwoolls you may be interested in using and contributing to the go-ldap/ldap fork of this repo


Add unit tests for Machine.Create

Add unit tests for the Machine.Create function, to increase test coverage.
While working on it, changed the function a bit to use the request context, instead of context.ToDo
Signed-off-by: Nahshon Unna-Tsameret<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release notes:

None

/ok-to-test


Possible issues with the function that defines grammar for capturing Tool Call.

The bug
The function that defines the Grammar for Triggering the tool Usage seems to behave in an un-expected manner.
Consider the following Code  , The definition for expression() is same as in Readme .
from guidance import capture, Tool
@guidance(stateless=True)
def calculator_call(lm):
    # capture just 'names' the expression, to be saved in the LM state
    return lm + expression()
@guidance
def calculator(lm):
    lm += f'$$$$'
    return lm
calculator_tool = Tool(calculator_call(), calculator)
lm5 =  models.Mock(b"<s> 5+4*5  Some English Text <s>")
lm5 += gen(max_tokens=30, tools=[calculator_tool], stop='\n\n')

Output :   5+4*5  +9$$$$ewxbewsawrglbrsgvqmntnyrfylaxttvaluu,giicndoeqgspkjloukxo(1$$$$
Rather than Calling the calculator after encountering the Token Sequence "5+4*5" , it seems that the function calculator_call changes the model state in an odd fashion causing it to generate the token +9 following which the call to calculator is finally made and after which the model goes on to produce the Gibberish text . However Intersetingly , the call to calculator is made correctly later on , when the token corresponding to "1" is generated.
Also, when I change the definiton of the calculator_call to
def calculator_call(lm):
    # capture just 'names' the expression, to be saved in the LM state
    return lm + capture(expression(),name = 'A')

Output :   5$$$$ln ymoadxdggcjbewxbewsawrglbrsgvqmntnyrfylaxttvaluu,gi6$$$$
So , In this case the call to calculator is made as soon as the number 5 is encountered , i.e it greedily finds the shortest pattern matching the grammar corresponding to expression() and calls the calculator ( I assume this is how it has been implemented ).
To Reproduce
Give a full working code snippet that can be pasted into a notebook cell or python file. Make sure to include the LLM load step so we know which model you are using.
import guidance
from guidance import one_or_more, select, zero_or_more
# stateless=True indicates this function does not depend on LLM generations
@guidance(stateless=True)
def number(lm):
    n = one_or_more(select(['0', '1', '2', '3', '4', '5', '6', '7', '8', '9']))
    # Allow for negative or positive numbers
    return lm + select(['-' + n, n])

@guidance(stateless=True)
def operator(lm):
    return lm + select(['+' , '*', '**', '/', '-'])

@guidance(stateless=True)
def expression(lm):
    # Either
    # 1. A number (terminal)
    # 2. two expressions with an operator and optional whitespace
    # 3. An expression with parentheses around it
    return lm + select([
        number(),
        expression() + zero_or_more(' ') +  operator() + zero_or_more(' ') +  expression(),
        '(' + expression() + ')'
    ])

from guidance import capture, Tool
@guidance(stateless=True)
def calculator_call(lm):
    # capture just 'names' the expression, to be saved in the LM state
    return lm + (expression())
@guidance
def calculator(lm):
    lm += f'$$$$'
    return lm
calculator_tool = Tool(calculator_call(), calculator)
lm5 =  models.Mock(b"<s> 5+4*5  Some English Text <s>")
lm5 += gen(max_tokens=30, tools=[calculator_tool], stop='\n\n')

Intended Use
I want to call the calculator as soon as an Expression is encountered ( but not in a greedy fashion , i.e the longest expression should be allowed to generate before making the call ) .
Also , I don't want to have to guide the LLM explicitly to generate the syntax " calculator( expression()  ) " as is the case in the Readme example provided.
Any help is appreciated.
System info (please complete the following information):

OS (e.g. Ubuntu, Windows 11, Mac OS, etc.) :  Ubuntu
Guidance Version (guidance.__version__): 0.1.10

Hi @anshmn ! Good questions. The reason the calculator is not called when you get to 5+4*5 is because the tool call is not greedy matching, it allows the model to make a longer tool call if it wants to (or emit the stop tokens, which you have set to \n\n). Since a space is a valid tool call continuation of 5+4*5 you get 5+4*5  generated by the model. At this point we are committed to making a tool call because 5+4*5 already happened, but we don't have a complete match yet, so the model is force to finish the grammar, which in this case results in +9) once the grammar is complete the stop token is chosen and the call is made.
I am going to leave this issue open however, because the change in greedy vs. non-greedy when you add a capture does seem off. Thanks for flagging that!
Some notes:
As part of debugging this I added in another PR (to be merged soon) a match method on the GrammarFunction so you can test matches more easily:

In your original output Output : 5+4*5 +9$$$$ewxbewsawrglbrsgvqmntnyrfylaxttvaluu�,giicndoeqgspkjloukxo(1$$$$ two calls are made (each $$$$ is a call result). The random text is just what the Mock model generates when it does not have a matching prefix.

Thanks for the detailed reply @slundberg .
Their seems to be some other issues with the way tool call is being made .
Consider the following code snippet
from guidance import capture, Tool
lst=[]
@guidance(stateless=True)
def calculator_call(lm):
    # capture just 'names' the expression, to be saved in the LM state
    return lm + expression()

@guidance
def calculator(lm):
    lst.append(str(lm))
    return lm

calculator_tool = Tool(calculator_call(), calculator)
lm5 =  models.Mock(b"<s>5456 +445 * 5Some English Text<s>")
lm5.echo = False
lm5 += gen(max_tokens=30, tools=[calculator_tool], stop='<s>')
print(lst)

['5', '54', '545', '5456 +4', '5456 +44', '5456 +445 * 5']
As is evident from the code output above  , the call to calculator is made as soon as a valid expression is encountered provided that the output generation after the point where the call is made still remains a valid expression !!
This explains the call being made at '5', '54', '545' but not '5456' because "+445 * 5" won't remain a valid expression .


Improve error handling in market microservice

Description
Usually the following errors are visible when the service is run for a while.
market_1                  | TypeError: tickers.filter is not a function
market_1                  |     at filterTickers (/home/lisk/lisk-service/market/shared/market/sources/bittrex.js:51:34)
market_1                  |     at reload (/home/lisk/lisk-service/market/shared/market/sources/bittrex.js:90:27)
market_1                  |     at async Task.controller [as execution] (/home/lisk/lisk-service/market/jobs/bittrex.js:30:4)
market_1                  | 2021-10-11T13:59:05.110 ERROR [bittrex] Cannot read property 'data' of undefined
market_1                  | 2021-10-11T13:59:05.110 ERROR [bittrex] TypeError: Cannot read property 'data' of undefined
market_1                  |     at fetchAllMarketTickers (/home/lisk/lisk-service/market/shared/market/sources/bittrex.js:41:19)
market_1                  |     at async reload (/home/lisk/lisk-service/market/shared/market/sources/bittrex.js:89:19)
market_1                  |     at async Task.controller [as execution] (/home/lisk/lisk-service/market/jobs/bittrex.js:30:4)

Improve data validation and prevent from throwing the call stack in on data retrieval errors.
Acceptance Criteria

The console logs keep the warning message with an explanation, such as validation error or data retrieval error.

This issue has already been addressed in PR https://github.com/LiskHQ/lisk-service/pull/959.


Resolved issues #696, #721, #722

Resolved issues #696, #721, #722

[ ] Remove unnecessary module https://github.com/WeaxMe/Orienteer/blob/develop/orienteer-core/src/main/java/org/orienteer/core/module/RoutingModule.java

[ ] Refactor mappers in https://github.com/WeaxMe/Orienteer/tree/develop/orienteer-pages/src/main/java/org/orienteer/pages

[ ] Add possibility for remove mapper https://github.com/WeaxMe/Orienteer/blob/develop/orienteer-pages/src/main/java/org/orienteer/pages/wicket/mapper/ODocumentAliasCompoundMapper.java

orienteer-users issues

[ ] Add possibility for register user throughout  social networks.
Right now if user is not registered in system and he clicks on login throughout social networks user account creates by default which confuses users.

[ ] Use OrientDB tokens for login user

[ ] Add flag which gives possibility for register user in login page by social networks

[ ] Fix font in login page

[ ] Add support for VK, GitLab, Twitter, LinkedIn

Changed font in login and register pages


Can't Update a Contact's Email

I'm trying to update the email field of a contact, but it isn't in the UpdateContactOptions (link to code) interface.
The docs include an email in the body for this request but providing the email field when updating contacts shows a type error:
await resend.contacts.update({
        id: contactId,
        email: payload.email || undefined,
        firstName: payload.name || undefined,
      })

It looks like this isn't just a type error, it also isn't sent in the body of the patch request.

Can you upgrade to the latest?


replace forever bouncing i-icon with 3 times pulsing

..and pulse on hover
also add shake.css from animate.css

fixes #147

@gahjelle OK?

Looks quite good. We should pay attention to whether the effect is strong enough so users find it. Since it is temporary, we could have a slightly stronger effect as well!
 :)


Add routing allocation exclusion in db.pl

Clearly describe the problem and solution
We're using an ES cluster where we dedicate/isolate nodes for storing sessions. Our rational doing that is that metadata indices (especially files) should not be lost even if nodes storing the sessions are overloaded: loosing 1 file means loosing all the sessions stored in the file).
ES provides a mechanism called index routing allocation, and in our case, we tag metadata nodes as such and exclude sessions indices from these metadata nodes.
ES template contains something like: index.routing.allocation.exclude.arkimeNodeType = metadata, arkimeNodeType = metadata being the tag of the metadata nodes.
This PR provides a way to initialise the sessions template with a specific index.routing.allocation.exclude value using the parameter --sessionsroutingallocationexcludekey key1=value1
Relevant issue number(s) if applicable
n/a
License
I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Thanks, but I'm wondering if you could do without a change to db.pl. Just make 2 new index templates (with different names than we use :)) One that matches arkime_sessions* and one the meta ones, and set the exclude yourself in that new template. I guess the issue is this won't change existing ones, but not sure the current db.pl handles that either.
If that won't work, please add help to showHelp, and I think the leading "index." isn't needed since

Thanks for the comment. I'll check the 2nd template and come back

Let me know if you can't get the multiple templates working


vm-repair extension: Region bug fix and typo fixes

Minor changes for 0.1.1

Region bug fix where 'repair create' failed when the resource group region and disk region was different.

Typo fixes

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/swbae31/azure-cli-extensions.git@region_bug_fix#subdirectory=src/$EXT&egg=$EXT"

Closed due to wrong branching. Long commit history due to this. New PR made with new branch https://github.com/Azure/azure-cli-extensions/pull/812/commits


[Notification] SP can`t see notifications when Parent applies to a workshop for himself.

Environment: Windows 10, 64-bit, Google Chrome 107.0.5304.107.
Reproducible: always.
Preconditions
1.Go to https://oos.dmytrominochkin.cloud/#
2.Log in as a Parent.
3.Parent hasn`t applied to the workshop yet.
4.The Parent submitted an application to join the selected workshop (for example, "Ластівка")
Steps to reproduce
1.Log in as a SP.
2.Pay attention to the absence of a notification on the main page in the site header.
3.Go to the "Заявки" tab of SP cabinet (by clicking the button in the top right corner).
Actual result
SP can`t see notifications when Parent applies to a workshop for himself on the main page in the site header.

Expected result
When Parent applies to a workshop for himself on the main page in the top right corner SP sees a bell with an orange circle next to it and the number of unread notifications in this circle.
Labels to be added
"Bug", Priority ("medium"), Severity ("low"), Type ("UI, "Functional").

Fixed

Fixed


How to log operations on Queryset (update, bulk_create, bulk_update)?

I have tried to do the operations on a Queryset object and these changes are not been logged in AuditLog tables
data = MyModel.objects.filter(field1='value')
data.update(filed2='new value')

In addition to the above operations I was not able to see the entries in auditlog table in bulk_create and bulk_update operations as well
Can anyone suggest how I can track change logs in above operations?

The following issues are related. Please take a look
https://github.com/jazzband/django-auditlog/issues/515
https://github.com/jazzband/django-auditlog/issues/203

Thanks for the reply


Small updates to Notification component

changes extraAlert to have max width and right alignment
fixes dismiss button and extraAlert placement
other minor styling tweaks for icons in notification

If you want to update the version and then build the docs I'll merge.


Error "Invalid ruleset provided"

Hi,
I've installed and configured the Spectral plugin in IntelliJ on macOS as follows:
Ruleset: https://github.com/SchwarzIT/api-linter-rules/blob/main/spectral.yml
Included files: **.yml
The linting fails with the following errors. I've tried an alternate ruleset file as well, unfortunately with the same results:
13:52	Linting failed
			Execution of Spectral failed using the following command: ""/Users/myuser/Library/Application Support/JetBrains/IdeaIC2021.3/plugins/spectral-intellij-plugin/spectralExecutables/spectral-macos" lint --format json --ruleset https://github.com/SchwarzIT/api-linter-rules/blob/main/spectral.yml /var/folders/jy/wp8k1n1d0nx2kqtt6tlfrs5c0000gn/T/5251202261066929309.tmp"
			Error Message:
			Invalid ruleset provided

Can you please look into this? Many thanks in advance!

Hi @kkeck-nordlb, it looks like you're using an invalid url for the ruleset.
If you try to open the link to the ruleset you're using, you'll see that it sends you to the github page of that file.
Since that's a website so basically HTML, spectral is not able to understand the ruleset. (It's even more noticable when you curl the url)
You need to use the url of the "raw" file hosted on github. For example: https://raw.githubusercontent.com/SchwarzIT/api-linter-rules/main/spectral-api.yml
If you navigate to this link you'll see, that everything github related is gone and you're just served the raw yaml file.
We have all the urls of our spectral rulesets in our readme, you can just copy them from there ;)
Please try again with this url and let me know if it solves the issue: https://raw.githubusercontent.com/SchwarzIT/api-linter-rules/main/spectral-api.yml

Hi @markbrockhoff. Yes, of course that was the problem - many thanks for the hint!


tests: init

Hi! This is me again, with the first of 4 PRs this time xD
This PR adds simple NixOS integration tests, which can be run with nix flake check (or individually with nix build .#checks.x86_64-linux.simple).
For now, there's two:

Simple: runs the vanilla server and check that it actually started
Files & Symlinks: runs the paper server, with two nix expression-generated files (ops.json and paper.yml), as well as symlink (a vanilla jar that is nescessary to avoid paper trying to download it at runtime).

In a follow-up PR (will open it in a few minutes), I will use this to test command execution:

I de-op Misterio7x, this mutates the ops.json, so validates that files are mutable.
I run a unknown command and grep the output, validating that the paper.yml unknown-command is actually used.

Looks pretty good to me. Will do local testing to make sure everything is good before merging.

Darn you Paper! Why must you download things at runtime!
It might be a good idea to put the corresponding vanilla jar for a paper server into the paper derivation's passthru.
Also would be a good idea to document the need for that in the readme.

For normal usage, we usually can assume the user will have internet connection, but I fully agree, we should totally add that to passthrough and document it; I'll do this in a minute.

There you go: https://github.com/Infinidoge/nix-minecraft/pull/44

Sorry for taking so long to merge this. While I do have the excuse that I've been busy with college, it really shouldn't have taken so long to merge a PR of this size. Thanks again for the contributions!


Add support for 'localhost' urls

Issue
The cookie jar can capture cookies set from a localhost server, but responses do not send the cookies to the same url.
Versions

node: 12.18.1
node-fetch-cookies: 1.4.0

Test Code:
require = require('esm')(module);
const {fetch, CookieJar} = require('node-fetch-cookies');
const {expect} = require('chai');
const {describe, it} = require('mocha');

/* The express server I was testing, using cookie-parser and express-session */
const {getCleanServer} = require('./util/server.js');

describe('When cookies are set by a response', function () {
  function cookieCount(cookeJar) {
    let count = 0;
    for (const cookie of cookieJar.cookiesAll()) {
      count += 1;
    }
    return count;
  }

  async function assertCookiesSetAndSent(url) {
    cookieJar = new CookieJar();

    await fetch(cookieJar, url);

    expect(cookieCount(cookieJar)).to.be.above(0);

    let options = {};
    await fetch(cookieJar, url, options);

    expect(options.headers, 'Header is undefined')
      .to.not.be.undefined;
    expect(options.headers.cookie, 'Header is defined, but cookie not defined')
      .to.not.be.undefined;
  }

  it('sends the cookies to normal urls', function () {
    return assertCookiesSetAndSent('http://www.pinterest.com/');
  });

  describe('When I setup a localhost server', function () {
    let server;
    beforeEach('setup the server', async function () {
      server = await getCleanServer();
    });

    afterEach('stop the server', function () {
      return server.close();
    });

    it('sends the cookies to localhost urls', function () {
      return assertCookiesSetAndSent('http://localhost:8080/signup');
    });
  });
});

Test Output
  When cookies are set by a response
    ✓ sends the cookies to normal urls (394ms)
    When I setup a localhost server
      1) sends the cookies to localhost urls

  1 passing (752ms)
  1 failing

  1) When cookies are set by a response
       When I setup a localhost server
         sends the cookies to localhost urls:
     AssertionError: Header is undefined: expected undefined not to be undefined
      at assertCookiesSetAndSent (test/test_cookie.js:29:17)
      at processTicksAndRejections (internal/process/task_queues.js:97:5)
      at async Context.<anonymous> (test/test_cookie.js:49:7)

Possible Solution
I am not an expert in cookies. Testing through some of the code seems to indicate that in cookieJar.mjs on lines 78-82, the expression attempts to split subdomains from the hostname. However, it assumes there is at least one period in the hostname, and on line 82 it drops the last entry. However, when the hostname is simply "localhost", the only entry before line 82 is ['localhost'], so the only domain is dropped. I don't know enough about the project to know if modifying those lines directly would have unintentional consequences.

I think that will also be the case with servers in the local network.

Yes, you're right. Lines 78-82 create ["sub.domain.com", "domain.com", "com"] from sub.domain.com, because cookies are stored in a map which maps domains to cookies. The slice(0, -1) at the end should just remove the TLD entry of the resulting array, because there won't ever be cookies stored for a TLD anyways. But I didn't have domains in mind, that just don't have any TLD, like localhost or similar.
Thanks for investigating this issue and providing test code!
I pushed https://github.com/jkhsjdhjs/node-fetch-cookies/commit/a9d9f6c6757f3c68594342813019525819544409, which should fix this issue. Can you test if it works for you? I would then release 1.4.1 with the fix.

After that commit was added to my project, both the sample test passed as well as the rest of my project's tests that relied on it. Looks resolved to me.

Thanks!


Funds lost from way2collector

One of the onboarded collectors lost his funds during minting because he got an error. Can you please check this. This would be the "most urgent" topic in my eyes...
Wallet address:
5H6ybEFCmNDbosZjx7cddcVKNCzu21SYegRmULf86Z49kSiB

He/She did not lost any funds
https://assethub-polkadot.subscan.io/account/5H6ybEFCmNDbosZjx7cddcVKNCzu21SYegRmULf86Z49kSiB
https://polkadot.subscan.io/account/163GjZWGd9V5FQaFukfdmmKUDpzYiJzgjBAFddeUee5fvx6h?tab=extrinsic
The second TX failed because he tried to mint NFT from H-emitters collection that is already minted out
https://assethub-polkadot.subscan.io/extrinsic/6814916-3
TL;DR
His funds are on AssetHub, no DOT were lost

He/She did not lost any funds
https://assethub-polkadot.subscan.io/account/5H6ybEFCmNDbosZjx7cddcVKNCzu21SYegRmULf86Z49kSiB
https://polkadot.subscan.io/account/163GjZWGd9V5FQaFukfdmmKUDpzYiJzgjBAFddeUee5fvx6h?tab=extrinsic
The second TX failed because he tried to mint NFT from H-emitters collection that is already minted out
https://assethub-polkadot.subscan.io/extrinsic/6814916-3
TL;DR His funds are on AssetHub, no DOT were lost

Ok thanks a lot for the clarification and sorry for the missunderstanding!


Update smallweb.txt - add alexsirac.com

added alexsirac.com (RSS feed only including posts in English, not all posts, to comply with the Kagi rules)

Hey Alex per guidelines when submitting own site, add two more blogs that are not yours and are not already in the list.


Feature request: Option to register/signup users

Currently there is no option for registration/signup for users. This feature would make onboarding new users easy and quick.

This can be easily done by changing two booleans in the code since Cognito and the Amplify Authenticator library offer this out of the box.
Cognito User Pool
Amplify Auth lib

@nagelpat Thanks, should I make the changes and raise a PR or this has not been done intentionally and we won't be doing it?

@ajaylamba-provar This is done intentionally. If you enable the registration, anybody who has access to the CloudFront Distribution will be able to sign up and use the application. I believe this was designed like this for security reasons. Otherwise, you will need a VPN or another mechanism to prevent unwanted users from registering and accessing the application.

Hmm ok. Makes sense.
Closing this then.


fn: uds timeout and cookie run ordering

UDS timeout is now set with request/call timeout. UDS wait
is moved to hot slot execution. This means, UDS startup time
is part of user function call execution. Alternatively, we
could set specific timeouts for this, but bundling it to
function call timeout seemed simpler.

Cookie Run in case of HTTP Stream format is very similar
to UDS timeout. Reordered this so that UDS wait accomplishes
the same barrier. This is a significant change as it incorporates
function/container start time into function execution time. This
means a cold start function needs to take into account this duration
as well as UDS startup time and function execution time.

This means, if a Java container with significant startup time of
10 seconds, with UDS startup delay of another 1 second, should
allocate at least 12 seconds of timeout for the function. In this
example, function execution is left with 1 seconds if cold, 12
seconds if hot execution.
Previously, both of these UDS startup and Cookie Run did not
have associated timeouts.

thanks, yea, I hit this one too... I have a patch I need to clean up but I think it's a bit simpler, there's less moving parts (just moved the context around, instead of moving the parts): https://github.com/fnproject/fn/compare/fdk2-test-image?expand=1#diff-02ea3c2166e8abe7a3c4684f68d78db5R781

Prefer to merge this after #1281 to remove branchy code.


Export properly Typescript typings

Hello there,
really enjoying this Package!
Is there any news related to this Issue? I cant get it running in typescript.
import { ISignaturePad, SignaturePadSavedSignatureDataURL, SignaturePadSavedSignatureFile } from '@/types';
gives me the error ts(2307) (Cannot find module)


Trim the debug log

While running plan, lots of apparently libjuju-related logs are added to the cou debug log:
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: charm add ch:amd64/focal/openstack-dashboard-601
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: relation add 8
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: charm add ch:amd64/focal/ovn-central-35
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: relation add 9
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: charm add ch:amd64/focal/ovn-chassis-33
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: charm add ch:amd64/focal/placement-99
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: relation add 10
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: charm add ch:amd64/focal/hacluster-113
2023-09-24 01:15:05 [juju.model] [DEBUG] Model changed: relation add 11

These are very misleading, as they make it appear as if cou is performing model manipulations. They should be hidden.

I'm debugging one cou behavior and the logs are unusable, they are huge due debug log from libjuju. I think we should filter all debug messages from libjuju.


Header refactoring

Description
This PR merges in the changes for the Header refactoring.
Motivation and Context
Resolves https://github.com/IBM-Swift/Kitura/issues/404
How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

This change is

After a lot of back and forth, we are finally ready to merge this in! It will be tagged immediately so the Kitura PR can see it.


lower bound for ReadOptions

Test may failed, because ci don't use a specified version of RocksDB.

Wait this pr?

When will we seek with a key smaller than the lower bound? Is that reasonable?

RocksDB updated with lower bound

Can you open another PR to update rocksdb first?


modified the dockerfile for better caching

Split the copy operation into minimal setup and requirement.txt file and defer the full repo copy at the end.
This prevent full dependency rebuild every time a user makes local changes and wants to quickly build an re-run on a remote cluster.

cc @tonyjohnchen, I believe this is the use case of docker_upload_runner.sh

Added one more fix.
Jax jax lib installation should be done last in the setup.sh
Since we allow user to control it via the build args.
The current order, install dependencies from requirements.txt last.
If any library in this list causes jax/jax lib version overwrite, the input from the the users will not be remain in the final build.
This leads to painful jax version debug.
To repro, you can build with JAX_VERSION=0.4.30 and MODE=stable
The final build will have jax/jaxlib and 0.4.33, which is not expected.
The reason is that jax version gets overwritten during the dependency installation via requirements.txt.


Vite prevents native autocomplete to be disabled

Describe the bug
I'm using KendoUI for vue in a Vite project.
When I'm using a combobox, or other input with a placeholder that refers to a part of an address, my browser tries to propse an auto-completion. This native auto-completion popup is over the page, so it hides (part of) the component's real popup
The component sets autocomplete=off to the input, but it's ignored when vite is used, here is an exemple of such an element :
<input autocomplete="off" id="4a5d55d9-d5c7-4d43-89d3-febb020d5031" type="text" placeholder="Société" class="k-input-inner" role="combobox" size="20" aria-autocomplete="list" aria-haspopup="listbox" aria-expanded="false" aria-controls="38d3a8ef-ffb7-4312-a715-fac368d8f157">

As a workaround, I can rewrite role=combobox in role=presentation, but :

it does not work great with dynamic elements
the correct role attribute is required for aria

Kendo support says the native autocomplete is showing because of vite, but did not explained why
Reproduction
https://www.geekwu.org/~bastien/demo_vite/vite/
Steps to reproduce
Run npm install followed by npm run dev
click on a dropdown, if you have some address recorded in your browser, if should pop :

System Info
System:
    OS: Linux 6.8 Arch Linux
    CPU: (4) x64 Intel(R) Core(TM) i5-4440 CPU @ 3.10GHz
    Memory: 2.94 GB / 15.29 GB
    Container: Yes
    Shell: 5.2.26 - /bin/bash
  Binaries:
    Node: 18.18.2 - /usr/bin/node
    Yarn: 1.22.22 - /usr/bin/yarn
    npm: 10.5.2 - /usr/bin/npm
    pnpm: 8.15.2 - /usr/bin/pnpm
  Browsers:
    Chromium: 124.0.6367.78
  npmPackages:
    @vitejs/plugin-vue: ^4.2.3 => 4.3.4
    vite: ^4.5.2 => 4.5.3

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I don't think this is caused by Vite. The HTML includes the autocomplete attribute correctly.
Browsers have been changing the behavior to disable "disabling autocomplete" and autocomplete=off doesn't work reliably.
https://developer.mozilla.org/en-US/docs/Web/Security/Securing_your_site/Turning_off_form_autocompletion#disabling_autocompletion:~:text=If a browser keeps on making suggestions even after setting autocomplete to off%2C then you have to change the name attribute of the input element.
https://stackoverflow.com/questions/60689757/how-to-disable-chrome-autofill-after-2020


maven build fails

The maven build fails, as the org.xenei:PA4RDF:1.1.1 dependency could not be resolved.

Closing this issue as fixed.


style: Allow using the ruff formatter using hatch fmt

Codecov Report
Attention: Patch coverage is 93.75000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 99.65%. Comparing base (afa9aa6) to head (b437043).

Files with missing lines
Patch %
Lines

tests/test_limiter.py
86.66%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##            master      #11      +/-   ##
===========================================
- Coverage   100.00%   99.65%   -0.35%
===========================================
  Files            2        2
  Lines          577      583       +6
  Branches        37       38       +1
===========================================
+ Hits           577      581       +4
- Misses           0        1       +1
- Partials         0        1       +1

Flag
Coverage Δ

unittests
99.65% <93.75%> (-0.35%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Video not able to play in iPad ios 9.2?

I am using Streaming-Meda-Cordova-Plugin version 0.1.4 for playing videos.
My Cordova version is 6.0.0
It is not playing in iPad which has an IOS 9.2 version. but it works well in my iPhone which has an IOS 7.0 version.
I am passing on my video path just by one simple line of code.
window.plugins.streamingMedia.playVideo(path);
It works pretty well in Android device and IOS mobile phones.
But not in iPad.
Even i tried to check the console for the path. so that there should not be path issue but it display the correct path in iPad console after debugging. When i click on play button there is no response it is not even showing the default video player.
Even i want to know why the sound is not coming while the hardware mute button is enable in IOS devices?
Can you help me to figure out what exactly the issue is and how to resolve it?

Hello, I have the same problem. Did you found any solution? @Kailash43

@Kennyhp nope i didn't got any solution for this issue.

I have the same problem,😢. It can't run on iOS,  I try to use HTML5 video!


Skeleton3D has bone indexes unpredictably shuffled around when importing GLTF

Godot version
4.0 alpha 9
System information
Windows 10, Vulkan, NVIDIA GTX 850M
Issue description
Every time a new GLTF scene containing a complex character skeleton is exported from Blender and then imported into Godot, the bone indexes are shuffled around and cause unpredictable animation and skeleton modification behaviors, requiring me to manually re-add the model to the player scene and change the modified bone indexes in the SkeletonModificationStack3D in order to fix the problems.
Note 1: This happens even if I do not modify anything in the Blender scene. I just need to export to a new glb file and Godot shuffles the indexes and ordering around when importing. The imported scene may stay the same if I re-export it in the same Blender session, but if I just close Blender, open it again and then try to re-export the scene, it is already unpredictable.
Note 2: This does not happen in the 3.x branch of Godot. The bone indexes and ordering are preserved as expected.
How the scene cointaning the model looks when everything is in place:
https://user-images.githubusercontent.com/79013106/171686022-137469d5-6198-411c-8b08-c1c4c1e8c6f3.mp4
How the scene containing the model looks after just closing and opening Blender and re-exporting the model without any modifications in the blender scene:
https://user-images.githubusercontent.com/79013106/171705250-7e15137f-5ecb-4d5e-92dd-0f7c0db5b864.mp4
(This is random, I never know what kind of crazy shape the character will take in the next export)
Steps to reproduce
Create a character with a complex armature in Blender (in this case, I used rigify and only exported deformation bones), export it to glb, add the model to a scene and set up the scene. Then try closing Blender, opening it again, and re-exporting the character to the same file and watch the scene that contains the model break with bone ordering issues.
Minimal reproduction project
No response

I think this is a problem with Blender exporter.

Also, can you try specifically Blender 3.2, the unreleased version?

It seems exporting from from this version does keep the bone indexes stable and essentially fixes the problem... although it has created its own set of problems, such as some bones being strangely twisted.

I wonder if that is something I have to fix myself in the scene. I'll investigate further when I have some more time.

Please try to contact the gltf blender team. https://github.com/KhronosGroup/glTF-Blender-IO/issues @julienduroure

Upon further inspection and testing, it seems Blender 3.2 is in fact twisting the bones strangely on export, it even gets twisted when re-importing the GLB into Blender. I'll try to open an issue over there.
However I still find it kinda strange that Godot 3.x imports a Blender 3.1 GLB with consistent bone indexes every time, while they get all messy in Godot 4.0 with the same files...

Upon further inspection and testing, it seems Blender 3.2 is in fact twisting the bones strangely on export, it even gets twisted when re-importing the GLB into Blender. I'll try to open an issue over there.
However I still find it kinda curious that Godot 3.x imports a Blender 3.1 GLB with consistent bone indexes every time, while they get all messy in Godot 4.0 with the same files...

@antourenein For Blender issue, please open a ticket here [1] with your test file. Thanks!
[1] : https://github.com/KhronosGroup/glTF-Blender-IO/issues

One thought that comes to mind is the possibility that the sorting algorithm has changed in Godot4. I don't know if this is true or not, but if the bone indexes are ordered by hierarchical and use unstable sort (https://en.wikipedia.org/wiki/Stable_sort), it is possible that they could be randomized on import.

Alright, I created a minimum representation of the model and its blender scene, which reproduce all the problems without needing to carry the entire original model.
I replaced it with a simple dummy and removed its huge list of animations, and left it with the idle animation only, for simplicity's sake.
Here's a folder with a minimally similar setup to the one I'm using in my project:
dummy-scene.zip
The structure looks like this:

The scene has a simple AnimationTree setup and a LookAtTarget node, which is the target the SkeletonModificationStack3DLookAt references (if it doesn't work, just move the node around the structure and it should cache)

The "original" folder contains the gltf exported from Blender 3.1 that works with the "dummy-setup.tscn" without problems. This is the same "dummy.gltf" that is in the root. The "alt" folder contains a gltf that was exported after closing and reopening Blender 3.1. If you replace the "dummy.gltf" and its binary in the root folder with the one in the "alt" folder, the "dummy-setup.tscn" looks just utterly broken.
I've included an export from Blender 3.2 in the "blender3.2" folder, too, for reference.
Now, here is the blend file, so you can try the export process yourselves and investigate what might be happening:
dummy-blend.zip
This blend file has my gltf export settings saved, too.

If you open alt/dummy.gltf and original/dummy.gltf in a text editor instead of opening them in Godot, you can see that the order of the nodes in them is different. So, it is not a bug in Godot for now.
Previously, we changed to use BoneName instead of BoneIdx in some Class to solve this problem, but it still looks unstable. And I am planning to discontinue it in #61596.

I can think of an alternative solution to this: after building the BoneTree in Impoter, reorder the items in the same level of the tree by BoneName and reassign the BoneIdx, which will generate a same BoneIdx regardless of the order of the nodes, as long as the number of bones does not change.
However, as a side effect, the order of the tree on gltf is different from the order of the tree in Godot. For example, the little finger may be listed before the thumb. Apart from that, we can stabilize the imported BoneIdx.
How do you think @fire?

Fixed in https://github.com/KhronosGroup/glTF-Blender-IO/issues/1659

I tried out exporting the scene from a new daily build of Blender 3.2 RC and it seems all problems relating to bone indexes really have gone away, as well as the strange twisting of bones!

 Death to "universities".  Death to open source. Hope you have a tragic life and painful death.

This bug is fixed on Blender side since June, this ticket can be closed!

Sounds good. Can close any time.


Added support to glance-simplestreams-sync

versionless property

Closes: #104

Merge after #125

Closing because #156 will be the approach.


✨ Stabilization of StrictlyPositiveDouble type

📝 Description
We would like to stabilize the StrictlyPositiveDouble type.
🔗 Dependencies
This issue is blocked by the following ones:

[ ] #355
[x] #485
[x] #487
[x] #567

✅ Checklist

[ ] Remove its experimental annotation, update its versioning annotation, update its tests and the unreleased changelog.
[ ] Close this issue as completed and update tracking ones if present.

See #628 for more details on why we are not going to implement this issue for now.


iTranslate: Translate into multiple languages at the same time

Description
Add new command Translate to Multiple: Translate into multiple languages at the same time
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

To solve the #2715


plugins/graphite: fix accessor init

Not caught by the tests because all this part it mocked.. for now.

This change is

 :

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Encoding inconsistency in get_current_track_info

All text returned by soco should unicode objects. get_current_track_info does not seem to respect that, as can be seen from the two python 2.7 example outputs below:
{u'album': 'Everything Glows', u'artist': 'D-A-D', u'title': 'As Common As', u'uri': 'x-file-cifs://TLE-SERVER/share/ogg/D-A-D%20-%20Everything%20Glows/11%20-%20As%20Common%20As.ogg', u'playlist_position': '11', u'duration': '0:04:23', u'position': '0:02:47', u'album_art': u''}
# ... and ...
{u'album': u'', u'artist': u'', u'title': '17.mp3', u'uri': 'http://www.noiseaddicts.com/samples/17.mp3', u'playlist_position': '12', u'duration': '0:00:04', u'position': '0:00:03', u'album_art': u''}

Ideally I think, that when the radio stations has been brough into the data structures fold, then this method should return data structure objects according to their type, but for now, this should be fixable by turning all strings into unicode after they have been pulled out of the XML.

@KennethNielsen
I know its been a a while, but think this is handled correctly now. A unicode string is returned whenever necessary. Please re-open if I am missing the point!

Ahh, but that was the point. I thought we wanted to return unicode object always, whether they were necessary or not, to ensure consistency in the output. What do you think?


fixed two YAML inconsistencies

fixed YAML inconsistencies:

? is not allowed in this case since in YAML it indicates a complex mapping key
in Ermolaev-e.yaml, a random 's' appears in the refractive index data, preventing to use e.g. np.loadtxt()

merged


Safari bug?

This is a mysterious Expand-Collapse-bug with Safari.  All actions are done with mouse clicks.

Open https://agnt.eu/ghwmm
Click left menu icon
Expand Warenkunde
Click on Das Messerhandwerk
Now go back via BROWSER BACK
Collapse  Warenkunde
Expand  Warenkunde

Now sub menu expands and collapses automatic infinite without any user action.

It seems to me, that safari is the only browser, which saves the view state of a dropdown. When going back in Safari via browser back the state is expanded. In all other browers, the state is reseted to collapse.

Its true :(

I have seen this too in the wild, I have overridden this in the past with jQuery to fire a page reload if event.persisted exists, but this is not a universal solution as most people would prefer not to reload the page. I had to do this because even targeting the menu to close via classes did not work. Only a hard reload worked.
Safari's handling of bfcache could be preferable for most things, but in this case we would rather not save the state of the DOM. A case specific solution for just this plugin would be the way to go, we just need to be careful not to effect other DOM elements that are preferably cashed. The team should discuss a solution

This also appears to be just the latest versions of Safari.
I can't replicate this on Version 8.0.8 (10600.8.9) for example.

Here is my code, I have had good luck with doing this. Of course watch your div IDs to match.
       $("#accordion-nav-menu ul li a").click(function(e) {
			e.preventDefault();

			if ($(this).next().length === 0) {
				//close all submenus

				$('#accordion-nav-menu li.has-submenu').attr("aria-selected","false").attr("aria-expanded","false");
				$('#accordion-nav-menu li.has-submenu > ul').attr("aria-hidden", "true").removeClass("is-active").removeAttr("style").css("display","none");
				$('#accordion-nav-menu').foundation('hideAll');

				//close main menu
				$('#main-nav').removeClass("is-active");
				$("#nav-icon").removeClass("open");

				//wait a tick and fire url
				setTimeout(function() {
					window.location.href = e.target.href;
				}, 500);
			}
		});

It's not a fix, but it works for me for the time being.

Just thought I'd add my input. I can't recreate this in Safari 10, so in at least Safari 10 this isn't an issue; I'm also on macOS Sierra.

@coreysyms it sounds like you have a sense of what is going on here - I'm not super familiar with bfcache and how it works... is there an event we can listen for here and trigger a reset?  Are there any other components where this would make sense?

https://coderwall.com/p/3t6ila/disable-the-back-forward-cache-of-firefox-and-safari
https://foundation.zurb.com/sites/docs/compatibility.html

We support the following browsers by the browserslist config:
npx browserslist
npx: Installierte 3 in 2.257s
and_chr 64
and_ff 57
and_qq 1.2
and_uc 11.8
android 62
android 4.4.3-4.4.4
android 4.4
android 4.2-4.3
android 4.1
android 4
android 3
android 2.3
baidu 7.12
bb 10
bb 7
chrome 64
chrome 63
edge 16
edge 15
firefox 58
firefox 57
ie 11
ie 10
ie 9
ie_mob 11
ie_mob 10
ios_saf 11.0-11.2
ios_saf 10.3
ios_saf 10.0-10.2
ios_saf 9.3
ios_saf 9.0-9.2
ios_saf 8.1-8.4
ios_saf 8
ios_saf 7.0-7.1
op_mini all
op_mob 37
op_mob 12.1
opera 50
opera 49
safari 11
safari 10.1
samsung 6.2
samsung 5

Not reproducible on browserstack with iOS 7 and Safari 7 / webkit 9537.53

The browserslist config should be
last 2 versions
ie >= 9
Android >= 2.3

Which results in
and_chr 64
and_ff 57
and_qq 1.2
and_uc 11.8
android 62
android 4.4.3-4.4.4
android 4.4
android 4.2-4.3
android 4.1
android 4
android 3
android 2.3
baidu 7.12
bb 10
bb 7
chrome 64
chrome 63
edge 16
edge 15
firefox 58
firefox 57
ie 11
ie 10
ie 9
ie_mob 11
ie_mob 10
ios_saf 11.0-11.2
ios_saf 10.3
op_mini all
op_mob 37
op_mob 12.1
opera 50
opera 49
safari 11
safari 10.1
samsung 6.2
samsung 5

I can't recreate this in Safari 10, so in at least Safari 10 this isn't an issue

That's the same result that I got when I tested in our browserstack instance. All versions before 10 have this issue as far as I know.
As this is a browser specific bug and a known one it should be probably handled in projects, not in the framework (or we would end up adding fixes and workarounds for quirks and bugs in old browsers).
@ncoden what do you think should we do with this issue?

As this is a browser specific bug and a known one it should be probably handled in projects, not in the framework (or we would end up adding fixes and workarounds for quirks and bugs in old browsers).

I'm not saying that the framework should fix every strange, obscure bug in old browsers, or even this specific bug, but in general I do appreciate when framework work around the most common quirks. In fact, that's one of the reasons I use Foundation; to have a rock solid grid, a normalized look and some well tested, cross-browser working components so I can concentrate on the creative process.

@ncoden what do you think? Imo we should not add more workarounds for such quirks in older browsers, but this is just my opinion.

Such decisions should not be made on the basis of gut instinct. Is there no official roadmap?

Is there no official roadmap?

Sure, there is one: https://github.com/zurb/foundation-sites/wiki/Project-Roadmap

Closing for now. We should not use special workarounds for a few outdated browsers.


Viewing transactions on an account page shows wrong account when scrolling

I was here looking for transactions on my account:
https://explorer.testnet.near.org/accounts/oracle-node.p.mike.testnet
and as I scrolled down I started seeing transactions from another account that was not mine. Something seems to be wrong with the paging/infinite-scroll.

@mikedotexe good catch! Thank you for using explorer. Will fix it ASAP

Thank you! 🙌🏼


Angular is not defined

Hello Matt,
I have created my own meanjs project and installed all the recomended modules and directives, as well as the dependencies listed in the documentation section, however, whenever I try to load the calendar I get a console error stating that Angular is not defined and that the mwl.calendar module is not found. Have you got any clue on what might be wrong and how to fix it?? I have even downloaded your project and received the same error when attempting to compile the code. I would really appreciate your help, thanks

Hi there,
Please can you share some sample code with me that replicates the error then we'll go from there. Thanks!

We have not added any code, we just installed the calendar using the guide in the documentation section and although we have the angular link in the header of the view where the calendar is being added we are receiving the angular is not defined error which causes the Module 'mwl.calendar' not to be recognized:
Uncaught ReferenceError: angular is not defined
angular-bootstrap-calendar-tpls.js:765
Uncaught Error: [$injector:modulerr] Failed to instantiate module proyectointegrador due to:
Error: [$injector:modulerr] Failed to instantiate module mwl.calendar due to:
Error: [$injector:nomod] Module 'mwl.calendar' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.2.26/$injector/nomod?p0=mwl.calendar
    at http://localhost:3000/lib/angular/angular.js:78:12
    at http://localhost:3000/lib/angular/angular.js:1677:17
    at ensure (http://localhost:3000/lib/angular/angular.js:1601:38)
    at module (http://localhost:3000/lib/angular/angular.js:1675:14)
    at http://localhost:3000/lib/angular/angular.js:3878:22
    at forEach (http://localhost:3000/lib/angular/angular.js:325:18)
    at loadModules (http://localhost:3000/lib/angular/angular.js:3872:5)
    at http://localhost:3000/lib/angular/angular.js:3879:40
    at forEach (http://localhost:3000/lib/angular/angular.js:325:18)
    at loadModules (http://localhost:3000/lib/angular/angular.js:3872:5)
    at createInjector (http://localhost:3000/lib/angular/angular.js:3812:11)
http://errors.angularjs.org/1.2.26/$injector/modulerr?p0=mwl.calendar&p1=Er…%20(http%3A%2F%2Flocalhost%3A3000%2Flib%2Fangular%2Fangular.js%3A3812%3A11)
    at http://localhost:3000/lib/angular/angular.js:78:12
    at http://localhost:3000/lib/angular/angular.js:3906:15
    at forEach (http://localhost:3000/lib/angular/angular.js:325:18)
    at loadModules (http://localhost:3000/lib/angular/angular.js:3872:5)
    at http://localhost:3000/lib/angular/angular.js:3879:40
    at forEach (http://localhost:3000/lib/angular/angular.js:325:18)
    at loadModules (http://localhost:3000/lib/angular/angular.js:3872:5)
    at createInjector (http://localhost:3000/lib/angular/angular.js:3812:11)
    at doBootstrap (http://localhost:3000/lib/angular/angular.js:1444:20)
    at Object.bootstrap (http://localhost:3000/lib/angular/angular.js:1459:12)
http://errors.angularjs.org/1.2.26/$injector/modulerr?p0=proyectointegrador…%20(http%3A%2F%2Flocalhost%3A3000%2Flib%2Fangular%2Fangular.js%3A1459%3A12)
angular.js:78

Are you including my calendar code before your code which instantiates the proyectointegrador module?

proyectointegrador is not a module it is the name of the whole project, would it help if I upload the project to github for you to see it?

Yes please that would be great

We have changed the links and angular no longer returns an error, so the error I included in my previous entry is no longer received, Thanks!  however, now we are getting this error. :
The value passed to the view attribute of the calendar is not set
The value passed to current-day attribute of the calendar is not set
Is there something we should add for it to work?

Yes you need to set those variables. See the documentation:
https://github.com/mattlewis92/angular-bootstrap-calendar
An example controller:
https://github.com/mattlewis92/angular-bootstrap-calendar/blob/master/docs/scripts/demo.js

I've got the same problem. It would have been nice to see how it was fixed...

At a guess it's because you're including the script files in the wrong order. It needs to be angular, then this calendar / any other vendor files, then your code

I think I found the issue, and I'm almost too embarrassed to say...
...
//= require angular-file-upload
//= require angular-rails-templates
//= moment/min/moment.min
//= angular-bootstrap-calendar/dist/js/angular-bootstrap-calendar
//= require myApp
//= require_tree .

Anyway, I'm now seeing the following error on starting Rails
ActionController::RoutingError (No route matches [GET] "/assets/angular-bootstrap-calendar/dist/js/calendarUtils.js.map"):

Oops, that shouldn't be included, it will be fixed in the next release: https://github.com/mattlewis92/angular-bootstrap-calendar/commit/25b131b65ac54edd62cd3618e0d14a2bfe3246f5


Add support for comma separated IntSliceFlag and Int64SliceFlag

What type of PR is this?

[x] bug
[ ] cleanup
[ ] documentation
[ ] feature

What this PR does / why we need it:
IntSliceFlag can't parse comma separated  like this -flag 1,2,3.
This PR adds support for that by :

Checking for comma in IntSliceFlag.Set and Int64SliceFlag.Set
If comma exists split the value string and add each number to the underlying slice

Which issue(s) this PR fixes:

Fixes #1097
Special notes for your reviewer:
My only knowledge of this project is from using it.
I came up with a pretty naive solution to this problem, so I would be happy to know if the change is implemented in the right place.

Testing

Added unit tests :

TestParseMultiIntSliceCommaSeperated
TestParseMultiInt64SliceCommaSeperated

These tests wouldn't fail if the error of Run isn't checked.
All the other parsing tests do the checking in Action but these tests won't even get there if the value is not valid.
Release Notes

Add support for comma separated `IntSliceFlag` and `Int64SliceFlag`

@lynncyrin The code currently errors on that input. Consider the following program:
package main

import (
	"fmt"
	"log"
	"os"

	"github.com/urfave/cli/v2"
)

func main() {
	app := cli.NewApp()
	app.Flags = []cli.Flag{
		&cli.IntSliceFlag{
			Name: "val",
		},
	}

	app.Action = func(c *cli.Context) error {
		fmt.Println(c.IntSlice("val"))
		return nil
	}

	if err := app.Run(os.Args); err != nil {
		log.Fatal(err)
	}
}

With go run main.go -val 5000 we get [5000]. If we pass 5,000, we get:
Incorrect Usage. invalid value "5,000" for flag -val: strconv.ParseInt: parsing "5,000": invalid syntax

That means we can add support for this feature without breaking backward compatibility.
I think that's a good call out that it allows potentially ambiguous inputs, but I'm okay with it in this case. We don't support that style of input in any other context (e.g., IntFlag), and it's consistent with how we handle slices of strings.

Should we be fixing float64 slices as well? From my initial look at the code, I think that would just be an identical change in flag_float64_slice.go, so that probably belongs in this PR.

Yeah, I can do that.
Also searching through the tests, I can see that there are no tests TestParseMultiFloat64Slice like there are for IntSlice so I am gonna add those too while I am at it.

I think the string slice flag does not support such behavior, right? That's why I had to add a workaround in a urfave/cli consuming project. I know that comma separation could be intended even for string slice flags, but maybe we would like to add an option for such cases as well.

You are right that it doesn't support comma separated string slices from command line.
If you pass string slice flag -s foo,bar the value will be ["foo,bar"] not ["foo", "bar"].

You are right that it doesn't support comma separated string slices from command line.
If you pass string slice flag -s foo,bar the value will be ["foo,bar"] not ["foo", "bar"].

We could add an option which enables that kind of parsing :thinking:

It could be done but I have no idea how I should implement that since this is the first time me touching this codebase aside from using it in my own applications :sweat_smile:

Sorry, I was under the impression that we were adding this to make it consistent with the behavior for strings.
The original issue (#1097) specifically cites:

This works for StringSliceFlag.

If that's not the behavior for string slices, then is #1097 really a bug, or just a misunderstanding of how the library was designed to work?
If we don't yet support this behavior for any sort of slice flag, I would actually caution against adding it at all. We can't actually make string slices behave this way at this point without breaking backward compatibility. Although it's not trivial, it is absolutely possible to get this behavior for very customized CLIs by implementing the cli.Flag interface manually.
So my recommendation here unfortunately would be not to add in this behavior right now.
If the change is desired, the process should be to open a feature request rather than a bug ticket, weigh whether we actually want this behavior against the cost of making a breaking change for StringSliceFlag, and then should we accept it, make sure the PR that implements it covers all four of the flag slice types together.

I agree with @rliebz and I guess we should not automate any splitting. I think it would make sense in general to be added as optional feature.

If that's not the behavior for string slices, then is #1097 really a bug, or just a misunderstanding of how the library was designed to work?

It must be a misunderstanding. Following programm :
package main

import (
        "fmt"
        "log"
        "os"

        "github.com/micro/cli"
)

func main() {
        app := cli.NewApp()
        app.Flags = []cli.Flag{
                &cli.StringSliceFlag{
                        Name: "val",
                },
        }

        app.Action = func(c *cli.Context) error {
                fmt.Println(c.StringSlice("val"))
                return nil
        }

        if err := app.Run(os.Args); err != nil {
                log.Fatal(err)
        }
}

When run go run main.go -val foo,bar would return [foo,bar] instead of [foo bar]
It doesn't fail with a parse error like in the issue, it just reads the the given string as one value.
So this really is not a bug, but I would be glad to open up this as a feature request and implement it.

Here's another example that builds off the one above
# main.go

package main

import (
	"fmt"
	"log"
	"os"

	"github.com/urfave/cli/v2"
)

func main() {
	app := cli.NewApp()
	app.Flags = []cli.Flag{
		&cli.StringSliceFlag{
			Name: "val",
		},
	}

	app.Action = func(c *cli.Context) error {
		val := c.StringSlice("val")
		fmt.Println(val)
		fmt.Println(len(val))
		return nil
	}

	if err := app.Run(os.Args); err != nil {
		log.Fatal(err)
	}
}

$ go run main.go -val cat,hat
[cat,hat]
1

So yep, looks like StringSliceFlag doesn't split on ,

Then I am closing this PR and gonna create a feature request for this since there are people that are interested in this kinda functionality


[guide] handling ARCs across clones / machines

Handling ARCs across machines (laptop, desktop, server, hpc, etc.) is not intuitive.
We need a practical guide / recommendation, e.g.:

DataHUB as "original" clone with all data
Laptop / desktop for small data, to create the ARC, for structuring and metadata annotation, creating scripts, etc.
Clone to server / HPC (ideally via git) where large data is stored for computations

Needs to include work with LFS, git / ARC commander handling.
Reminders about arc syncing (keep clones up-to-date)
notes on working with branches

I think the above are a great basis for recommendations!
I have worked on an ARC across machines and as suggested using git pull worked well to pull the changes made on the respective other machine, so this could also be a good general recommendation to use regularly.
However, this was before I added the raw data to the ARC. Now I am wondering how to e.g. synchronize my laptop without pulling all those large files. If we find a solution, this would also be an important recommendation.


Rework all the functions to become as auto-configuration

See https://github.com/spring-cloud/spring-functions-catalog/tree/main/supplier/kafka-supplier or https://github.com/spring-cloud/spring-functions-catalog/tree/main/consumer/kafka-publisher for examples.
I mark this issue as an Epic since every single function change could be tracked in their own issue/PR.

The issue has been fixed with a number of commits like this: https://github.com/spring-cloud/spring-functions-catalog/commit/e1874eee928cdbd666bf07c98abd1a1715fc55f6.
Now all the functions in the project are auto-configuration.


[Fix] point cloud export broken in Dockerfile

Rebased the docker image to one built on ubuntu 22.04 where the libraries are matching the required versions for pymeshlab. Pointcloud export is now working again.
The new image comes with some additional changes (only a side node):
Python is now version 3.10
CUDA is now version 11.6
Colmap needs to be compiled with c++14
Didn't find any issues with the new versions so far.
Fixes #1129

When will this merge in the main branch?

How to get version of python 3.10 + cuda 11.6 to run on google colab demo for nerfstudio? Any idea? I can change cuda 11.3 to 11.6 on google colab. Please let me know. Thanks in advance. It can help me fix ns-export issue maybe @tancik

How to get version of python 3.10 + cuda 11.6 to run on google colab demo for nerfstudio? Any idea? I can change cuda 11.3 to 11.6 on google colab. Please let me know. Thanks in advance. It can help me fix ns-export issue maybe @tancik

Sorry, I have no experience with Colab, also I don't see this related to the Dockerfile/docker image.

Thanks for looking into this. The documentation should also be updated to reflect the new prerequisites.

True, I updated the description and added some more hints there. The only requirement that chaned is actually the nvidia driver which needs to be compatible with CUDA 11.7.


Upper marks create new file

when jump to an upper mark taht set in other file ,it will create a new file in vscode.

it seems neovim has jump to the mark and send the buffer to vscode ,but vscode handle this buffer as new file.

Check your init.vim. For me, jumping to upper mark works properly.

when jump to an upper mark taht set in other file ,it will create a new file in vscode.

I have the same problem. Do you solve it?

when jump to an upper mark taht set in other file ,it will create a new file in vscode.

I have the same problem. Do you solve it?

Have you tried with empty config?

I use the empty init.vim, it still not work and make new buffer in vscode.

when jump to an upper mark taht set in other file ,it will create a new file in vscode.

I have the same problem. Do you solve it?

I found this problem only happens with windows and remote dev,if i use a local linux nvim and vscode it never happen

I am able to reproduce on windows.

I have a similar issue happening in multiple scenarios (not only jumping to marks), but most basic commands.
It seems to mostly happen when in a Windows remote session or when the system is under a heavy workload (compiling...), so I suspect a timing problem.

Getting this too - on Windows 10. Tried empty init.vim, no luck.

Same here. I think it may be related to the path separator character / vs \.
See the output of :marks (folder names)
 A      4    9 __vscode_neovim__-file:\\\c%3A\work\play\snek\1\app.fnl
 C   2052   10 __vscode_neovim__-file:///c%3A/work/play/morelib/lyte2d/src/api.c

Closing in favor of https://github.com/vscode-neovim/vscode-neovim/issues/1223


Some blocks should be treated as unstyled even when not defined on the blockRenderMap

This will guarantee if user does not define paragraphs will still split them into different blocks

This fixes https://github.com/facebook/draft-js/issues/395

Hi Howardjing, please feel free to finalize this PR, I will only be able to work on it after 15th of October.

@howardjing in case you missed the comment above from @mitermayer ^^

@adamlong5 thanks -- the PR is here: https://github.com/facebook/draft-js/pull/584

will close this PR in favour or https://github.com/facebook/draft-js/pull/584


New Check - Schema changed

Inspired by the two sessions by @SQLDBAWithABeard about the devops process and dbachecks I thought it's strange that there isn't there a check already that would alert that when we were out, or perhaps on holidays, somebody changed the schema and didn't put the change to the repo.
The repo part would be rather difficult to implement but a simply hash of sys.sql_definition, perhaps last modify_date from sys.all_objects should be enough to quickly see if the schema was changed or not.
This check as well as the #300 (suspended checks) would require some state persistence. Perhaps more tests would benefit from it? What would be the best way to do it? json files or tables?  I supposed it would depend whether the checks are supposed to be personal per dba or shared by a team.

I like this
(Reason it is not here is because I would use DLM Dashboard from Redgate to do it)
A configuration item for "days since change" or "date since last change"
a test for Has any object changed since that date?

Something to consider on checking dates is converting them to local time. In situations where instances are on UTC or some other offset would have to be accounted for when comparing that date to what is in your versioning system.
Configuration wise, you could simply make a configuration that held the date value. If you are aware when the last change was submitted to your versioning system for a given database, just keep that value updated in your configuration prior to running the check(s).

All date comparisons should be using UTC - It will probably be easier in the first instance to display and configure  in UTC as well. As long as we make it clear that that is what we are doing


Review solution for missing unit tests

There are lots of missing unit tests for various things.  They need writing.

Tests needed:

FractionType

FractionType is untestable, due to using static NH methods.


Error "Cannot read property 'UriResolver' of undefined" in v2.4

We've updated our dependencies and all of a sudden ZOS starts to fail, even when called without any arguments:
zos
/home/doodloo/.config/yarn/global/node_modules/@resolver-engine/imports-fs/build/index.js:16
    UriResolver: core_1.resolvers.UriResolver,
                                  ^

TypeError: Cannot read property 'UriResolver' of undefined
    at Object.<anonymous> (/home/doodloo/.config/yarn/global/node_modules/@resolver-engine/imports-fs/build/index.js:16:35)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at Module.load (internal/modules/cjs/loader.js:600:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
    at Function.Module._load (internal/modules/cjs/loader.js:531:3)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (/home/doodloo/.config/yarn/global/node_modules/zos/lib/models/compiler/solidity/SolidityProjectCompiler.js:24:22)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at Module.load (internal/modules/cjs/loader.js:600:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
    at Function.Module._load (internal/modules/cjs/loader.js:531:3)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (/home/doodloo/.config/yarn/global/node_modules/zos/lib/models/compiler/Compiler.js:17:35)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at Module.load (internal/modules/cjs/loader.js:600:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
    at Function.Module._load (internal/modules/cjs/loader.js:531:3)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (/home/doodloo/.config/yarn/global/node_modules/zos/lib/commands/push.js:20:20)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at Module.load (internal/modules/cjs/loader.js:600:32)

Doing a bit of research showed another project (cryptopunkers) with the same error - given who handled the issue, I'm wondering whether a bad dependency made its way into ZOS ;)
As I'm not sure what exactly got broken, a hint on what dependencies to force would be great until resolution!

Hello @hickscorp! Sadly, this issue was introduced after releasing v2.4. We are already tracking it in https://github.com/zeppelinos/zos/issues/1046, and we have already pushed a fix in https://github.com/zeppelinos/zos/pull/1059 which we will release early this week.
I can provide you two solutions in the meantime:

Install zos globally using npm

or

Install zos locally, but using npm instead of using yarn.

Hey @hickscorp, this issue has been fixed in 2.4.1. I'm closing it, but please comment if you stumble upon it again. Thanks for the report!

@spalladino This issue might be closed now, but a new one surfaced. Should I open a new issue?
$ /whatever/node_modules/.bin/zos push --verbose --network cv_dev --skip-compile
<EMAIL_ADDRESS><started> Using session with network cv_dev, sender address 0x717634cfe06FFAB2CEAA7fcf1b9019813f4B25FE, timeout 30 seconds
<EMAIL_ADDRESS><started> Uploading AddressSet library
<EMAIL_ADDRESS><started> Deploying logic contract for AddressSet
<EMAIL_ADDRESS><started> TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined
    at validateString (internal/validators.js:125:11)
    at Object.basename (path.js:1289:5)
    at Object._log (/whatever/node_modules/zos-lib/src/utils/Logger.ts:126:32)
    at Object.succeed (/whatever/node_modules/zos-lib/src/utils/Logger.ts:101:10)
    at NetworkController.<anonymous> (/whatever/node_modules/zos/src/models/network/NetworkController.ts:277:11)
    at Generator.next (<anonymous>)
    at fulfilled (/whatever/node_modules/zos/lib/models/network/NetworkController.js:4:58)
    at process._tickCallback (internal/process/next_tick.js:68:7)
<EMAIL_ADDRESS><started> Updated zos.dev-18021981.json
error Command failed with exit code 1.

@hickscorp no need, already reproduced it. Expect a new patch soon!

@hickscorp actually, I have a hunch that the error may only happen in verbose mode. Could you check removing the --verbose flag?

Unfortunatelly our tooling change too much (We've been a bit lost between the @openzepelin/... packages and old zos / zos-lib and messed up our yarn.lock file). Be sure that I'll post back if we can reproduce it.


Fix test result info downloads

Motivation and Context

This PR fixes a few issues related to downloading test results as json or csv:

if two test_group_results have the same created_at value, the previous query could not determine which one is the most recent.

This PR solves this issue by selecting the associated test run with the largest id (created last) to break this possible tie

test data included grouping ids, test group ids which are only meaningful within the database and should not be made visible to end users

This PR removes those ids

the csv download format did not include the same information as the json download

This PR updates the csv download format so that no information is lost

Your Changes

Description:
Type of change (select all that apply):

[x] Breaking change (fix or feature that would cause existing functionality to change)

Testing

Questions and Comments (if applicable)

Checklist

[x] I have performed a self-review of my own code.
[x] I have verified that the pre-commit.ci checks have passed.
[x] I have verified that the CI tests have passed.
[x] I have reviewed the test coverage changes reported on Coveralls.
[x] I have added tests for my changes.
[x] I have updated the Changelog.md file.

Pull request to make documentation changes (if applicable)

revert change to csv file format

to revisit later


pip install ezbee 的时候报错

pip install ezbee 的时候报错呢  （win10 python3.8）
Collecting sklearn<0.1,>=0.0 (from fast-scores<0.2.0,>=0.1.3->ezbee)
  Using cached https://pypi.tuna.tsinghua.edu.cn/packages/46/1c/395a83ee7b2d2ad7a05b453872053d41449564477c81dc356f720b16eac4/sklearn-0.0.post12.tar.gz (2.6 kB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [15 lines of output]
      The 'sklearn' PyPI package is deprecated, use 'scikit-learn'
      rather than 'sklearn' for pip commands.

      Here is how to fix this error in the main use cases:
      - use 'pip install scikit-learn' rather than 'pip install sklearn'
      - replace 'sklearn' by 'scikit-learn' in your pip requirements files
        (requirements.txt, setup.py, setup.cfg, Pipfile, etc ...)
      - if the 'sklearn' package is used by one of your dependencies,
        it would be great if you take some time to track which package uses
        'sklearn' instead of 'scikit-learn' and report it to their issue tracker
      - as a last resort, set the environment variable
        SKLEARN_ALLOW_DEPRECATED_SKLEARN_PACKAGE_INSTALL=True to avoid this error

      More information is available at
      https://github.com/scikit-learn/sklearn-pypi-package
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

感谢报告问题。
这个是因为ezbee用的 fast-scores 包将 sklearn 列为依赖包。我已经更新了 fast-scores 去掉了sklearn 这个依赖包。
可以再试一下，有问题的话发到这里我再试着修一修。


fix fPowAllowMinDifficultyBlocks

We need to make sure that the new fPowAllowMinDifficultyBlocks only affects block 277299 and onwards, otherwise previous blocks that were created without this logic will be invalid because the difficulty calculations for the block will be incorrect.

LGTM. I made a comment for a possible small refactor to clarify the if clause but then I saw the comment block was also updated and very clear, so I'm ok with it.


Only delete insignificant entries

I think it'd be nice to set a threshold of significance so that it won't autodelete links that are more visited than others; say, the visit count. There are some reddit comment threads that I wouldn't want to be autodeleted, but I don't visit them very routinely anyway.
(Come to think of it, I think the right solution is to just use bookmarks, but eh.)

If a user can use regex rather than just a simple URL then they can craft a description of what they do and do not want to delete. This eliminates any question about the definition of "insignificant".


Create account

User Story Description
As a user
I want to create an account
So I can so I can chat with others
Steps to Follow (optional)

[ ] #69
[ ] 2
[ ] 3

Additional Considerations
Any supplemental information including unresolved questions, links to external resources, screenshots, etc.

Fixed....
https://github.com/chingu-voyages/v39-bears-team-11be-a/blob/main/controllers/userController.go
Provide the following user information to be able to create an account:
Username (min2, max100)
Email
Password


lowdown: 0.7.9 -> 0.8.0

Motivation for this change

Error handling improvements
Improved man and ms renderers

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

nixpkgs-review pr 112130

5 packages built:
hydra-unstable lowdown nix-update nixFlakes nixpkgs-review

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112130 run on x86_64-linux 1

5 packages built:

hydra-unstable
lowdown
nix-update
nixFlakes (nixUnstable)
nixpkgs-review


Use allowlist instead of blocklist in build script

Prerequisite:

[x] #28 (this doesn't account for channel_layout_fixed.h, which is why CI fails >= 5.1)

I don't think we need to expose anything outside of the FFmpeg library headers. There's currently a lot of stuff (see below) from the standard libraries.
On my machine (FFmpeg 6.1.1), bindings.rs shrinks from 24450 to 20200 lines.
I uploaded the diff (everything that was removed) as a gist. Some notable changes:

built-in consts (INT_MAX, libc errors), type aliases
libm stuff (exp, log10, sin)
file APIs
string APIs
timer API

Man, I wish Github had a Rebase button in the UI.

Give me a minute, I'll take care of it

Oh, I guess the paths don't match on Windows because of the \ path separator. Hm...


x/image/font/{sfnt,opentype}: expose more SFNT tables

What version of Go are you using (go version)?

$ go version
go version devel +135c9f45ec Wed Mar 31 04:13:44 2021 +0000 linux/amd64

Does this issue reproduce with the latest release?
yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/binet/.cache/go-build"
GOENV="/home/binet/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/binet/dev/go/gocode/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/binet/dev/go/gocode"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/binet/sdk/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/binet/sdk/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="devel +135c9f45ec Wed Mar 31 04:13:44 2021 +0000"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build354109135=/tmp/go-build -gno-record-gcc-switches"

text layouting and typesetting
in the context of text layouting and typesetting, it would be great for the sfnt package to expose way to access more informations from the font tables.
e.g. somebody trying to replicate or port (parts of) the HarfBuzz library would want to have access to the informations provided by the tables sfnt is currently decoding but hiding from public consumption:

https://godocs.io/github.com/benoitkugler/textlayout/harfbuzz
https://godocs.io/github.com/benoitkugler/textlayout/fonts/truetype#LayoutTables
https://pkg.go.dev/github.com/tdewolff/canvas/font#SFNT

for the purpose of drawing mathematical equations, having access to the math table informations would also be quite useful.

https://docs.microsoft.com/en-us/typography/opentype/spec/math
(but this might be a separate issue)

@nigeltao

Sure, but we need a specific API proposal, and I don't have much spare time to work on that myself.

I guess API-wise, there are 3 main avenues:
a) export a bunch of methods (attached to the sfnt.Font type) that return a specific exported table type:
func (f *Font) CMapTable()  CMapTable
func (f *Font) KernTable()  KernxTable
func (f *Font) KernxTable() KernxTable
func (f *Font) MathTable()  MathTable
// etc...

b) export a bunch of table fields, each with its own type
type Font struct {
    // ...
    CMapTable CMapTable
    MathTable MathTable
}

c) export a single sfnt.Font method that takes as argument a Table interface that knows how to decode itself from a provided range of bytes (the sfnt.table type)
type Table interface {
    Kind() TableKind // or just: Name() string where string is the 4b-tag. eg: "OS/2", "math", "maxp", ...
    decodeFrom(src table, f *Font) error
}

func (f *Font) Table(tbl Table) error {
    src, err := f.table(tbl.Kind())
    if err != nil { return err }
    return tbl.decodeFrom(src, f)
}

type MathTable struct { ... }
func (tbl *MathTable) Kind() TableKind { return MathTableKind }
func (tbl *MathTable) decodeFrom(src table, f *Font) error { ... }

// use:
func foo(f *sfnt.Font) {
    var tbl sfnt.MathTable
    err := f.Table(&tbl)
    if err != nil { panic(err) }
}

then comes the question of whether for the Table types:

to be "just" a view into the table+Font sea of bytes (and thus only export methods to decode+retrieve that data on each invocation) - this complicates a bit the ownership model of the src source field of the Font values, or
to have the Table types decode+store the decoded data into their own dedicated fields.

from the precedent of the existing sfnt.PostTable table, I guess the latter would be preferred, but that may not be advisable if the tables are large ( eg. the "glyf" table)

then comes the question of whether for the Table types:

Well, that's the important question for me. What are the actual fields of type CMapTable, etc.


Tag a release with compat bounds

@JuliaRegistrator register branch = main

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


update producer docs

Commit 8f101fa637020dc85214e058007e8c601a3cb868 changed the configuration to use a singular :producer rather than multiple :producers, so this updates the docs to match.

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Typo: "Project" vs "Projects" in menu

Using the default menu, the main page shows "Projects" in the menu. If I navigate to /blog/, the menu shows "Project" without the s. Can this be changed to display "Projects" consistently? Thanks very much to Victoria and other contributors for this theme.

I sent a pr that I think fixes this issue. I'm still learning my way around all the in's and out's of how this works.

Closed in #241. Thank you @thegreatluke!


Dependencies

Does it support python3.6?

I'm running it on Python 3.8.2 with no compatibility issues that I can tell.

Documentation states even older versions of Python.
https://python-docx.readthedocs.io/en/latest/user/install.html

Dependencies
Python 2.6, 2.7, 3.3, or 3.4
lxml >= 2.3.2