mypy raises many warnings

I ran mypy against the codebase. I got 38 warnings, albeit I admit that my mypy settings are rather strict.
From my experience fixing mypy issues helps to catch subtle and not-so-subtle bugs. Thus I think having detectable type violations is a sign that something is odd.
For convenience I post the list that was detected below. I think tackling these one by one could be beneficial.
src/tests/samples/violations/except_verbose_reraise.py:23: error: Name "func" already defined on line 12
src/tests/samples/violations/call_check_continue.py:21: error: Return value expected
src/tryceratops/analyzers/base.py:18: error: Method must have at least one argument
src/tryceratops/analyzers/base.py:29: error: "BaseAnalyzer" has no attribute "visit"
src/tryceratops/analyzers/try_block.py:29: error: Incompatible types in "yield from" (actual type "AST", expected type "stmt")
src/tryceratops/analyzers/exception_block.py:18: error: Argument 1 to "is_raise_without_cause" has incompatible type "AST"; expected "stmt"
src/tryceratops/analyzers/exception_block.py:38: error: Argument 1 to "is_raise_with_name" has incompatible type "AST"; expected "stmt"
src/tryceratops/analyzers/exception_block.py:54: error: Argument 1 to "_is_vanilla_exception" of "ExceptBroadPassAnalyzer" has incompatible type "Name"; expected "stmt"
src/tryceratops/analyzers/exception_block.py:58: error: Argument 1 to "_is_vanilla_exception" of "ExceptBroadPassAnalyzer" has incompatible type "expr"; expected "stmt"
src/tryceratops/analyzers/call.py:42: error: "expr" has no attribute "value"
src/tryceratops/analyzers/call.py:61: error: "expr" has no attribute "id"
src/tryceratops/analyzers/call.py:67: error: "stmt" has no attribute "targets"
src/tryceratops/analyzers/call.py:82: error: Argument 1 to "is_if_returning" has incompatible type "StmtBodyProtocol"; expected "stmt"
src/tryceratops/analyzers/call.py:83: error: Argument 1 to "append" of "list" has incompatible type "StmtBodyProtocol"; expected "stmt"
src/tryceratops/analyzers/call.py:86: error: "stmt" has no attribute "test"
src/tryceratops/analyzers/call.py:89: error: "expr" has no attribute "func"
src/tryceratops/analyzers/call.py:90: error: "expr" has no attribute "func"
src/tryceratops/analyzers/call.py:98: error: "expr" has no attribute "func"
src/tryceratops/analyzers/call.py:99: error: "expr" has no attribute "func"
src/tryceratops/analyzers/call.py:124: error: Argument 1 to "_scan_deeper" of "CallAvoidCheckingToContinueAnalyzer" has incompatible type "stmt"; expected "StmtBodyProtocol"
src/tryceratops/filters.py:69: error: "def () -> Tuple[builtins.str, builtins.str]" object is not iterable
src/tryceratops/filters.py:70: error: Cannot determine type of "code"
src/tryceratops/filters.py:70: error: Unsupported right operand type for in ("Optional[Iterable[str]]")
src/tryceratops/filters.py:79: error: Item "None" of "Optional[Iterable[str]]" has no attribute "__iter__" (not iterable)
src/tryceratops/analyzers/main.py:32: error: Cannot instantiate abstract class "BaseAnalyzer" with abstract attribute "violation_code"
src/tryceratops/analyzers/main.py:34: error: Only concrete class can be given where "Type[BaseAnalyzer]" is expected
src/tryceratops/analyzers/main.py:61: error: Argument 1 to "len" has incompatible type "Iterable[Tuple[str, AST, FileFilter]]"; expected "Sized"
src/tryceratops/files/parser.py:45: error: Argument 1 to "parse_tree" has incompatible type "TextIO"; expected "TextIOWrapper"
src/tryceratops/files/parser.py:48: error: Argument 1 to "parse_ignore_comments_from_file" has incompatible type "TextIO"; expected "TextIOWrapper"
src/tests/fuzztests.py:10: error: Missing positional argument "global_filter" in call to "analyze" of "Runner"
src/tryceratops/files/discovery.py:2: error: Library stubs not installed for "toml" (or incompatible with Python 3.9)
src/tryceratops/files/discovery.py:2: note: Hint: "python3 -m pip install types-toml"
src/tryceratops/files/discovery.py:2: note: (or run "mypy --install-types" to install all missing stub packages)
src/tryceratops/files/discovery.py:2: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports
src/tryceratops/files/discovery.py:50: error: Incompatible types in assignment (expression has type "Generator[str, None, None]", variable has type "List[str]")
src/tryceratops/files/discovery.py:103: error: Missing return statement
src/tryceratops/files/discovery.py:111: error: Missing return statement
src/tryceratops/files/discovery.py:112: error: Argument 1 to "find_pyproject_toml" has incompatible type "Sequence[str]"; expected "Tuple[str, ...]"
src/tryceratops/flake_plugin.py:32: error: Argument 1 to "parse_ignore_tokens" has incompatible type "Optional[Iterable[TokenInfo]]"; expected "Iterable[TokenInfo]"
src/tryceratops/flake_plugin.py:34: error: Argument 1 to "_create_global_filter" of "TryceratopsAdapterPlugin" has incompatible type "Optional[str]"; expected "str"
src/tryceratops/flake_plugin.py:51: error: Argument 1 to "analyze" of "Runner" has incompatible type "List[Tuple[Optional[str], AST, FileFilter]]"; expected "Iterable[Tuple[str, AST, FileFilter]]"
Found 38 errors in 12 files (checked 38 source files)

You're correct! How can we claim we're a linter if we don't even lint ourselves? haha
Thanks for the push. I'll try to take the time to solve that by the end of this week, this card is in the roadmap already, but hey,
Don't wait for me!
This project is ours ❤
if you @MaxG87 or anyone else has some time and are willing to contribute, please, go for it!
I just ask to follow the conventional commits standard.

I'll resolve this PR soon! 😁

I resolved just part of it and ignored some files.
I'd appreciate help from anyone in case interested :)


Fix startOfMonth and endOfMonth implementation for alternative calendars

Previous implementation's output was incorrect when using alternative calendars (e.g. Persian).

Hmm. Can you let me know what was the error you were seeing that needs this change?

When It was 1396/1/1 in the Persian calendar (21 April 2017 in the Gregorian), the firstOfMonth method returns January first of 1396 in Gregorian calendar.

i would really like to reproduce this issue.
Can you paste me your configureCalendar code?
I do not know how you set up the persian calendar.

I configure calendarView like this: (I changed the startDate and endDate to just Date())
func configureCalendar(_ calendar: JTAppleCalendarView) -> ConfigurationParameters {
        let testCalendar = Calendar(identifier: .persian)
        let startDate = Date()
        let endDate = Date()
        print(startDate)
        let parameters = ConfigurationParameters(startDate: startDate,
                                                 endDate: endDate,
                                                 calendar: testCalendar,
                                                 generateInDates: .forAllMonths,
                                                 generateOutDates: .tillEndOfRow,
                                                 firstDayOfWeek: .saturday)
        return parameters
}

and in the cellForItemAt function you can print it's date parameter and there is almost 600 years difference between them which is the difference between Persian and Gregorian calendar.

I have the same problem, it happens when the phone calendar is different from the one used in JTAppleCalendar.
for example when the phone locale is en_US and you use .persian Calendar for JTAppleCalendar.
aside from that I cant reproduce Persian calendar in latest version (7.0) as I did in version 6!?
btw, thanks a lot for your great works!

I think that after all this time, i finally understood the real issue.
Someone made a pull request last week and fixed it.
I simply forgot to set the formatter's time zone.
I think the Persian calendar bug is now resolved.
--> https://github.com/patchthecode/JTAppleCalendar/commit/83590428019788a97d2306f081c49ecc8be4b39b
If the issue is still not resolved, I can take a 2nd look.


Replacement on Commons is highly erratic

Sometimes it takes months to replace redirects. For example, Template:Railway line legend contains CPICr and KBHF-La, which have been redirects since 28 April and 20 May respectively. I have no idea why this is, but it might be because the global usage table only sometimes includes usage at Commons itself, so a separate search could be done on Commons if that's not done already.

Local usage should be included in the global usage. There is an API parameter to filter out local usage (gufilterlocal=1). The local filter works on other projects as it should, so I don't think that is the issue.
The API globalusage for CPICr doesn't include anything on Commons. Neither does Special:GlobalUsage for CPICr. It makes sense that those two match each other but not that Commons usage is not included.

Is there a Phabricator ticket for this yet?

I didn't see one at a brief glance, but I can look again tomorrow. If there isn't, I will create one.

T199398

I've remedied this to some extent by using a deleted file (which has a manual replacement specified) inside the navbox used at the bottom of the catalogue pages.

I implemented a workaround for this since I don't know if or when T199398 will be resolved.


Operators |= and ^= implemented

Solves #70

Thanks!
And you're more than welcome here to collaborate with us, as we're trying to learn the compiler concepts by making one. I'll create these "good first issues" more if you're interested.

Thanks for marking this with "good first issue"! This is the first I've contributed to a project out of work/school :)
Would definitely be interested in contributing more. I'd love to learn more about compilers.
Also I noticed that the ~ operator doesn't work? Didn't really dive deep into the code past your example for &=, but trying to use it in a test case gave an undefined op error.
Additionally I encountered an error in compilerChangeStack when running some of the tests.
Maybe there could be a "testing" section to the README? Not sure what python version to use, and I had to replace the 'pocket' command with '../build/debug/pocket'.
Thanks!

Thanks for reporting the issue, It's an assertion failure I've added after #67, and it'll fix it by today. but If you encounter a crash with a TODO: message (like this bellow) it's something that hasn't implemented yet.
>>> ~1
Assertion failed: TODO: It hasn't implemented yet.
        at runFiber() (C:\dev\pocketlang\src\pk_vm.c:1220)

Open an issue reporting these assertions, or if you want to implement it by yourself, I can help you on the issue thread. The source code heavily commented for readability and feel free to ask me if you have trouble understanding the internals.

@ThakeeNathees awesome, thanks!


Have an obvious way of finding how to add new extensions

@emmanuelbernard I think we are missing that

I don't understand.

BTW do you mind adding me as a member to the fabric8-launcher, It seems necessary to be able to change the email  address github sends you notifications to.

do you mind adding me as a member to the fabric8-launcher
@emmanuelbernard I just added you..

I don't understand.

I mean when you see the list of extensions, you may ask yourself:

Why is that extension missing?
How can I add/integrate a missing extension to the list

I still don't understand. In which context is the person asking that question.

@emmanuelbernard on code.quarkus.io, you can pick extensions, copy add extension commands, generate application but there is not place where you can find info on how to contribute extensions.
Example, I want to connect to a Cassandra DB, I type cassandra in the search box:

So I was thinking about adding a text like: "We don't have any extension for 'cassandra' yet, follow [this guide] to provide a new extension in the Quarkus ecosystem".
And something at the bottom of the full list. "You can't find the extension you are looking for? Follow [this guide] to provide a new extension in the Quarkus ecosystem


Unable to generate tf records

Im getting the following error while trying to generate tf records
 File "/Users/anaconda/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 48, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "generate_tfrecord.py", line 90, in main
    tf_example = create_tf_example(group, path)
  File "generate_tfrecord.py", line 79, in create_tf_example
    'image/object/class/label': dataset_util.int64_list_feature(classes),
  File "/Users/Documents/models-master/object_detection/utils/dataset_util.py", line 26, in int64_list_feature
    return tf.train.Feature(int64_list=tf.train.Int64List(value=value))
TypeError: None has type NoneType, but expected one of: int, long

Hi, its easy to fix
https://github.com/datitran/raccoon_dataset/blob/master/generate_tfrecord.py#L30
this function returns None for anything other than 'raccoon' you might want to change that to your desired class and you can add other classes with elif if u have more than 1 class
That function should never return None. It should return a int value depending on the class.

@P-a-i-s  Did you find the solution? I am still unable to do it.

@JoltonDsouza Hmm.. its been a while since I've completed this project. Trying to re-collect the reason for this error in my mind.
I am guessing you have multiple classes and hence are running into an error  when defining the if-else loops. Is that so?
If yes, and I pretty much suspect that. try the following:
if row_label == 'pineapple':
return 1
elif row_label == 'papaya':
return 2
elif row_label == 'mango':
return 3
elif row_label == 'banana':
return 4
elif row_label == 'kiwi':
else:
return 0
Most forums talk about return equal None. It should be zero.
Hope this works for you.

You just need to change the else statement so it returns 0 instead of "None".
def class_text_to_int(row_label):     if row_label == 'macncheese':         return 1     else:         return 0

Hi Admin sorry if i seem trolling old thread.
Hey @P-a-i-s , @rfgm6 , your solution works!. Made me happy until i made this reply just to thank you guys.
Found out the error prompt came out due to it reached EOL hence if 'None' will cause the module to output an error. But despite the error the file test.record is already in the folder.
Anyway i prefer my code to run error free.

@Shivakishore14 : that solution didin't work for me, unfortunately. I do have multiple classes and have used elif to describe them.
When checking across different solutions available on the net, most people (including datitran) pointed out that it might be a missing class or a misspell of a class in the train csv file. Am not able to figure that out since the labelling is done using labelImg, it saves these classes as xml, the xml_to_csv.py converts this to a csv. Am not sure under what circumstance I could have had the opportunity to miss out or misspel any class incorrectly.
Here's my error msg:
(OT) nishxxxx@xxx:~/Desktop/OD/models/research/object_detection$ python generate_tfrecord.py --csv_input=data/train_labels.csv --output_path=data/train.record
Traceback (most recent call last):
File "generate_tfrecord.py", line 192, in
tf.app.run()
File "/home/nishxxxx/Desktop/test_OD/OT/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 48, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "generate_tfrecord.py", line 184, in main
tf_example = create_tf_example(group, path)
File "generate_tfrecord.py", line 173, in create_tf_example
'image/object/class/label': dataset_util.int64_list_feature(classes),
File "/home/nishxxxx/Desktop/test_OD/models/research/object_detection/utils/dataset_util.py", line 26, in int64_list_feature
return tf.train.Feature(int64_list=tf.train.Int64List(value=value))
TypeError: None has type NoneType, but expected one of: int, long
Has anyone found a solution for this?

Hi, its easy to fix
https://github.com/datitran/raccoon_dataset/blob/master/generate_tfrecord.py#L30
this function returns None for anything other than 'raccoon', you might want to change that to your desired class and you can add other classes with elif if u have more than 1 class
That function should never return None. It should return a int value depending on the class.

I encountered the same issue, also i fixed the return to 0, the problem still persists

@JoltonDsouza Hmm.. its been a while since I've completed this project. Trying to re-collect the reason for this error in my mind.
I am guessing you have multiple classes and hence are running into an error when defining the if-else loops. Is that so?
If yes, and I pretty much suspect that. try the following:
if row_label == 'pineapple':
return 1
elif row_label == 'papaya':
return 2
elif row_label == 'mango':
return 3
elif row_label == 'banana':
return 4
elif row_label == 'kiwi':
else:
return 0
Most forums talk about return equal None. It should be zero.
Hope this works for you.

C:\Users\admin\Desktop\object-detection>python generate_tfrecord.py
Traceback (most recent call last):
File "generate_tfrecord.py", line 99, in
tf.app.run()
File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\platform\app.py", line 125, in run
_sys.exit(main(argv))
File "generate_tfrecord.py", line 85, in main
writer = tf.python_io.TFRecordWriter(FLAGS.output_path)
File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\lib\io\tf_record.py", line 112, in init
compat.as_bytes(path), compat.as_bytes(compression_type), status)
File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\framework\errors_impl.py", line 526, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: Failed to create a NewWriteableFile:  : The system cannot find the path specified.
; No such process
This is the error i'm facing

Hi Admin sorry if i seem trolling old thread.
Hey @P-a-i-s , @rfgm6 , your solution works!. Made me happy until i made this reply just to thank you guys.
Found out the error prompt came out due to it reached EOL hence if 'None' will cause the module to output an error. But despite the error the file test.record is already in the folder.
Anyway i prefer my code to run error free.

I also found the .record files in my directory even though I got the "TypeError: None has type NoneType, but expected one of: int, long" error, but will those .record files work despite the error?

@JoltonDsouza Hmm.. its been a while since I've completed this project. Trying to re-collect the reason for this error in my mind.
I am guessing you have multiple classes and hence are running into an error when defining the if-else loops. Is that so?
If yes, and I pretty much suspect that. try the following:
if row_label == 'pineapple':
return 1
elif row_label == 'papaya':
return 2
elif row_label == 'mango':
return 3
elif row_label == 'banana':
return 4
elif row_label == 'kiwi':
else:
return 0
Most forums talk about return equal None. It should be zero.
Hope this works for you.

If we are giving 0 as return, should we add the index in labelmap.pbtxt file too??

Just change the label name whatever you labeling them during crop the image by labelImg tool.
def class_text_to_int(row_label):
    if row_label == 'raccon':
        return 1
    else:
        None

Instead of 'raccon' put label name for ex:- 'car'.


[feature request] Allow requesting spot instance instead of an on-demand instance.

It would be nice to be able to request a spot instance when it fits the use-case, to reduce cost.
I think it would be straightforward to add a conditional in startEc2Instance (or another function entirely) to call requestSpotInstances instead of runInstances here.

+1

This would be a great addition. The current AWS SDK for Javascript v2 EC2.runInstances() allows definition of market options which would probably serve a good amount of use cases.
requestSpotInstances is deprecated and should not be used as per https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/spot-best-practices.html#which-spot-request-method-to-use

+1 for this!

Proposing to address this with #171. I have conducted few tests with an actual AWS account.

If I look at the pull requests, Im a bit worried that this repo isnt maintained anymore?

@tonyhutter u were the last to commit changes to this Repo. Maybe you can take a look at the proposed solution?

If I look at the pull requests, Im a bit worried that this repo isnt maintained anymore?

the maintainer is looking for others who can take over the support and maintenance https://github.com/machulav/ec2-github-runner/discussions/172  i proposed to help, will do my best within limited scope, but we need more people 💯 . let's see what the decision will be. not many chime in so far though 😓 because nobody sees the announcement, it should be inserted to the readme header.


Fixes to Godot 3.1

Made some changes so it's now usable in Godot 3.1
Only downside to my changes is I commented the opacity change since it is now controlled by the modulate variable.

Looks good, though there are more changes that could be done; replacing methods with properties (e.g. set_position with position=) and adding types everywhere, just to mention a few.
Going to merge it anyways. Thanks!


Update SourceMap to use find_opt

D9967600

i think we'll need to update sourcemaps.opam to require ocaml 4.05. looks like I need to update to opam 2.0 format first...

actually we can do that later, this isn't published to opam yet


Disable JS variable renaming when sourcemaps are enabled

Disable variable renaming when JS sourcemaps are enabled in order to ease JS debugging with browser debugger (e.g. Chrome Console).

We can't do this because if we aren't mangling in dev we should just disable uglify in dev entirely. I don't want to disable uglify in dev right now. As such: the underlying issue is the sourcemaps that need to get fixed.


⚠ GitLab Server has degraded performance

In 6c2be24, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11667 ms

Resolved: GitLab Server performance has improved in ad51521.


InvalidArgumentError while training, when using a different dataset.

I used generate_vocab.py to create vocab.sources.txt and vocab.targets.txt files. Then when I run train I get the following error -
The vocab size for vocab.sources.txt is 135.
I use the example_configs/nmt_small.yml model.
This is the error I get: -
InvalidArgumentError (see above for traceback): Assign requires shapes of both tensors to match. lhs shape= [138,128] rhs shape= [23,128]
[[Node: save/Assign_70 = Assign[T=DT_FLOAT, _class=["loc:@model/att_seq2seq/encode/source_embedding/W"], use_locking=true, validate_shape=true, _device="/job:localhost/replica:0/task:0/cpu:0"](model/att_seq2seq/encode/source_embedding/W, save/RestoreV2_70)]]
Does some one know why is this happening?

I think you might not be providing correct vocab file path to the model. Model is expecting size of 138 for the vocab.
138 = 135 + 3
vocab size = current vocab size + 3 special tokens to be added (unk,start,end)
and you are providing it with a size of 20+3

I cleared the previously learned model i.e. removed the $MODEL_DIR directory; before training on new data.
And the error was resolved.


'dev' is not recognized as an internal or external command,

The Quickstart instructions have the following step:
poetry run dev
Output:
PS C:\users\danie\chatgpt-retrieval-plugin> poetry run dev
'dev' is not recognized as an internal or external command,
operable program or batch file.

Note that poetry run start does seem to work.

This project moves fast!  I noticed that I didn't even have the local-server directory locally, so I did a git pull, noticed relevant files were committed less than an hour ago, and now it's working.


pkg/encoding/csv puts extra newline in encode?

bug.cue:
import "encoding/csv"

foo: csv.Encode([["foo","bar"],["ban","baz"]])

$ cue eval bug.cue
foo: """
        foo,bar
        ban,baz

        """

Perhaps this is the intended behavior (csv is older than I am and has more history than I am knowledgeable of) but since baz is already proceeded by a newline character should reasonably work fine as input to commands like cat it strikes me as odd that the additional one is there and perhaps a bug.

hex.Dump seems to have a similar quirk:
bug.cue:
import "encoding/hex"

foo: hex.Dump('\xff')

$ cue eval bug.cue
foo: """
      00000000  ff                                                |.|

        """

but unlike csv it does not prepend last newline character with in the encapsulated """ with spaces.

I guess this extra newline character is not needed or intended. Since the POSIX spec specifies the following for a line and text file.

A sequence of zero or more non-  characters plus a terminating  character.
https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap03.html#tag_03_206
A file that contains characters organized into zero or more lines.
https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap03.html#tag_03_403

The extra newlines should not be there, indeed.

Actually, this is working as intended. There is actually only one newline in this string. Looking at the spec, this is clear from the example, but it is not clearly worded. Will change the spec to be more clear.
Note that this multiline string semantics is identical to that of Swift. A language guide (user-friendly spec) would include some text along the lines Swift does:
https://docs.swift.org/swift-book/LanguageGuide/StringsAndCharacters.html

This issue has been migrated to https://github.com/cue-lang/cue/issues/179.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Stepping outside the box with transactions

I unfortunately need to create a user in the user table and store a password in a user_auth table.
This needs to happen in the UserController#create method and should be transactional so it all happens or nothing happens just like if my password was in my user table and I was simply creating a user.
I'm having a hard time digging through the code to see where I should be customizing for this functionality.
Thank you for your help in advance.

Two options:

Use a callback on the model such as after_save and update the second table. It should all run in a transaction, and you could raise an exception which would trigger the rollback in the OperationsProcessor and return an appropriate error response.

To do it all with resources you could add a second operation. Unfortunately there's not yet a good way to create extra operations.  I'm planning to rework this section of code to allow more flexibility for this sort of thing.

To work around this I think you could modify a copy of process_request and use it to override your controller action. In there add a second operation to @request. It's a hack, but should work.
I'd try option 1 first.

@lgebhardt How do I handle the extra fields on the model / resource? If the user table doesn't have a password_digest field and the user model doesn't implement has_secure_password for example then how can I tell the user resource to accept password and password_confirmation fields but then split them out of the params before save and use them to update the second table etc. This would all normally happen in the controller create method and before_create action, etc.

This might work. Declare the attributes on the resource and implement setters on the model that store the values in instance variables on the model. They would then be available for you to use in your callbacks, but wouldn't be stored in the model's table.

Hey @lgebhardt and @boobooninja, in regards to this would it be possible to do something like this?
#...
around_create :handle_update_auth

def handle_update_auth
  ActiveRecord::Base.transaction do
    yield
    # your code to update auth_table here
  end
end
#...

For anyone referencing this later, what @patrickgordon mentioned works (I have done it before). The only tricky part is: when using a database that does not support nested transactions (such as MySQL), you will have to prevent JR from wrapping the whole operation in a transaction otherwise the one you define in the callback will do nothing.
One way to do this is to override process_operations in your base controller to call a new transactional? method to determine if it should wrap the operation in a transaction. Then you can define this method on any controllers where you need to do this e.g.,
class SomeController < ApplicationController
  def transactional?
    context[:action] != :create
  end
end


the phone on the right keeps crashing

The create a form element keeps crashing on me. I cant type past line 48 and the phone on the left keeps crashing. I have cleared my browser and re-started my computer, but it keeps crashing. help please?

Hi @VarCI-bot
The phone on the right  keeps saying
Chrome detected unusual code on this page and blocked it to protect your personal information (for example, passwords, phone numbers, and credit cards).
and no one on the help chat room knows why either.

i have the same problem

Opera isn't supported here

i'm not using Opera i am using chrome @barkinet

@coolwolf21 It's a known issue with Chrome, switch to another browser for now.

Hi, I had the same problem and resolved it by clearing my recent browser history. :)

thanks for the report, and we confirm that this is a known issue, check #13727
Closing as a duplicate.


Overhaul event media tests

Overhaul event media tests to use fixtures instead of having custom setup per tests. This allows all tests to use the image loading behavior, in a more streamlined way, bringing uniformity to the tests and increasing test coverage.

Codecov Report

Merging #716 (6a704f0) into main (cd88ea2) will increase coverage by 0.22%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #716      +/-   ##
==========================================
+ Coverage   91.95%   92.18%   +0.22%
==========================================
  Files          19       19
  Lines        1741     1727      -14
==========================================
- Hits         1601     1592       -9
+ Misses        140      135       -5

Files Changed
Coverage Δ

google_nest_sdm/__init__.py
100.00% <ø> (ø)

google_nest_sdm/auth.py
96.19% <ø> (ø)

google_nest_sdm/event_media.py
95.32% <ø> (+1.03%)
:arrow_up:

google_nest_sdm/google_nest_api.py
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Screenshot

Add screenshot in the readme file :)

Yep, not release 1.0.0 yet so it will come ;)

:thumbsup:


Database and API Review Ticket - Drew Reed

Individual: CPT Grading
Link: click here
The final reflection on all of the submissions so far is found at the end, in its own section.
CPT Project: Videos
IMPORTANT: I made these videos on Saturday, January 21st before Mr. Mortensen announced on Slack that we were expected to include voiceover in the videos. Instead, I included captions that explain what I'm doing. I did these videos early in the weekend because I knew I would be too busy to work on them at all on Sunday. I do not have time to record new voiceover, re-edit both of my videos and upload them. The captions should be more than enough to follow along.
Database/Model Hack Video: click here
In the video, I go through the process of showing the SQLite table being built through initEvents() using the step into button. I then show myself creating a new row and then using step into to watch it be added to the table.
API/Control: click here
In the video, I show the backend endpoint of the Events API, using Postman to GET (Read) and POST (Create) in my database/API, my custom validation conditions, and the frontend endpoint of the Events API on our group's website on the Events page.
To clarify, the table on the site doesn't read the JSON data from the API, as, if it did, the table would only appear on my machine while the local server is up. The table on the page is capable of reading any amount of (correctly-formatted) JSON, meaning that I will be able to use fetch later to use the API once's it's up.

1 API
1 DB
.9 CPT


Provider better indication and workflow for artifacts waiting for approval

The Service view shows an artifact to be deployed into a stage even when it is waiting for an approval. This is misleading and does not reflect the actual state. There is also not a good workflow for a user to get to approve or reject the artifact at that point. Either allow approval/reject in that view as described in #2135 or provide a link to the environment view.
The following screenshot should highlight the current implementation and how to make it better:

This is related to #2135

In the community meeting from 6th August, we discussed to add a confirmation dialog to the action of accepting/declining an approval. I propose to not add the additional click, but rather show the user the artifact version that is going to be overridden:

In the community meeting from 6th August, we discussed to add a confirmation dialog to the action of accepting/declining an approval. I propose to not add the additional click, but rather show the user the artifact version that is going to be overridden:

I like that idea. Showing that information, already indicates what will happen, when you click on approve button. So I also think we don't need an additional confirmation dialog for that.

Due to the fact, that we know the evaluation result (score) we can show the user more details:

As discussed with @johannes-b, I provide now the Score from the last evaluation next to the deployable artifact and on hover you see details from that evaluation.


Add instructions for running the unit tests

The unit tests are massively out of date and haven't been working since I ditched the gulp build plan, somewhere around v1. If you'd like to work on them, I'm planning to update to angular v6 soon and use the angular CLI, and I can either move them across to that test system or use it as a chance for a fresh start.


Support lightgbm decision trees

It looks like this is a combined categorical split and lightgbm thing.  I really think we should separate those two.  We can do the lightgbm merge before we can figure out the categorical split thing I would say, right?

It looks like this is a combined categorical split and lightgbm thing.  I really think we should separate those two.  We can do the lightgbm merge before we can figure out the categorical split thing I would say, right?

also it looks like you will have to re-base.  Note that I have changed the testing/dtreeviz directory name to avoid  module name issues.

also it looks like you will have to re-base.  Note that I have changed the testing/dtreeviz directory name to avoid  module name issues.

I see that we have some conflicts from notebooks....hard to merge on them. What do you suggest to do with them ?
Since the notebooks are not related to lightgbm, maybe we should accept the notebooks from master.

Yep, i think that's what I did in my test branch.


Feedback on week 6 project progress

Subject: feedback on week 6 project progress
See mid-v1.1 progress guide for more details of the activities mentioned below.
Team progress
Progress noted (well done! :+1:):

Team org and repo set up with the correct names.
PR created from team repo to AB-4.

Suggestions (to do soon):
None
Individual progress of @afiqlattif:
Progress noted (well done! :+1:):

Team repo forked to your GitHub account.
PRs created to update documents.

Suggestions (to do soon):

Create PRs to do a local change to the Java code.

Individual progress of @scrubbius:
Progress noted (well done! :+1:):
None
Suggestions (to do soon):

Fork team repo to your GitHub account.
Create PRs for your share of updating documents.
Create PRs to do a local change to the Java code.

Individual progress of @kratious:
Progress noted (well done! :+1:):

PRs created to update documents.

Suggestions (to do soon):

Fork team repo to your GitHub account.
Create PRs to do a local change to the Java code.

Individual progress of @sharan8:
Progress noted (well done! :+1:):

Team repo forked to your GitHub account.
PRs created to update documents.

Suggestions (to do soon):

Create PRs to do a local change to the Java code.

Individual progress of @imarbles:
Progress noted (well done! :+1:):

PRs created to update documents.

Suggestions (to do soon):

Fork team repo to your GitHub account.
Create PRs to do a local change to the Java code.

Tutor: @joanneong
Note: the above observation was done by the CS2103-feedback-bot and covers changes up to 18/9/2018 only. If you think the above observation is incorrect, please let us know by replying in this thread. Please include links to relevant PRs/comments in your response.

Hi! The bot detected that I did not:

Fork team repo to your GitHub account.

Which I did here: https://github.com/kratious/main
On Wed, 19 Sep 2018, 09:11 asiddharth<EMAIL_ADDRESS>wrote:

Subject: feedback on week 6 project progress
See mid-v1.1 progress guide
https://nus-cs2103-ay1819s1.github.io/cs2103-website/admin/project-w06-mid-v11.html
for more details of the activities mentioned below.
Team progress Progress noted (well done! 👍):

Team org and repo set up with the correct names.
PR created from team repo to AB-4
https://github.com/nus-cs2103-AY1819S1/addressbook-level4.

Suggestions (to do soon):
None
Individual progress of @afiqlattif https://github.com/afiqlattif: Progress
noted (well done! 👍):

Team repo forked to your GitHub account.
PRs created to update documents.

Suggestions (to do soon):

Create PRs to do a local change to the Java code.

Individual progress of @Scrubbius https://github.com/Scrubbius: Progress
noted (well done! 👍):
None
Suggestions (to do soon):

Fork team repo to your GitHub account.
Create PRs for your share of updating documents.
Create PRs to do a local change to the Java code.

Individual progress of @Kratious https://github.com/Kratious: Progress
noted (well done! 👍):

PRs created to update documents.

Suggestions (to do soon):

Fork team repo to your GitHub account.
Create PRs to do a local change to the Java code.

Individual progress of @sharan8 https://github.com/sharan8: Progress
noted (well done! 👍):

Team repo forked to your GitHub account.
PRs created to update documents.

Suggestions (to do soon):

Create PRs to do a local change to the Java code.

Individual progress of @iMarbles https://github.com/iMarbles: Progress
noted (well done! 👍):

PRs created to update documents.

Suggestions (to do soon):

Fork team repo to your GitHub account.
Create PRs to do a local change to the Java code.

Tutor: @joanneong https://github.com/joanneong
Note: the above observation was done by the CS2103-feedback-bot and covers
changes up to 18/9/2018 only. If you think the above observation is
incorrect, please let us know by replying in this thread. Please include
links to relevant PRs/comments in your response.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CS2103-AY1819S1-W16-2/main/issues/56, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AGZ0XM8MavCubrOO2RT59H9m5BXkR5Yoks5ucZmxgaJpZM4WvO7t
.


Split library into separate script

Allows library to be updated by git independently of presentations
Makes presentation files more clean

Manually merged


tag name doesn't work with foriegn keys

e.g.
class Onion(dexml.Model):
price = fields.Integer(tagname="price")
class VegetableBasket(dexml.Model):
purple_onions = fields.Model(Onion, tagname="purple_onion")
a = serializers.Onion(price=10)
b = serializers.VegetableBasket(purple_onions = a)
b.render()
'10'
is this by design?

Thanks for reaching out. Unfortunately I am no longer actively maintaining this project, so I'm going to move it into archive mode:
https://rfk.id.au/blog/entry/archiving-open-source-projects/


Avoid unnecessary pip package installations when editing and rebuilding the source code

Since the Dockerfile.template file has COPY ./app/ /usr/src/app/ before the pip install instructions, each time one edits the code and rebuilds, all pip packages are reinstalled.
This takes some time, especially when working in local mode.
Thus, the Dockerfile statement
COPY ./app/ /usr/src/app/
Should be moved to after the pip install instructions
Fixing this will make it more convenient for people toying with and evaluating Balena to tinker with the code as you won't waste time reinstalling the same pip packages.

Pull request: https://github.com/balena-io-projects/balena-cam/pull/54

Pull request merged. Issue resolved.


left rotation and Huffman decoding added

solved left rotation in python closes #9

@subhangi2731 add proper comments and use folder structure as in the Gitter channel.


Pool{Type}Array has no method 'has'

Godot version: 3.2.3.rc1
OS/device including version: Linux 5.7.9-1-MANJARO
Issue description: Unlike Array, PoolStringArray, PoolIntArray and others doesn't have a method to check if an element is present.
Steps to reproduce / Minimal reproduction project:
var _array: Array = ['a', 'b', 'c']
_array.has('b') # true
_array.has('z') # false

var _array: PoolStringArray = ['a', 'b', 'c']
_array.has('b') # The method "has" isn't declared on base "PoolStringArray"

There are two workarounds to solve this:
var _array: PoolStringArray = ['a', 'b', 'c']
(_array as Array).has('b') # true
(_array as Array).has('z') # false

static func has_string(src: PoolStringArray, find: String) -> bool:
	for item in src:
		if item != find:
			continue

		return true

	return false

...

var _array: PoolStringArray = ['a', 'b', 'c']
has_string(_array, 'b') # true
has_string(_array, 'z') # false

This was implemented by https://github.com/godotengine/godot/pull/32144, but I'm not sure if it's worth remaking for the 3.2 branch.

Fixed by #60856


bug：using wechat, baidu, etc, login fail: There was a problem signing you in..

There was a problem signing you in..
The username may only contain alphanumeric characters, underlines or hyphens, cannot have consecutive hyphens or underlines, and cannot begin or end with a hyphen or underline.

Have you tried 1.134.0?
The same problem has been fix, See https://github.com/casdoor/casdoor/issues/1223

Yes, no problem when using v1.134.0, thanks.


Tweak example app so Plotly plots fill their cards

I talked this over with @cpsievert and neither of us thinks this is the last word on how we'll support Plotly FigureWidgets having 100% height, but for this release, we can either choose between having this, or changing these examples to matplotlib.
BTW I think we'll also need a dedicated doc page on how to use Plotly with Shiny for Python. I think there's both enough interest and enough hard-to-discover ways to do things to justify it.

In theory, yes, this is how the R binding does it
More generally, some more thought should be put into how output_widget() should better handle this for any widget via ipywidgets sizing API https://ipywidgets.readthedocs.io/en/stable/examples/Widget Layout.html


Fix two issues with loopProtect comment detection

Fixes two issues with loopProtect not detecting comments correctly. One caused to not look back correctly in single line comment detection and one caused it to not look at line 0 for the start of single line comments.
Closes #7072

LGTM :+1:


⚠ Secret Website 3 has degraded performance

In 223feea, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2390 ms

Resolved: Secret Website 3 performance has improved in 5218230.


fix: show install pwa btn only when it's installable

We set global msg variable at beforeinstallprompt event. However beforeinstallprompt isn't fired if the user already installed the app, and in that case, clicking install btn causes msg.prompt() call with nulled msg. I think that caused #53
This PR fixes the above situation. With this change we show the Install PWA button only when the app is ready to install, and hide it otherwise.
closes #53

@kt3k Great addition, thanks


@conda-forge-admin please add user @liZe

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @liZe in conda-forge/pyphen-feedstock#13.


sample project?

hi,
could you please create a sample project of howto use this component?
thanks in advance.

we are curently using it in https://github.com/fabric8-ui/fabric8-ui and there is a demo if you run npm start

Are you looking for how to use an Angular component or what this one does?


Please provide signed VAA bytes for stuck Wormhole transaction.

Hi there,
Would appreciate some help, specially from leoluk, as I have seen him help and provide correct signed VAA bytes to others with similar issue.
I transferred wLUNA through the V1 Wormhole Bridge from Solana to Ethereum.
It was successful on the Solana side, but I accidentally rejected the transaction on the Ethereum side.
The wLUNA never got back to my Solana wallet and is stuck in the Wormhole.
The transaction:
https://explorer.solana.com/tx/2wk6jHYbzzKTmkriJRrzjMX62fyDfBFruxn48iRTVfdqJN7dZ3eHfm2nLoDtnoZZHtQcfD4uyKYESBHu8rbcvL4y
I have seen that you can "Manually submit the signed VAA bytes to the submitVAA method on the Wormhole contract".
But I don't know which signed VAA bytes to submit to the contract.
Could you please provide the correct signed VAA bytes for my transaction?
I would greatly appreciate your help.
Thank you.
P.S.:
I have tried https://www.interstellar.tk/
but unfortunately, that does not work for me, as it gives me this log - "The target address doesn't match connected wallet, abort".

for some reason the target address on the Solana transaction starts with 0x19c5.. instead of 0x019c5.. apart from that, the address is identical

Is there a way to get around this?

RESOLVED
Thank you @bagz11 , I wrote to @hx8888979 and they gave me the VAA for my transaction. I was then able to restore the transfer by submitting the VAA into the contract and afterwards I accepted the transaction in Metamask. It went through.
I am closing this issue.


File Time/Date Stamp update

Description of the new feature / enhancement
Allow users to much more simply change the time/date stamp on files
Scenario when this would be used?
I occasionally receive files in email, and by the time I move them into my file system, the time/date of their arrival has long gone past. So I can re-locate the email that contained them, I like to change the time/date stamp on the file to match the email.
Supporting information
This feature is supported by a variety of other tools, but to have it included in a tool set I already use would save time.
Also, knowing how detailed Power Toys is being maintained, I can see it taking much fewer keypresses to use.

The bot identified two other issues with a similar request.
Looking them over, I would tend to agree...but what is being done with either of them.
Or a third one that is also similar: 22954


Initialize Azure.ResourceManager.DataProtection

Corresponding track 1 PR: https://github.com/Azure/azure-sdk-for-net/pull/31590
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

/azp run prepare-pipelines

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.DataProtection


cannot open authentication link in browser

VSCode Version: 1.40.0
OS Version: Ubuntu 18.04

Steps to Reproduce:

In command, I typed ">vs online: sign in", and enter.

A notification indicated.

After a while, vs code seems to try to open link in browser.

But the page in browser is HTML code opened from local file, NOT a web page.

This issue occurs not only when signing into vscode online, but also whenever vscode try to open an external link. A cached page file with HTML source file is always opened.

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

@githubfzq Can you please reopen this issue on the VSOnline extension here: https://github.com/MicrosoftDocs/vsonline Thank you!

@RMacfarlane, this looks like an issue with VS Code rather than the VS Online extension. VS Online uses the vscode.env.openExternal API to open urls in the browser. From the original issue description, this is broader than just the VS Online extension (emphasis mine):

This issue occurs not only when signing into vscode online, but also whenever vscode try to open an external link. A cached page file with HTML source file is always opened.


No support for static rives

I've implemented the rive-react package and am loading rives with the useRive hook. When downloaded and tested my configuration with the poison-loader.riv file from this repo it worked great. When I tried to use one of my own (that has no animations) it throws an error saying Artboard has no animations.
I'd like to be able to load and use all my assets (animated and otherwise) into the app just using the rive package. Is this possible or will this be possible in the future?
Seems like this repo should be able to support rive files which are static and have no animations. Thanks!

Hey @kadenBeckstead, we currently don't supporting playing rive files without animations. The simple workaround for this is to add an animation to the artboard with no keyframes and export your file again.


Enhance curve config

Rename extrapolator properties to be consistent with other locations.
Allow curve to be used for multiple discount currencies.
Javadoc and tidy.
Add tests.

Reviewed post-merge.


Adding_some_spanish_radios

Hi Stefan ,  thanks for repairing  the previous broken links and the VLC blocked  (user agent issue)  .
Now from  my vscode , playing RUN , it works flawesly . ,

but I do not know why , it does not work from my external terminal ???

In this PR , let me add some spanish radios .
Cheers,

it does not work from my external terminal

emerge PyQt5

Thank you for the Spanish radio stations.
I implemented tabs, now that we have three different categories.

You need "python3_12" added to your "PYTHON_TARGETS" in "/etc/portage/make.conf".
You can see which USE flags were enabled for the installed version in the output of eix -se PyQt5.

Hi , initially , I had that condition and not working .  (I do not understand that (fallback , meaning ) .
eselect python list
Available Python interpreters, in order of preference:    (then I guess it was selected the 3.11 !!!!! )
[1]   python3.11
[2]   python3.10
[3]   python3.12 (fallback)
Checking USE flag,  seems installed with PYTHON_TARGETS="python3_12

Normally I used different command ,  this ,

Anyway , in  in "/etc/portage/make.conf". I do not have that line .
But I tried to send
sudo eselect python set 3 , and now , I guess that the PYTHON interpreter activated is [1]

And it works !!!!!!!
I thought that the system usually was activating the latest Python version , same as Kernel list , but it seems , not...
Great , eureka !!!  Now  it works ,  thanks again , !!!!


Add Article and Methodology page types

NWP-258 and NWP-281

Adds a methodologies app with methodology page model and separate fields for the background data. Note that I chose a simple output in the template (rather than the table)
Adda an articles app with: article series and article page models
Moves the bulletin StreamBlocks to core app
Moves the topic <-> page relation to the taxonomy app

Next step is to extract the common elements to abstract models for DRY, but IMHO this is enough for this POC

@MebinAbraham

Articles/Methodology doesn't seem to add to a bundle correctly. If you click "Add to bundle" from Article/Methodology page, then go to the bundle, you can see it has tried to link it but seems invalid. If you click Save on the bundle, the pages get removed from the bundle.

Sorted in 3ed79f4


fix: use custom body analyzer

see https://gravitee.atlassian.net/browse/APIM-851
see https://github.com/gravitee-io/issues/issues/8470

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 4.0.1-fix-log-body-template-4-x-SNAPSHOT
download it from Artifactory here

:tada: This PR is included in version 4.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


NIFI-11778 refactored Groovy tests in nifi-elasticsearch-restapi-processors to Java (and JUnit 5)

Summary
NIFI-11778 / NIFI-9290
Refactored all the Groovy unit tests in nifi-elasticsearch-restapi-processors to Java.
Along with the refactor, the building of the individual JSON documents were removed and replaced with files which are now read in thereby reducing the size of the unit tests and increasing the readability of the unit tests.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

@exceptionfactory I see 2 of the builds failed but the errors are not related to the code I checked in.

@dan-s1 I re-triggered the Checks, definitely looks like ES related assertion failures now (originally you were right that the tests for for some unrelated transient issue)
Error:  Failures:
Error:    PutElasticsearchRecordTest.testDateTimeFormatting:600 org.opentest4j.AssertionFailedError: expected: <2> but was: <0>
Error:    PutElasticsearchRecordTest.testRecordPathFeatures:337 org.opentest4j.AssertionFailedError: expected: <1> but was: <0>
Error:  Tests run: 124, Failures: 2, Errors: 0, Skipped: 0
Error:  Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:3.0.0:test (default-test) on project nifi-elasticsearch-restapi-processors: There are test failures.
Error:
Error:  Please refer to /home/runner/work/nifi/nifi/nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-restapi-processors/target/surefire-reports for the individual test results.

@dan-s1 I re-triggered the Checks, definitely looks like ES related assertion failures now (originally you were right that the tests for for some unrelated transient issue)
Error:  Failures:
Error:    PutElasticsearchRecordTest.testDateTimeFormatting:600 org.opentest4j.AssertionFailedError: expected: <2> but was: <0>
Error:    PutElasticsearchRecordTest.testRecordPathFeatures:337 org.opentest4j.AssertionFailedError: expected: <1> but was: <0>
Error:  Tests run: 124, Failures: 2, Errors: 0, Skipped: 0
Error:  Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:3.0.0:test (default-test) on project nifi-elasticsearch-restapi-processors: There are test failures.
Error:
Error:  Please refer to /home/runner/work/nifi/nifi/nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-restapi-processors/target/surefire-reports for the individual test results.

@ChrisSamo632 Do you have any suggestions how to reproduce these failures? On my local box all unit tests are passing. I thought it might be a locale issue and I tried setting the locale to "FR" but the unit test still passed.

NM I got it to fail by using this
TimeZone.setDefault(TimeZone.getTimeZone("Europe/Paris"));

@ChrisSamo632 Would it be okay to fix the test errors by placing in setUpBeforeClass
TimeZone.setDefault(TimeZone.getTimeZone("UTC"));
?

@ChrisSamo632 Sorry by accident closed this PR and the reopened. Would it be okay to fix the test errors by placing in setUpBeforeClass
TimeZone.setDefault(TimeZone.getTimeZone("UTC")); ?

I'm not sure this sounds like the right way to fix this - the Checks run against multiple timezones for good reason, we want to be sure that processors can handle changes in locale. Having to set the tests to use a specific timezone to pass, seems like we might be hiding a potential issue either in the tests or the processor code itself.
Unfortunately, handling multiple timezones isn't my strong suite, although I'll try to take a look. I wonder whether @exceptionfactory or @markap14 might be able to provide some guidance on this one?

@ChrisSamo632 Sorry by accident closed this PR and the reopened. Would it be okay to fix the test errors by placing in setUpBeforeClass
TimeZone.setDefault(TimeZone.getTimeZone("UTC")); ?

I'm not sure this sounds like the right way to fix this - the Checks run against multiple timezones for good reason, we want to be sure that processors can handle changes in locale. Having to set the tests to use a specific timezone to pass, seems like we might be hiding a potential issue either in the tests or the processor code itself.
Unfortunately, handling multiple timezones isn't my strong suite, although I'll try to take a look. I wonder whether @exceptionfactory or @markap14 might be able to provide some guidance on this one?

I will try to take a closer look at the tests soon, but I agree, changing the default timezone in test setup code is not the right approach.
All of the automated builds run with different timezones to ensure expected behavior in various locations, so all tests must pass without changing the system timezone.
In general, the issue means that there is some timezone conversion happening, which could point to a runtime issue, or simply an issue with the sample data used for testing. I'm not sure exactly why this has not surfaced earlier, so it points to an issue with the test-related changes.

@ChrisSamo632 Checking in to see if my latest changes are okay. Thanks!

@ChrisSamo632 Checking in to see if my latest changes are okay. Thanks!

@dan-s1 I've been away, but hoping to take a look at your latest changes soon/this week.
Note that you've now got some conflicts after #7441 was recently merged - couple of new tests to migrate over for the updated processors

@ChrisSamo632 Do I manually cut and paste those changes as in my branch those Groovy files have been deleted?

@ChrisSamo632 Do I manually cut and paste those changes to my branch since those Groovy files have been deleted in my branch?

I'd guess so, and make them work for the Java setup rather than Groovy

@exceptionfactory Just double checking before I proceed that the way to resolve the conflicts is for me to cut and paste the changes from 7441 to my code. Thanks!

@ChrisSamo632 I am trying to implement the changes from #7441 For testBatchingAndErrorRelationship in PutElasticsearchJsonTest.java  for the equivalent line of  runner.assertTransferCount(PutElasticsearchJson.REL_SUCCESS, 4)  is failing as the transfer count is 6 and not 4. Please advise as I do not see any changes in PutElasticsearchJson.

@ChrisSamo632 NM I forgot to account for the changes to src/test/resources/common/sampleErrorResponse.json

@ChrisSamo632 I have made the changes you had requested. I also incorporated the changes from https://github.com/apache/nifi/pull/744. I broke up PutElasticsearchRecordTest#testErrorRelationship into five separate tests

testFailedRecordsOutput
testFailedRecordsOutputGroupedByErrorType
testNotFoundResponsesTreatedAsFailedRecords
testNotFoundFailedRecordsGroupedAsErrorType
testErrorsLoggedWithoutErrorRelationship

and I refactored out the common code into the method testErrorRelationship

@dan-s1 I'll take a look shortly hopefully, but note that you've still got some conflicts with main, you'll need to rebase against (or merge in) main

@ChrisSamo632 I am not sure I can do anything regarding these conflicts. I already incorporated the changes of https://github.com/apache/nifi/pull/7441 for PutElasticsearchRecord and other related changes.  As for the groovy files, naturally there will be a conflict as my branch deleted them and replaced them with Java files.

@ChrisSamo632 I am not sure I can do anything regarding these conflicts. I already incorporated the changes of #7441 for PutElasticsearchRecord and other related changes. As for the groovy files, naturally there will be a conflict as my branch deleted them and replaced them with Java files.

@dan-s1 You will need to rebase your PR and then force-push to this feature branch to resolve the conflicts. If you have already incorporated the changes, then during the rebase process, you should be able to just mark the groovy files as deleted, but the change to PutElasticsearchRecord needs to be incorporated.

@exceptionfactory Thanks for the clarification. I have made those changes and pushed.


Alphabetize requirements.txt

Summary of Changes
Alphabetize requirements.txt
Tests
None, no tests needed.
Documentation
None, no documentation required.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes make test
[x] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

Codecov Report

Merging #36 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #36   +/-   ##
=======================================
  Coverage   77.66%   77.66%
=======================================
  Files          13       13
  Lines         394      394
  Branches       38       38
=======================================
  Hits          306      306
  Misses         74       74
  Partials       14       14

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 351920f...5078637. Read the comment docs.

This is a very small commit, but this will help me merging changes in from Amundsen upstream to our local repos in future.

thanks @worldwise001


Wrong path in package.xml

If you install the Package project you get a package.xml file in the root of the solution that includes a file and folder that points to the dll and the app plugins folder of the package. Automatically replaces the name, and is a very nice touch!
https://github.com/umbraco/Package.Templates/blob/master/ProjectTemplates/UmbracoPackage.1/package.xml
Problem I've found though is that the dll is not exisiting when I build, it is found in the .website folder instead, so fx last time I had to edit from:
  <files>
    <folder path="src/Package.Workshop/App_Plugins/Package.Workshop" orgPath="App_Plugins/Package.Workshop" />
    <file path=".src/Package.Workshop/bin/release/Package.Workshop.dll" orgPath="bin/Package.Workshop.dll" />
  </files>

To:
  <files>
    <folder path=".website/App_Plugins/Package.Workshop" orgPath="App_Plugins/Package.Workshop" />
    <file path=".website/bin/Package.Workshop.dll" orgPath="bin/Package.Workshop.dll" />
  </files>

Are we expecting the dll to exist in the core project? Guess I may have needed to build as release instead of debug for that?
Maybe it's easier to change it to the website folder instead as we'd expect the testsite to have the correct files and be tested before packing it up, right?

ideally we want it to still reference the package not the site, because the .website folder isn't source controlled, and you have the option not to create a .website when you create your template.
I think there is a %configuraton% style variable (i will have to look it up) that you can replace the %release% with.

PR #17 Fixes this ?

Yep, will close this 🙂


Generate more than 25 frames with SVD

Is there a way to alter refine_video.py to generate longer sequences than 25 frames?

Hi, currently SVD is limited to generating 25-frame videos.


F-Droid can't build

> Task :app:assembleRelease

FAILURE: Build failed with an exception.

* What went wrong:
Configuration cache problems found in this build.

4 problems were found storing the configuration cache, 2 of which seem unique.
- Task `:app:buildKotlinToolingMetadata` of type `org.jetbrains.kotlin.gradle.tooling.BuildKotlinToolingMetadataTask$FromKotlinExtension`: invocation of 'Task.project' at execution time is unsupported.
  See https://docs.gradle.org/7.4/userguide/configuration_cache.html#config_cache:requirements:use_project_during_execution
- Task `:app:collectDependencies` of type `com.mikepenz.aboutlibraries.plugin.AboutLibrariesCollectorTask`: invocation of 'Task.project' at execution time is unsupported.
  See https://docs.gradle.org/7.4/userguide/configuration_cache.html#config_cache:requirements:use_project_during_execution

See the complete report at file:///home/vagrant/build/com.cliffracertech.soundaura/build/reports/configuration-cache/cd5i9ei2v02kw3qqp2uwsq0d1/7cdob8p9t9g6koghep7ungayk/configuration-cache-report.html
> Invocation of 'Task.project' by task ':app:buildKotlinToolingMetadata' at execution time is unsupported.
> Invocation of 'Task.project' by task ':app:collectDependencies' at execution time is unsupported.

Ref: https://gitlab.com/fdroid/fdroiddata/-/jobs/3642651400#L517
/LE: https://gitlab.com/fdroid/fdroiddata/-/commit/80204172449ead0f06cf74c9a89928362ac1501a

I think this should have been fixed by commit 10f1037. Because the commit didn't change anything that should affect the output APK, I deleted the version tag associated with the new release and recreated it after commit 10f1037.


Default settings

Hello,
I edited plugins/redmine_git_hosting/init.rb before installing the plugin to change some default settings. Some of the change made to init.rb are applied to redmine settings but others don't.
It's ok with these one :
      :gitolite_ssh_private_key       => '/home/redmine/.ssh/redmine_gitolite_admin_id_rsa',
      :gitolite_ssh_public_key        => '/home/redmine/.ssh/redmine_gitolite_admin_id_rsa.pub',

For these settings it doesn't work :
:ssh_server_domain                => 'localhost:54376',

:http_server_subdir               => '/redmine',

The setting value in redmine remains default ('localhost', './') even if changed in init.rb.
However, after install these settings can be applied with the restore_default_settings task :
redmine@af6948ccbb15:/opt/redmine$ bundle exec rake redmine_git_hosting:restore_default_settings RAILS_ENV=production
[DEPRECATION] requiring "RMagick" is deprecated. Use "rmagick" instead
Reloading defaults from init.rb...
Changing 'gitolite_temp_dir' : /tmp/redmine_git_hosting/ => /opt/redmine/tmp/redmine_git_hosting
Changing 'gitolite_recycle_bin_expiration_time' : 24.0 => 24.0
Changing 'ssh_server_domain' : localhost => localhost:54376
Changing 'http_server_subdir' :  => /redmine
Committing changes ...
Success!
Done!

Hi!

I edited plugins/redmine_git_hosting/init.rb before installing the plugin to change some default settings. Some of the change made to init.rb are applied to redmine settings but others don't.

I think this comes from the DB migrations which are very... complex. Some settings might be overriden during the migration.
The best way to update settings after install is to run the rake task like you did :
redmine@af6948ccbb15:/opt/redmine$ bundle exec rake redmine_git_hosting:restore_default_settings RAILS_ENV=production


⚠ Bofrost.de has degraded performance

In 3fb3a70, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1938 ms

Resolved: Bofrost.de performance has improved in 8b228df after 9 minutes.


XCode build failure - "Code Sign error: No codesigning identities found" after latest release

Having had a functional build of a Cordova app for a couple of months, it has suddenly (in the past 5 days) started to fail with the error:
Code Sign error: No codesigning identities found: No codesigning identities (i.e. certificate and private key pairs) that match the provisioning profile specified in your build settings (“ExampleBrand News Dev”) were found.
It appears to succesfully create/configure/unlock the keychain and import the .p12 certificate, locating the valid identity.
However, when it comes to signing the build we have a failure.
Our configuration for Signing & Provisioning is as follows:
Override Using: File Contents
P12 Certificate File: (build variable)
P12 Password: (build variable)
Provisiong Profile File: (build variable)
Remove Profile After Build: false
I attach the build output for the Xcode build step:
xcode_task_output.txt
To clarify: No configuration or cert files were altered. Build is failing for unknown reason.

@Squiggle: I released a fix for this yesterday, let me know if you are still seeing issues. Sorry for the inconvenience.
Thanks,
Madhuri

@Squiggle: Let me know if you are still seeing issues with your builds.
Thanks,
Madhuri

Sorry for the slow response.
Yes, it was (silently) resolved after an update. Thanks so much!


🐛 Remove duplicate projects with different casings

What kind of change does this PR introduce?
Bug fix: there is URL duplication simply due to projects specified with unique casings. This PR fixes that.
(Is it a bug fix, feature, docs update, something else?)

[ ] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
What is the new behavior (if this is a feature change)?**

[ ] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

We noticed that when canonicalizing repos on our end we'd end up with "duplicate" scorecards for a particular repo definition - this looks like it would fix that to me! Thanks for looking into it.


“Go to Feed” menu option not working on all window sizes

This is on iOS TestFlight 6.1.6 (6138).
Using the “Go to Feed” option on an article in one of the smart feeds goes to a screen titled Timeline with no articles shown in it.
This happens on iPhone, and smaller window sizes on iPad (i.e. multitasking). When using the app fullscreen on iPad it works as expected.

Temporarily disabling Go to Feed on compact size classes. We’ll have to restore the feature in a follow-up.


Viewing a redacted snapshot should not erase IO data [WA-343]

Fixes regression introduced in WA-291/https://github.com/DataBiosphere/terra-ui/pull/2209
Repro steps in ticket: https://broadworkbench.atlassian.net/browse/WA-343
Tested locally that:

The redacted snapshot erasing behavior is gone
The original bug remains fixed, i.e. renaming a WDL input does not break launching workflows

Accepted suggestion, re-tested both relevant flows


retreive by user ID doesnt work

const searchParams = { filter: { field: "id", param: "in", value: [22, 33] } };
the above code in JS examples doesnt work.
neither does {user_id : xx}  or {id: xx}

@krisbaum74
according to documentation it looks good
https://developers.connectycube.com/js/authentication-and-users?id=retrieve-users-by-id
does it just return all users?

Is it permissions possibly? Documentation doesn't explain that well
On Mon, May 25, 2020, 5:38 PM Dave Pol Lomber<EMAIL_ADDRESS>wrote:

@krisbaum74 https://github.com/krisbaum74
according to documentation it looks good
https://developers.connectycube.com/js/authentication-and-users?id=retrieve-users-by-id
does it just return all users?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ConnectyCube/connectycube-web-samples/issues/22#issuecomment-633481131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANXOUQHS2T4GHYDUZOHR7SLRTI4ARANCNFSM4NIIMNIA
.

It returns none.
Try it. I have no idea how to retrieve by id
On Mon, May 25, 2020, 5:38 PM Dave Pol Lomber<EMAIL_ADDRESS>wrote:

@krisbaum74 https://github.com/krisbaum74
according to documentation it looks good
https://developers.connectycube.com/js/authentication-and-users?id=retrieve-users-by-id
does it just return all users?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ConnectyCube/connectycube-web-samples/issues/22#issuecomment-633481131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANXOUQHS2T4GHYDUZOHR7SLRTI4ARANCNFSM4NIIMNIA
.

@krisbaum74 could you share your settings here

Since there is no activity I’m going to close the issue. Please raise a separate issue in case of any other questions.


Axis in paste_to function

It seems that in the paste_to function axis are treated (x,y,z). However, the loaded NumPy arrays (both target and patch) are of shape (z,x,y).

Hi,
Thanks for pointing it out. Actually, in function paste_to, the 'x', 'y', and 'z' just mean the first, second, and third dimension of the 3D volume, i.e., tgt_data and tgt_seg, instead of the corresponding dimensions of the CT images. To avoid ambiguity, we have updated the variables' names to keep them consistent.


docs: update minimum supported Talos version to 1.4

Related to: https://github.com/siderolabs/omni/issues/434

Actually not sure what's going on with the docs now, as they don't build. Should I use gitbook instead?


issue #3346: Text Selection Example

Text selection example.

Just noticed this example recently underwent a complete re-write. I had a functional version working with the old example (just mis-aligned text issue I am working through). Is there significant or notable changes in the latest example?
Here is the commit I am referring to:
https://github.com/mozilla/pdf.js/commit/513a3d8c91c2948fcc95d2f1f297d787f45fa416

@apoku please open a separate issue for that an specify more information. Shortly, #5552 creates a better way to handle the painting of the entire page (including canvas, text layer and annotation). You may still use TextLayerBuilder (but it's really hard to tell since you did not provide any code you are refrerring to).

Thanks @yurydelendik, your short explanation is perfectly sufficient--I was just wanting to understand conceptually what that re-write encompassed.


GH-188 Make aec_miner unit tests more stable

Please refer to commit messages for details.
Supersedes PR #196.

I see 2 reviewers approved (actually 3!). I see CI passed. Merging.


chore: 🤖 Select table columns info styling changed

https://metakeen.atlassian.net/jira/software/projects/FEE/boards/10/backlog?selectedIssue=FEE-828

LGTM, just wondering why yarn.lock has been generated?


Recursive values give "no enclosing entity" error

Fable 0.3.26 fails to translate the following:
module Foo

let delay (f:unit -> unit) = f

let rec a = delay (fun () -> b())
and b = delay a

You get an error:

the value or member doesn't have an enclosing entity

The workaround is not to use recursive value definitions (recursive functions work just fine).

(This is not very urgent issue, but I wanted to log it with a small repro as I just encountered it with more fancy example :-))

Not urgent issues nicely reported with code to reproduce the problem is what I like 😉 Well, I like it even more when they also come with a PR 😄 but it's ok, this shouldn't be very difficult: recursive bindings are a bit tricky and so far I didn't pay too much attention to them. I'll have a look, thanks for the report!

Wow, I had a look at this and it's really complicated to fix as the F# does a lot of magic with recursive values. Taking about values... Do you think there's much value at adding this feature to Fable? We could just detect the warning 40 and throw a message saying that recursive value definitions are not supported in Fable (together with the location of the problem).

As the fix is complicated and I'm not sure many people will use recursive value definitions, for now I'm just disallowing it with a proper error message. I hope that's ok. If not, please feel free to reopen the issue and we can try to tackle it down later.

That sounds pretty reasonable :) recursive values are weird


enhancement: User defined vars

Description

Fixes #
Checklist

[x] The PR title has the correct prefix
[ ] PR is linked to the corresponding issue
[x] All commits are signed-off (git commit -s ...) to provide the DCO

TODO:
[ ] s/aliases/globals/g
[ ] condition should refer to globals map, not to an individual variable.
[ ] if possible to calculate globals only once inject a call to an appropriate function in Rego code

This PR introduces global vars for resource policies only.
I haven't updated the docs yet. The current resource policy example is simple and will not benefit from globals.
I suggest adding the second more complex example.

Let's not worry about the once-per-policy evaluation in this PR. Let's get the globals working (even if they are a bit inefficient) and document how to use them. We can come back to the once-per-policy optimization later.

@charithe I've updated the documentation. Please have a look.

@charithe I've resolved outstanding comments (except activations). Please have a look.


Facets with global scope

Hi.
Is it possible to create a facet query with global scope:
http://www.elasticsearch.org/guide/reference/api/search/facets/
I looked the documentation and the source code, but if this is supported I could not find it. I can probably modify the generated JSON DSL afterwards. Or perhaps create the feature myself if there is any interest?
Thank you.
Jussi

Hi Jussi,
I think you're right - I can't think of any way of doing that. I've never needed that, though.
Of course, you're more than welcome to create a patch adding this missing piece of functionality.
Cheers!
M.

Apparently the need for global facets numbers in our app does not exist at the moment.
I shall probably get back on this if other stuff permits. I think there will yet be a need for this feature..


build errors in person detection

$ cd tflu-apps/person-detection/
$ make clean
$ make
...
...
'/opt/riscv/bin/riscv32-unknown-elf-g++' -std=c++11 -DTF_LITE_STATIC_MEMORY -DNDEBUG -O3 -D__riscv__z -march=rv32imfcxpulpv2  -mfdiv -fdata-sections -ffunction-sections -MMD -MP -DTF_LITE_MCU_DEBUG_LOG -DTF_LITE_USE_GLOBAL_CMATH_FUNCTIONS -DTF_LITE_USE_GLOBAL_MAX -DTF_LITE_USE_GLOBAL_MIN -fpermissive -fno-rtti -fno-exceptions --std=gnu++11 -Itensorflow/lite/micro/quicklogic-arnold -I/home/tim/quicklogic/arnold-sdk/pulp-builder/install/include -I/home/tim/quicklogic/arnold-sdk/pulp-builder/hal/include -I/home/tim/quicklogic/arnold-sdk/pulp-builder/archi/include -I/home/tim/quicklogic/arnold-sdk/pulp-builder/install/include/archi/chips/arnold -I. -I./third_party/gemmlowp -I./third_party/flatbuffers/include -I./third_party/ruy  -c tensorflow/lite/micro/quicklogic-arnold/programFPGA.cc -o tensorflow/lite/micro/quicklogic-arnold/programFPGA.o
tensorflow/lite/micro/quicklogic-arnold/programFPGA.cc: In function 'void programFPGA(const char*)':
tensorflow/lite/micro/quicklogic-arnold/programFPGA.cc:62:7: error: 'sprintf' was not declared in this scope
       sprintf(rx_buffer,"LoAd %s\n",design);
       ^~~~~~~
tensorflow/lite/micro/quicklogic-arnold/programFPGA.cc:63:22: error: 'NULL' was not declared in this scope
       rt_event_alloc(NULL,4);
                      ^~~~
tensorflow/lite/micro/quicklogic-arnold/programFPGA.cc:63:22: note: suggested alternative: 'N_FLL'
       rt_event_alloc(NULL,4);
                      ^~~~
                      N_FLL
tensorflow/lite/micro/quicklogic-arnold/programFPGA.cc:63:7: error: 'rt_event_alloc' was not declared in this scope
       rt_event_alloc(NULL,4);
       ^~~~~~~~~~~~~~
tensorflow/lite/micro/quicklogic-arnold/programFPGA.cc:64:7: error: 'rt_uart_conf_t' was not declared in this scope
       rt_uart_conf_t conf;
       ^~~~~~~~~~~~~~
tensorflow/lite/micro/quicklogic-arnold/programFPGA.cc:64:7: note: suggested alternative: 'timer_conf_set'
       rt_uart_conf_t conf;
       ^~~~~~~~~~~~~~
       timer_conf_set

The tensorflow submodule is at 224fd9873557a2e2d8a8fd9f1c6d3e7d2128ccdc.
Should I check out master instead?

This looks like some of the pulp libraries have not been built.  Check that the pulp-builder directory and the
pulp-builder/install exists.   regardless you make want to rerun the arnold-sdk/patch/patch.bash script and then
run pulp-builder/scripts/build-runtime to create the appropriate libraries.

Thanks Greg, I'll look into this after my morning meetings.

Yep, it seems I had trouble with the pulp-builder build --- prettytable:
plpflags gen<EMAIL_ADDRESS>--config=platform=board  --config=**/rt/type=pulp-rt --output-dir=/home/tim/quicklogic/arnold-sdk/pulp-builder/build/pulp-rt/arnold --makefile=/home/tim/quicklogic/arnold-sdk/pulp-builder/build/pulp-rt/arnold/config.mk  --property=fc/archi  --property=pe/archi  --property=pulp_chip  --property=pulp_chip_family  --property=cluster/version  --property=host/archi  --property=fc_itc/version  --property=udma/hyper/version  --property=udma/cpi/version  --property=udma/i2c/version  --property=soc/fll/version  --property=udma/i2s/version  --property=udma/uart/version  --property=event_unit/version  --property=perf_counters  --property=fll/version  --property=soc/spi_master  --property=soc/apb_uart  --property=padframe/version  --property=udma/spim/version  --property=gpio/version  --property=udma/archi  --property=udma/version  --property=soc_eu/version  --property=compiler  --property=rtc/version  --property=udma/mram/version  --lib=rt  --lib=omp  --lib=rtio  --lib=bench
Traceback (most recent call last):
  File "/home/tim/quicklogic/arnold-sdk/pulp-builder/install/ws/bin/plpflags", line 8, in <module>
    import plptree
  File "/home/tim/quicklogic/arnold-sdk/pulp-builder/install/ws/python/plptree.py", line 25, in <module>
    import Regmap as regmap
  File "/home/tim/quicklogic/arnold-sdk/pulp-builder/install/ws/python/Regmap.py", line 21, in <module>
    from prettytable import PrettyTable
ModuleNotFoundError: No module named 'prettytable'

My install.log had this:
---------------install prettytable
Requirement already satisfied: prettytable in /home/tim/.local/lib/python3.8/site-packages (1.0.1)
Requirement already satisfied: setuptools in /usr/lib/python3/dist-packages (from prettytable) (45.2.0)
Requirement already satisfied: wcwidth in /usr/lib/python3/dist-packages (from prettytable) (0.1.8)

I'm guessing it's some issue with Python virtual environments, or Python 2 vs 3.   I'm somewhat new to this area.

I redid a few things, reinstalled python2 and pip, and now it seems to work.   At least, make gets me to a message ft2232: ftdi_stream_inout has failed, which probably means it's trying to talk to the board, which I haven't yet hooked up.   So in other words, it appears to be compiling correctly.

fetid_stream errors is indicative of not having the olimex JTAG connected or not mapped to the VM if you your using one.  Also issues with the udev stuff can cause this which results in the driver not connected correctly


Gates without json serialization

As part of #2626 we are checking that gates satisfy gate.on(*qubits).gate == gate. To do this we need instances of the gates.  Instead of doing all of these by hand, we leverage the json test data.  In doing this there are a few gates that do not have json serialized forms, but probably should.  These include
cirq.DiagonalGate, cirq.TwoQubitDiagonalGate, cirq.ThreeQubitDiagonalGate, cirq.PauliInteractionGate, and cirq.ArithmeticGate

cirq.ArithmeticGate is an abstract class and so likely does not need serialization (and may be difficult as it represents a general reversible classical computation)

All our now handled.


Add 'Description' to Findings

This is so when a RuleID is not present, there is an
understanding based on the Description
Description:
When a RuleID is not in the configuration, this will help identify the matching rule.
Checklist:

[x] Does your PR pass tests?
[x] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

Consider posting closing comments please.

My bad, Description is added in the latest release. Thank you for opening this PR


[Feature Request]: Fix Hover effect in About Section as you scroll down.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
As you scroll down the hover effect is not working well on the curves it is overflowing that looks not good.
Use Case
I willl fix the hover effect.
Benefits
This will make it look same as the other cards as you point it from curser.
Add ScreenShots
ScreenRecording for better understanding to what am saying:-
https://github.com/user-attachments/assets/51420428-3c1b-4172-b5ce-92fdfe585727
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@KomalSrivastava Go ahead and contribute.


Is it possible to abort loading of higher res image?

Hi folks,
I was wondering (if on really slow connections) we can abort/stop the deferred loading of higher resolution images? For example, a cookie is set for low resolution images only.
Is this behaviour possible via the JS API?
Cheer!

This depends on the browser. Safari (including Safari mobile) does not support aborting image requests.


reactor: select for update executor

Signed-off-by: charlie<EMAIL_ADDRESS>
What this PR does:

reactor the "select for update executor"
fix the unit test for "select for update executor" failed

Which issue(s) this PR fixes:

Fixes #442
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #478 (7754726) into master (437f712) will decrease coverage by 0.08%.
The diff coverage is 29.46%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #478      +/-   ##
==========================================
- Coverage   39.05%   38.98%   -0.08%
==========================================
  Files         144      146       +2
  Lines        9120     9342     +222
==========================================
+ Hits         3562     3642      +80
- Misses       5264     5403     +139
- Partials      294      297       +3

Impacted Files
Coverage Δ

pkg/client/client.go
0.00% <0.00%> (ø)

pkg/datasource/sql/exec/at/at_executor.go
0.00% <0.00%> (ø)

pkg/datasource/sql/exec/at/config.go
0.00% <0.00%> (ø)

pkg/datasource/sql/exec/at/delete_executor.go
22.77% <0.00%> (-0.70%)
:arrow_down:

.../datasource/sql/exec/select_for_update_executor.go
0.00% <0.00%> (ø)

pkg/datasource/sql/types/executor.go
0.00% <0.00%> (ø)

.../sql/undo/builder/mysql_update_undo_log_builder.go
0.00% <0.00%> (ø)

...tasource/sql/exec/at/select_for_update_executor.go
29.95% <29.95%> (ø)

pkg/datasource/sql/exec/at/update_executor.go
25.00% <33.33%> (ø)

pkg/datasource/sql/exec/at/insert_executor.go
41.40% <57.14%> (+0.03%)
:arrow_up:

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Document valid conditions for making assertions

The documentation (see https://esrally.readthedocs.io/en/stable/recipes.html?highlight=debug#checking-queries-and-responses) does not describe what fields are available and what conditions are supported in assertions on responses.
For example, I tried using assertions on a raw-request, which was just silently ignored by the system.
Further, it is unclear what fields are available and what comparison operators you can use.

Additionally, rally currently silently ignores assertions (just logging a DEBUG message), maybe this should be treated as a hard failure (see https://github.com/elastic/rally/blob/067b27cbc9d807d4c37ec17fc066c461d8fba6d9/esrally/driver/runner.py#L399)


Update scalactic to 3.1.2

Updates org.scalactic:scalactic from 3.0.8 to 3.1.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalactic", artifactId = "scalactic" } ]

labels: library-update, semver-minor

Codecov Report

Merging #57 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #57   +/-   ##
=======================================
  Coverage   31.31%   31.31%
=======================================
  Files          41       41
  Lines        1705     1705
  Branches       27       27
=======================================
  Hits          534      534
  Misses       1171     1171

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2134f7f...b975354. Read the comment docs.


Load OSM file (Version 10.4x64)

The tool Load OSM file seems to work until you open the filegeodatabase, which is empty.
Added : the same results with the tool  OSM File Loader (Load only)

Version used : OSM 10.4x64 on ArcMap 10.4.1
Am I doing something wrong or is the tool not working properly?
Regards,
Robert

The tool can't find any elements in the OSM file as the counting elements line doesn't report any nodes, ways, or numbers. How did you generate the OSM file?

I have examined the file and indeed it is empty. It gave an error message that it ren out of memory. The file itself was downloaded from OSM after making a selection.  I have to say that this very strange. I will try another file and look what it does. Thank you for your reply. I will give an update later.

The export button on the OSM website results momentarily in an webpage error, I don't if it is related. Via the Overpass API I could download a file, which was imported successfully. So the error seems to be the file and not the tool.
Thanks again!

Be aware that even using the Overpass API, there may at times, and especially with larger extracts, be query time-outs, meaning the produced XML file is incomplete. AFAIK Overpass API has some limits set on the maximum amount of RAM a query may consume, and also the time a query may run, so as to avoid using up the entire servers resources by a single user.
For country size extents, it is best to download an extract from Geofabrik, where you can easily select your country:
http://www.geofabrik.de/

@cloveman

Thank you all for your replies and insights. It was very useful for me!


[SUPPORT] MOR Hudi failed to upsert after upgrading EC2 instance.

Describe the problem you faced
We encountered Errors in the production pipeline yesterday when we increased our EC2 instance type from c7g.4xlarge to c7g.8xlarge
To Reproduce
Steps to reproduce the behavior: Upgrade EC2 instance to c6i.8xlarge and increased spark memory
Expected behavior
Job runs successfully with improved performance.
Environment Description

Hudi version : 0.13.0

Spark version : 3.3

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : No

Additional context
Once we reverted back the EC2 upgrade, the job started working properly again.
Stacktrace
PFA  the change we did and the errors we got: -
Case 1.txt
Case 2.txt

Had a discussion with @PhantomHunt . as we are using local[*} mode, Spark tries to maximize parallelism by creating as many parallel tasks as there are available cores or threads on your machine. While this can speed up data processing, it also introduces the risk of ConcurrentExceptions. These exceptions happen when these parallel tasks are not synchronized properly, leading to conflicts when accessing shared resources like variables or data structures.
Thus bigger machine will cause more threads and more risk of ConcurrentException.

@PhantomHunt Closing out this issue as anyway it is unrelated to hudi. Please reopen in case you have any issues on the same.


add request-a-team

Docs Build status updates of commit 0d50447:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/request-a-team-one.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/request-a-team-two.jpg
:white_check_mark:Succeeded
View

msteams-platform/samples/app-templates.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 3df3bcc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/request-a-team-one.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/request-a-team-two.jpg
:white_check_mark:Succeeded
View

msteams-platform/samples/app-templates.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit dd28f6b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/request-a-team-one.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/request-a-team-two.jpg
:white_check_mark:Succeeded
View

msteams-platform/samples/app-templates.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 6aeb729:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/request-a-team-one.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/request-a-team-two.jpg
:white_check_mark:Succeeded
View

msteams-platform/samples/app-templates.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add “x-maturity: production” and “translator-id” to all existing SmartAPI registrations

Due Nov 2.

Fixed by #446 .


#2073 Allow Users To Review Exercise Changes Prior To Approval

What's included?
When a user unlocks an exercise to edit it, ensure the subsequent approver can see a list of the fields that have changed.
Closes #2073
Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?
TEST 1

Select an existing live exercise
Click to Unlock the exercise for editing
Make some changes
Click Submit For Approval on the Overview page
The approver should see a message above the Approve/Reject buttons showing them the fields that have changed

TEST 2

Create a new exercise
Fill in all the sections so that the Submit For Approval button is displayed on the Overview page
Submit it for approval
Approve it
Publish it
Click to Unlock the exercise for editing
Make some changes
Click Submit For Approval on the Overview page
The approver should see a message above the Approve/Reject buttons showing them the fields that have changed

Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context
See screenshot:

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

I added some code to ensure the firestore timestamps aren't overlooked in the comparison

The current code will work for new exercises but not for existing ones as they will still experience the issue whereby the following fields are false in exercise but null in vacancy so show up as having changed when they haven't:
welshRequirement
inviteOnly
previousJudicialExperienceApply
welshPosts
aboutTheRoleWelsh
aSCApply
roleSummaryWelsh
We need to run a nodeScript to fix this on all environments!

@drieJAC Are they testing this on Develop or Staging?

Develop

This is awaiting a retest from @nickaddy because the testers were unable to test it

@drieJAC Happy for this to be merged now - as agreed at the stand-up, having users use different sign-ins likely to cause confusion.

The changes were not displaying so I added a fix for this in this codebase and also in Digital Platform, see: https://github.com/jac-uk/digital-platform/pull/926

@Franceswog @nickaddy would you be able to review (and approve) this one please. @drieJAC is happy to demo if you like


When E-mail is required, UI still shows "optional"

In the client, when the admin sets  data-isso-require-email="true", the E-mail input box still shows E-mail (optional). This is confusing to users.
The same to require-author.

I've found that it's a translation problem. zh_CN and zh_TW translation can't match the regex / (.*)/


cleanse command doesn't delete the upstart file

The chef-server-ctl cleanse command in Chef Server 12 doesn't delete the upstart file because it's looking for /etc/init/chef-server-runsvdir.conf when the file is actually still named /etc/init/private-chef-runsvdir.conf.
https://github.com/chef/omnibus-ctl/blob/master/lib/omnibus-ctl.rb#L223

@jeremiahsnapp Thank you for the report.  This issue has been moved to the new chef-server repository: https://github.com/chef/chef-server/issues/179