Fix Time Period filter - 508 Issue

Required Fix

[ ] RF-11
[x] RF-12
[x] RF-13

Context
See EPA CAMD Accessibility Report  for how to fix and additional details

Issue #
Issue Description

RF-11
The messages shown to the user as a result of input errors in form fields on the page are not accessible to Assistive Technology. This issue is a violation of Section 508 and of Success 3.3.1 in WCAG 2.0.

RF-12
There is a label associated with the form field but the label itself is not descriptive or accurate. This issue is a violation of WCAG 2.0 Success Criterion 2.4.6.

RF-13
Generic link text such as ‘Click Here’, ‘Read More’, etc. is used, and there are no clues to link destination or function in the text immediately surrounding the link. This issue is a violation of WCAG 2.0 Success Criterion 2.4.4

Testing
Tester: @aprematta

Testing complete, RF 12 and 13 are good. RF-11 is still an issue. I can see it's in a live region however neither screenreader I use is announcing the error message when it displays. The check marks are also still inaccessible. I see that aria checked is used but this os only valid for form selections.

Issue RF-11 covered in Bug: Make Time Period Error Message Screen Reader Compatible #1549


Implement the date range in Strategy

Strategy has a start and end date that are specified but not used by the engine. Make the engine use these.

Delivered to master.
The sample range is calculated from the given dates and used instead of the whole dataset in the strategy engine now. I also added a utility class for getting the unix timestamp in seconds from a DateTime and fixed a bug in writing/reading datetimes to json.
Commit
commit 40925417b63f747637a10854cff973a28e1cfe33
Author: Alex Hunt<EMAIL_ADDRESS>Date:   Sat May 21 11:25:47 2022 -0400
Issue75: Implemented the date range functionality in Strategy.

Testing

Select no date range and verify the old behavior still works
Select a start and end date, save and reload the app, run the strategy and verify it is only run on the specified range.
Step thru with a debugger and verify the exact sample is correct (no off by one error)
Run a date range that is out of bounds of the data and verify no crash occurs and it doesn't buy/sell anything.


3.3. Implement validation to verify that the quiz ID exists before proceeding with the edit operation.

Task 3.3 has been completed


Feature/migrate to user ids in configs

Pull request checklist:

[ ] CHANGELOG.md was updated, if applicable
[x] Documentation in docs/ or install-docs/ was updated, if applicable
[ ] I have tested all changes

To document it in writing as well: I'm ok with this upgrade, but I need it to be backwards-compatible. All old ways of configuring the bot must continue working.

this is now in a testable state, I will deploy it to my prod system and test it out

$(tb:broadcaster) doesn't seem to work anymore:

Based on my logs, there might be some other ones not working:
pajbot@weest[841]: [2021-11-21 09:22:38,809] [INFO   ] Set current stream chunk here to <pajbot.models.stream.StreamChunk object at 0x7f4525753d00>

Based on my logs, there might be some other ones not working: pajbot@weest[841]: [2021-11-21 09:22:38,809] [INFO ] Set current stream chunk here to <pajbot.models.stream.StreamChunk object at 0x7f4525753d00>

Exactly the same in master: [2021-11-21 12:34:55,719] [INFO   ] Set current stream chunk here to <pajbot.models.stream.StreamChunk object at 0x7f2d225e5c40>
this is the original debug output


New Smissmas 2020 unusuals that doesn't works

3049 - utaunt_tf2smissmas_tree_parent (Particle stays in the ground)
3050 - utaunt_tf2smissmas_tree_parent_w (Same as first)
3053 - utaunt_balloonicorn_reindeer_teamcolor_red (Same as first)
3054 - utaunt_balloonicorn_reindeer_teamcolor_blue (Same as first)
3055 - utaunt_twinkling_rgb_parent (Particle is buggy)
3056 - utaunt_twinkling_goldsilver_parent (Same as 3056)

Only 3051 (utaunt_spirits_blue_parent) and 3052 (utaunt_spirits_purple_parent) works properly.

I have the same issue...
The "Creepy Crawlies Unusual Effect" also isn't working properly as it dosen't functions like shown on it's workshop page:

Source

Yeah do they work now?

Let me explain, the unusual are applied to the body of the character in the world as if it were an entity, causing some effects to be broken like this one or the ones from Halloween last year that follow the movement of the character.

so it can't be fixed basically... sad

Yes but no, we would have to find a different way to apply particles to the taunts, i think that with TF Econ Data it is possible.

Yes but no, we would have to find a different way to apply particles to the taunts, i think that with TF Econ Data it is possible.

If it is, it would be awesome to have it fixed! :D


Build template changes alias

In the build.cmd template (https://github.com/aspnet/Universe/blob/dev/build-template/build.cmd) there is this part:
CALL packages\KoreBuild\build\dnvm install default -runtime CoreCLR -arch x86

:run
CALL packages\KoreBuild\build\dnvm use default -runtime CLR -arch x86

The problem is that this globally changes the default alias, which messes up with existing setups.
The template should be changed to either use no or another korebuild specific alias, and this should be propagated to all other repos that use the build template, as building any of these will change (and potentially break) existing settings.

Yeah I do find this a little bit odd, but it is likely designed with the assumption that somebody running this is constantly working on the product, so setting it globally "isn't so bad". But I dunno, I could see it either way on this...

We could choose a different alias but need some alias because it's used not only in build.cmd and build.sh but also _k-test.shade. Current approach also works well when $env:SKIP_DNX_INSTALL is defined, as it is on our CI machines.

dnvm is no longer used in KoreBuild so this will not be a bug anymore


Running Spec

Hi,
I am trying to run the crystal all_spec target on OSX Yosemite. I installed all required libraries and the project builds fine. However, when I run make all_spec after make clean I get the following error:
$ make all_spec
./bin/crystal build  -o .build/all_spec spec/all_spec.cr
Error in ./src/logger.cr:60: instantiating 'IO#<<(Char)'

    io << severity[0] << ", [" << datetime << " #" << Process.pid << "] "
       ^~

in ./src/io.cr:283: type must be IO::ARGF, not NoReturn

  def <<(obj) : self
      ^~

make: *** [.build/all_spec] Error 1

and when I run it after running make I get:
$ make all_spec
./bin/crystal build  -o .build/all_spec spec/all_spec.cr
Using compiled compiler at .build/crystal
/bin/sh: line 1: /usr/bin/cc: Argument list too long
Error: execution of command failed with code: 126: `cc -o .build/all_spec "/Users/rbradberry/Projects/devdazed/crystal/.crystal/Users/rbradberry/Projects/devdazed/crystal/spec/all_spec.cr/main.o" "/Users/rbradberry/Projects/devdazed/crystal/.crystal/Users/rbradberry/Projects/devdazed/crystal/spec/all_spec.cr/Iterator5858Stop.o"

.........

which continues to have a log of filenames.  Not sure what I may be doing wrong here.
Thanks

Uh, are you sure your repo is up to date? What's git log --online -1 and git status?

Yes, this is one reason we'll make a release today: the specs don't compile in the current compiler. They do, however, in the new compiler. Try running make clean crystal spec

repo is up to date with master.  I tried running make clean crystal spec and got the following
$ make clean crystal spec
rm -rf .build
rm -rf ./doc
CRYSTAL_CONFIG_PATH=`pwd`/src ./bin/crystal build  -o .build/crystal src/compiler/crystal.cr
./bin/crystal build  -o .build/all_spec spec/all_spec.cr
Using compiled compiler at .build/crystal
/bin/sh: line 1: /usr/bin/cc: Argument list too long
Error: execution of command failed with code: 126: `cc -o .build/all_spec "/U

again with a huge list of every file

Related to https://code.google.com/p/chromium/issues/detail?id=386046?

-       $(BUILD_PATH) ./bin/crystal build $(FLAGS) -o $@ spec/all_spec.cr
+       $(BUILD_PATH) ./bin/crystal build $(FLAGS) -o -Wl,-filelist spec/all_spec.cr

didn't seem to make a difference

actually
diff --git a/Makefile b/Makefile
index 3809a67..86d9a7f 100644
--- a/Makefile
+++ b/Makefile
@@ -19,11 +19,11 @@ all_spec: $(O)/all_spec

 $(O)/all_spec: $(SOURCES) $(SPEC_SOURCES)
        @mkdir -p $(O)
-       $(BUILD_PATH) ./bin/crystal build $(FLAGS) -o $@ spec/all_spec.cr
+       $(BUILD_PATH) ./bin/crystal build $(FLAGS) -o -Wl,-filelist spec/all_spec.cr

 $(O)/crystal: $(SOURCES)
        @mkdir -p $(O)
-       $(BUILD_PATH) $(EXPORTS) ./bin/crystal build $(FLAGS) -o $@ src/compiler/crystal.cr
+       $(BUILD_PATH) $(EXPORTS) ./bin/crystal build $(FLAGS) -o -Wl,-filelist src/compiler/crystal.cr

 clean:
        rm -rf $(O)

gets me back to this error:
$ make clean crystal spec
rm -rf .build
rm -rf ./doc
CRYSTAL_CONFIG_PATH=`pwd`/src ./bin/crystal build  -o -Wl,-filelist src/compiler/crystal.cr
./bin/crystal build  -o -Wl,-filelist spec/all_spec.cr
Error in ./src/logger.cr:60: instantiating 'IO#<<(Char)'

    io << severity[0] << ", [" << datetime << " #" << Process.pid << "] "
       ^~

in ./src/io.cr:283: type must be IO::ARGF, not NoReturn

  def <<(obj) : self
      ^~

make: *** [.build/all_spec] Error 1

@devdazed Try with HEAD, I just fixed the "Argument list too long"

spec runs now, thanks!


Update chart doc with details about the license file

Closes #4754
Signed-off-by: Martin Hickey<EMAIL_ADDRESS>

@bacongobbler Thanks for reviewing and merging.


Calc needs a variable to tell how many $nn values were defined on the command line

So, for instance, Java expands wild-card file specs, so "c -- test*.log --" will expand to put maybe 20 file names into the $0, $1, $2, etc. variables, but it is awkward to tell how many were defined. So, two things come to mind to help:

Define "$#" as the number of defined values ($0 .. $#-1)
Or define "$*" as the array of them, so you can do "loop $arg in $* { .. }"

Maybe do both? so there is a choice as to how to process.

This could be expanded to function parameters also, so that varargs constructs can be supported, as such:
def f($*) = { ... }
then you could write loop $* { process $_ }
Then $# and $* would be defined for the function. This would mean that the command line values wouldn't be available inside the function (although access via $0, $1, etc. WOULD still work). Probably not that horrible a situation...

Related issue #70 found trying to test this automatically. Not going to close this one until that is fixed.

The global variables are set, but the function args version of this issue is not done yet. Leaving open.

I think it would be better to use a more standard varargs format: ".." or "..." (0x2026) (same as loop control syntax), and to allow formal parameters first, with varargs last.

Fully implemented function varargs, as well as the "$*" and "$#" local variables for functions. Closing now as this is DONE.


Prepare base64 string improved to accept not only png but jpeg,jpg,bm…

…p,gif.
Otherwise on Obj C side  NSData image is nill

@DimaFr have you tested this change both on Android and iOS? The base64 string is properly transformed in an image for all the format?
If so I'll merge the pull request asap


Visual Studio 2022 installation path

Visual Studio 2022 version 17.1 or later on a 64-bit machine
C:\Program Files (x86)\Microsoft Visual Studio\2022\ ...
on my Windows 10 64bit correct path is:
C:\Program Files\Microsoft Visual Studio\2022\ ...
btw, great article!

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 513f1e35-aba1-8b92-845e-1ed4ccad8e62
Version Independent ID: 45eb46ce-a9b1-f141-5a97-c448b9349c48
Content: Command-line shells & prompt for developers - Visual Studio (Windows)
Content Source: docs/ide/reference/command-prompt-powershell.md
Product: visual-studio-windows
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@Zer0cul You're absolutely right, we'll get that fixed ASAP and report back when the fix is published. And, thanks for the compliments on the page, too!

@Zer0cul The updated page is published; we'll close this issue now with our thanks to you for your interest & help with improving the Visual Studio documentation!


chore: remove apollo [WEB-611]

The size check CI job is failing because it checks out master to compare the size against, but I've removed the codegen script so the generated files are missing, hence it fails. It should not fail in the next PR after this one has been merged as master will have been updated to not rely on any generated files


find project path in ios folder when named differently from package.json

Adds a new flag --ios-project that can be specified to override looking for an Xcode project with the exact name specified in the package.json.
Helps with #32
Feel free to reject or add suggestions.  Thanks

I think that RN projects will most likely only have one .xcodeproj in the ios/ dir. Maybe instead of adding another option, we should just find and select the first one in programOpts.ios.
(It's funny because it felt kind of wrong using appPkg.name in the first place.)

@stovmascript I started a new branch on my fork and didn't think I could re-use this PR, so I opened #34.  Thanks!


Adds options to which models are included

Pull Request
Are you a current Fivetran customer?
Partner
Chris McClellan
Director
Visualisedata Pty Ltd
AUSTRALIA
What change(s) does this PR introduce?
Adds options to dbt_project.yml to optionally load some of the models.  I wanted to use this package to show what Fivetran was doing (during the trial period), but some of the models are not available and that causes an error.  With these changes I don't get any errors and I can report on the data because the run finishes without any errors
Did you update the CHANGELOG?

[x] Yes

Does this PR introduce a breaking change?

[x] Yes (please provide breaking change details below.)
[ ] No  (please provide an explanation as to how the change is non-breaking below.)
I'm going to say YES to be safe.  I only thought of that now - if users don't have the vars defined then there will be an error I guess.  I haven't tested this, but I'm happy to do the work if this is an issue that needs to be fixed and this PR is going to benefit the package.

Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

[x] Yes

Is this PR in response to a previously created Bug or Feature Request

[ ] Yes, Issue/Feature [link bug/feature number here]
[x] No

How did you test the PR changes?

[ ] CircleCi
[x] Local (please provide additional testing details below)

I tested it with all the vars properly defined in the root dbt_project.yml and it passed all tests.
I DID NOT test when the vars are not defined in the root dbt_project.yml
Select which warehouse(s) were used to test the PR

[ ] BigQuery
[ ] Redshift
[x] Snowflake
[ ] Postgres
[ ] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

:grinning: hopefully, but a lot of :worried: at the same time
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

Hey @cmcau thanks so much for opening this PR!!
These changes make complete sense and I understand why you would want to remove these models from the run as they do not exist during the trial phase. My one question I have is regarding the value of the end models with these disabled. Are you still able to gain actionable insights in your opinion with these disabled?

Well, I don't know what I'm missing right ? ;)
I have created a visualisation that shows the data in the log_audit table (so I can see Deletes, Updated and Replaced or Inserted by Connector and Table Name.
Active Volume seems to be the most interesting one that I'm missing (but I'm only going on the name of the model)


For the comp band analysis it would be useful to differentiate fractional bands and corporate ones

As a manager, my ability to set compensation may be limited by corporate bands and not just the sub-bands within them, so I would like to see which type of band problem I am looking at.

Decided to go with an absolute dollar value above/below band instead.


Revert "net: wifi: Fix DPP disabled build"

This reverts commit 1da74ef705bcf1723a3271266622948251253c0a.

to fix issue https://github.com/zephyrproject-rtos/zephyr/issues/79897

Please add relevant information why the old commit needs to be reverted to the commit message.

Please add relevant information why the old commit needs to be reverted to the commit message.

updated

But why do we need to revert the commit 1da74ef705bcf1723a3271266622948251253c0a, what kind of issue it is causing now?

But wouldn't the original issue then be present what the 1da74ef is fixing if we revert it?

If hostapd disables DPP (by disabling CONFIG_WIFI_NM_WPA_SUPPLICANT_DPP, which is zephyr only way to disable DPP),
L2 DPP code will be still there and can be tested, no build error, which is from original review comments request.
Only hostap will return fail when DPP L2 api is called, which won't be trouble.
So I don't see what 1da74ef is trying to fix. Could you pls explain it in detail?

But wouldn't the original issue then be present what the 1da74ef is fixing if we revert it?

If hostap disables DPP (by disabling CONFIG_WIFI_NM_WPA_SUPPLICANT_DPP, which is zephyr only way to disable DPP), L2 DPP code will be still there and can be tested, no build error, which is from original review comments request. Only hostap will return fail when DPP L2 api is called, which won't be trouble. So I don't see what 1da74ef is trying to fix. Could you pls explain it in detail?

Please see https://github.com/zephyrproject-rtos/zephyr/issues/79897#issuecomment-2415994308 just including the header file is enough to get this error, but for some reason not seen with upstream.

@jukkar @krish2718 updated fix. thanks for clarification.

Not sure about twister policies in upstream, but should we add these combinations to avoid such issues?

Not sure about twister policies in upstream, but should we add these combinations to avoid such issues?

That would be a good idea to add various config combinations to tests, not sure if tests/net/wifi/wifi_nm is a good place for those. Perhaps separate directory next to it to compile test various config combinations or enable-all type config.

Perhaps separate directory next to it to compile test various config combinations or enable-all type config

Just saw a PR merged with build_all that builds all drivers, we can have something similar for WPA supp?

Just saw a PR merged with build_all that builds all drivers, we can have something similar for WPA supp?

Yeah we could, I can prepare a PR for that.


WebApp

SUMMARY

added some hover effects features and some Ui design
PR Checklist
Please check if your PR fulfills the following requirements:

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

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Testing type

[x] Manual Testing
[ ] Unit Testing

What is the current behavior?

it did not have the hover effect feature like udemy and coursera  have
Issue Number: N/A
What is the new behavior?
now it has hover effect if you hover the element it will show you its description part
Does this PR introduce a breaking change?

[x] Yes
[ ] No

Other information

N/A
PIC

Update the proper title  instead of  just writing WebApp.


Regression: 2.0 now returns Attr nodes instead of their parents on xpath ending with /@attr

Hey.
I've already reported this issue as #36 and thought it's fixed with 2.0.1, but unfortunately, I was testing against yet another problem, which was #37. The original #36 issue is not fixed and is still reproducible with version 2.0.1.
Original bug report below:

Consider this example HTML that I've extracted from one of the websites I'm using:

				 DEVICE NAME
				 Sharing Status
				 LIBRARY LAST ACCESSED
				 BY STEAM USER

						 1
						 redacted

															Authorized (
								Revoke )

						 redacted
						 redacted

		You can allow up to 5 accounts to use your game library on any of your authorized computers.

				 User Name
				 Sharing Status
				 Time added

						 1
						 redacted

													Authorized (
							Revoke )

						 redacted

Until now with AngleSharp.XPath version 1.1.7, I've been using the following xpath expression which worked fine:
(//table[@class='accountTable'])[2]//a/@data-miniprofile

The objective of the above is to extract all a links containing data-miniprofile attribute, but only from second accountTable.
Unfortunately, the same xpath in version 2.0.0 or 2.0.1 doesn't return any rows. It worked fine as of version 1.1.7.

I consider this being a regression. If this is planned, may I know what is the recommended way to update in regards to match previous use case?
Thank you in advance for your time.

/cc @denis-ivanov

@JustArchi , https://github.com/AngleSharp/AngleSharp.XPath/pull/41

@denis-ivanov Sorry for misunderstanding, I dug deeper. I'll try to explain in detail now.
Initial problem was indeed related to #37. The current one is that response type has changed.
I've checked on another
This is from old 1.1.7 version:

As you can see, AngleSharp.XPath returns list of HtmlDivElement objects, each having Attribtues which contain Attr objects. This matches behaviour of for example HtmlAgilityPack and was the original code I've migrated over to AngleSharp.XPath.

With version 2.0.1, after #37 being fixed, the equal amount of nodes are returned, but they're now of Attr type:

This is a breaking change (regression), because previously I had HtmlDivElement objects on which I could call GetAttribute("attr-name") and also had access to other data (for example, other attributes than in the xpath query). Right now I have access only to the resulting Attr without original node.
I've send PR to your issue-40 branch to reproduce the issue. You can verify that it passes on old version of 1.1.7: https://github.com/AngleSharp/AngleSharp.XPath/pull/42
I'm deeply sorry for misunderstanding and mixing things up. Thank you for your patience.

@JustArchi, current behavior is correct. 1.1.7 - did not support attributes selection, because inside AngleSharp IAttr interface did not implement INode interface. Since it started to implement it, I updated AngleSharp and major version.
If you want old behavior, you have to use 1.1.7, in other case just find all xpath expressions with attributes and add /parent::*.
You can also check browsers behavior:
$x("(//table[@class='accountTable'])[2]//a/@data-miniprofile")
[data-miniprofile]

Thanks, if this behaviour is intended then I don't have any problem with it and will just adapt the usage. I just wanted to be sure, since I don't want to rewrite a lot of code paths only to revert it later on.
Thanks for your patience and work in regards to this project, I appreciate it, have a nice day! :+1:


Custom header

I need to create a pull request to allow for custom headers so we can track usage of the new node SDK.  Alex, do you have a preference for how that comes through?  I can add another parameter after the callback.  Or I could add it to the 'options' parameter and remove it from the options object after I process it.  Thoughts?

I would take a look at how the REST client does it. I assume you're talking about object-wide headers, not request based headers? Both a possible, but based on this PR it looks like you're going for request based headers.
Rest Client Examples (object-wide headers) (request specific headers)
A simple lodash merge would accomplish what you want.


Plugin is defined but does not automatically enable

Additional java script is required to  "activate" the plugin. This should rather happen as part of the inclusion of the plugin.
It is simple to do, just:

link the constructor on the prototype
link the constructor on the plugin
link the bootstrap data-api
add a no conflict (just in case)

What do you mean "activate"? Do you mean include the delegate code that is in the documentation under "Implementation"?
This is how it was originally but it can cause conflicts with other libraries so was removed.

Hi Ashley,
Yes that is what I mean.
Is'nt it better to just add a "NoConflict" function that can be used by those where a conflict exists?
Add a variable before the plugin is declared to that any old conflicting plugin can be returned
 var old = $.fn.ekkoLightbox;
  $.fn.ekkoLightbox = function(options) {

Then add a noConflict function for the case where a conflict occurs.
  $.fn.ekkoLightbox.Constructor = EkkoLightbox;

    // MODAL DATA-API
    // ==============
  $(document).on('click.bs.ekkoLightbox.data-api', '[data-toggle="lightbox"]', function (e) {
      event.preventDefault();
      $(this).ekkoLightbox();
  });

    // MODAL NO CONFLICT
    // =================

  $.fn.ekkoLightbox.noConflict = function () {
      $.fn.ekkoLightbox = old;
      return this;
  }

This makes implementation of the plugin consistent with other plugins. Don't you agree?

There is a PR for this issue https://github.com/ashleydw/lightbox/pull/178

the automatic binding wasn't added as it would be a breaking change from current system where manual instantiation is required
noConflict concern has been addressed in latest version 5 for BS4


Injection does not work with Generic Classes

Hey I just realized that injection doesn't work when I subclass view controller from generic view controller. I have a file like this.
import UIKit

class GenericController<A>: BaseViewController {

}

class NonGenericController: BaseViewController {

}

class LoginViewController: GenericController<UIView> {

    override func viewDidLoad() {
        super.viewDidLoad()
        view.backgroundColor = .white
    }

    #if DEBUG
    @objc func injected() {
        for subview in self.view.subviews {
            subview.removeFromSuperview()
        }
        viewDidLoad()
    }
    #endif
}

When I subclass like this it's doesn't reload and I'm getting following output:
`Compiling /Users/mk/proexe/dogWalkTrophy-ios/DogWalkTrophy/DogWalkTrophy/Screens/Login/LoginViewController.swift

Injection: Loading /Users/mk/Library/Developer/Xcode/DerivedData/DogWalkTrophy-ckqnnpzyqukneiczeuwblphvsqyg/Logs/iOSInjectionProject/build/Debug-iphonesimulator/InjectionBundle100.bundle
objc[35393]: Class _TtC13DogWalkTrophy20NonGenericController is implemented in both /Users/mk/Library/Developer/CoreSimulator/Devices/78BAC6B7-531C-4136-9581-65147C03C856/data/Containers/Bundle/Application/FE4C6AD6-4AF6-4270-88F7-08D182370AE7/DogWalkTrophy.app/DogWalkTrophy (0x10f994810) and /Users/mk/Library/Developer/Xcode/DerivedData/DogWalkTrophy-ckqnnpzyqukneiczeuwblphvsqyg/Logs/iOSInjectionProject/build/Debug-iphonesimulator/InjectionBundle100.bundle/InjectionBundle (0x13269e7a0). One of the two will be used. Which one is undefined.
80 injections performed so far.
Injection: Ignore any warning, Swizzled DogWalkTrophy.NonGenericController 0x13269e7a0 -> 0x10f994810`

but when subclass from non generic code everything works fine:
class LoginViewController: NonGenericController {
Why is that? Anyone has some idea? Thanks!

Generic classes are not visible to the Objective-C runtime which injection uses so this is not possible.

@johnno1962 thanks for answer, that makes sense!


grandpa: log everything under afg target

For some reason we were logging some grandpa messages under the finality target rather than afg. I don't remember any clear reason for doing this and in practice it's one more logging target that we need to think about adding. I changed it so that all messages are logged under the afg target.

It should definitely be an all-or-nothing. I could see everything moving over to a finality target when we have swappable consensus, but for now this will work.


First addition to aggregate

types-setuptools <IP_ADDRESS>40125
Destination channel: main
Links

{PKG-4071}
Upstream repository

Explanation of changes:

added abs.yaml for 3.12 because why not
Updated version and hashes
Removed no-arch
Added no-deps and no-build-isolation

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_c41sp9ipak/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:33: missing_description: The recipe is missing a description

===== ERRORS =====

clone/recipe/meta.yaml:27: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
clone/recipe/meta.yaml:33: missing_dev_url: The recipe is missing a dev_url
clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
clone/recipe/meta.yaml:33: missing_documentation: The recipe is missing a doc_url or doc_source_url
clone/recipe/meta.yaml:27: missing_pip_check: For pypi packages, pip check should be present in the test commands
clone/recipe/meta.yaml:33: missing_license_family: The recipe is missing the about/license_family key.
===== Final Report: =====
6 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_bdl0kssmkx/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:35: missing_description: The recipe is missing a description

===== ERRORS =====

clone/recipe/meta.yaml:35: missing_dev_url: The recipe is missing a dev_url
clone/recipe/meta.yaml:35: missing_documentation: The recipe is missing a doc_url or doc_source_url
clone/recipe/meta.yaml:35: missing_license_family: The recipe is missing the about/license_family key.
clone/recipe/meta.yaml:29: missing_pip_check: For pypi packages, pip check should be present in the test commands
clone/recipe/meta.yaml:29: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
===== Final Report: =====
5 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_5fvvgcyv7y/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:43: missing_description: The recipe is missing a description

===== ERRORS =====

clone/recipe/meta.yaml:33: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
===== Final Report: =====
1 Error and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_8ehji0c4be/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:48: missing_description: The recipe is missing a description
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_18za9bnzij/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:46: missing_description: The recipe is missing a description
===== Final Report: =====
0 Errors and 1 Warning were found


home

@MihaiDejeu this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2020-12-30T09:46:59.744273.
Estimation here is 250 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@PerteGeorge please keep in mind that manual assignment of tickets is a bad practice and it is discouraged. Next time, please let me elect the assignee.

@MihaiDejeu Don't forget to close this ticket before the deadline (2020-12-30T09:46:59). You are past the first half of the allowed period.

@MihaiDejeu Looks like you've missed the task deadline (2020-12-30T09:46:59). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@PerteGeorge this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2021-01-09T09:51:42.875515.
Estimation here is 250 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@PerteGeorge Don't forget to close this ticket before the deadline (2021-01-09T09:51:42). You are past the first half of the allowed period.

@PerteGeorge Don't forget to close this ticket before the deadline (2021-01-09T09:51:42). You are past the first half of the allowed period.

@PerteGeorge Looks like you've missed the task deadline (2021-01-09T09:51:42). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@PerteGeorge Looks like you've missed the task deadline (2021-01-09T09:51:42). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@PerteGeorge I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.

@PerteGeorge I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.


Custom Operator Perf Script and Error Message

Description
Some minor improvement to C++ custom operator, optionally can go into 1.7
Add reproducible performance test script and improve some error messages. It is an address of comments from PR #17885

@ciyongch Can you tell you if the backport happens on 4.25 11:59 pm? If it is then I can backport this to 1.7

@rondogency yes, the code freeze will be 4/25 11:59 pm PST. Please arrange your time for this patch, thanks!

@ciyongch after discussion I decide to change this PR to only modify docs and examples and merge it later, so that it won't need to go into 1.7 (no change to binary). You can ignore this PR now.

Ok, then I'll not track this PR in the release, thanks for the update :)

@mxnet-bot run ci [all]


Implement inference for constructor calls in annotations

The analyzer currently doesn't do type inference + instantiate to bounds for constructor calls used in annotations.   This is relatively low priority, but should be fixed at some point.
cc @bwilkerson @stereotype441 @devoncarew @lrhn

Is this, er, still true? Are there examples or tests we can refer to?

Is this, er, still true? Are there examples or tests we can refer to?


Fix link to model versions history in README

Thank you so much for your work!
This fixes the link to model versions history in the README, which I think was outdated.
I hope this helps!

Thanks good catch!


project endpoint v2

@halitbatur and this also what is happening with this pr ?

@edaguler00 status on this? it's been more than a week since the last review.

@yaman3bd PTAL

@edaguler00 make sure to do git pull origin feature/crud_project_endpointv2 on this branch before committing new changes since we solved the conflicts and merged the updated main FYI


Improve travis config

[ ] Multiple platforms
[ ] Coverage only ones after success

This one is not relevant anymore since we migrated to GH Actions


New plugin: Vercel CLI

Platform or tool
Vercel
Relevant links
https://vercel.com/docs/getting-started-with-vercel
https://vercel.com/docs/cli
Goal or desired behavior
Authenticate & Connect to Vercel  CLI
Contribution

[X] I'd be up for building the plugin myself.

Additional information
No response

@AndyTitu @volodymyrZotov Please assign this issue to me

Hey @bala-ceg, thank you for your interest in building a shell plugin for Vercel! There's already a Vercel shell plugin in the final stages of the review:

https://github.com/1Password/shell-plugins/pull/273

We'd love to see shell plugins for other Developer Experience tools you use though!

I will be re-opening this until the plugin for Vercel gets merged, since there is no other issue opened to track this change.
@bala-ceg if you'd like, please do take a look at the already existing PR Arun mentioned to review additionally or to pick it up from the almost completed state. It seems the PR is approved and only blocked by some unsigned commits though, so only reviewing would make more sense.

The Vercel  plugin has been merged https://github.com/1Password/shell-plugins/pull/273


set severity and retry condition of errors

This PR adds severity and retry condition to error and propagate them into info messages for blob-contentTree-Volume-App flow.
Signed-off-by: Petr Fedchenkov<EMAIL_ADDRESS>

Did you already look at all tjhe code (in zedmanager, volumemgr, etc) which copy an error from one status to another? I think it makes sense to have all of those transfer the whole errordescription as part of this PR.

I think it will affect too many lines to be compatible with existing PRs

I think it will affect too many lines to be compatible with existing PRs

OK, let's do that in a follow-on PR.
So just address the remaining code comments and we'll run tests.


WIP: Bulk sync of upstream operator-registry

Synch'd commits from upstream:

494b68e62 opm serve: use pre-existing cache, if set and up-to-date (#1005)        !!— blacklist —!!
bd3c80489 Updating workflows to pull go version from go.mod file (#1025)
6d762ada5 bump k8s to 1.25 (#1024)
0271e7847 go upgrade to 1.19 (#1029)
89594183c left-right swap edge vertices (#1041)
0080ea0a9 semver veneer: error when unknown fields are encountered in input file (#1043)
b7307b57c opm migrate: add help text note about concatenated json format (#1055)
fe5123023 Bump k8s dependencies to 1.26.1 (#1072)
0dff01052 bump golang.org/x/net from 0.4.0 to 0.7.0 (#1064)
3bfcb23fa opm render: update help text to mention that output is a stream to stdout (#1056)
49eae7b7f (tag: v1.26.5) Add Support for Fetching Remote Catalog (#1079)
7835ea9c6 Package sort for deterministic ordering in ListBundles API (#1084)
8006f50ca build(deps): bump github.com/containerd/containerd from 1.5.13 to 1.5.18 (#1062)
0aebc3ceb take image domain name from env (#791)
45f491b93 Remove vendor dir (#1085)
00a4cce48 refactor to move more processing to the API (#1082)
41fa00f33 comment out CatalogDestination.BaseImage while we mature the concept (#1090)
e1eebae21 (tag: v1.27.0) Add WalkMetas* functions and meta.Name field (#1089)
7629c6f99 (tag: v1.27.1) resolve channel collisions by a preferred channel type, where they are otherwise identical (#1095)
2ee231b32 render: improve olm.bundle.object rendering for bundles (#1094)
b51aaf05b bump golang-migrate to v4.16.1 (#1107)
647537d94 declcfg/load: improvements (#1106)
6c6028419 (tag: v1.28.0) re-introduce and fix declcfg.Meta unmarshal error (#1109)
337b60ddc Updates GH Actions to avoid using deprecated node (#1117)
b854557bd Switches to codecov GH action (#1118)
43225def6 Bump k8s.io/client-go from v0.26.1 to v0.27.1 (#1119)
bf549304d (operator-registry/master) Avoid making the termination log executable (#1110)

/retest


sak-hydrosphere module is broken

Helm repo URL is not working.
See https://github.com/provectus/sak-hydrosphere/issues/1

We can remove this module because hydrosphere is not supported anymore


Breakout3

Addressing #3
Added requested changes that undo some of reviewer two comments.
Changed a few of reviewer 2's comments to make them look bad. Changed from int/255 to the decimal values so the code is consistant.
Split the main subscripts and solutions. We can figure out if we want this or not.

Thanks for working on this! I'm gonna hold off on a review until after tomorrow's meeting.

I hate the solutions in the script files.
If you want to put them there I'll let you. But if you don't want them copying and pasting like you said in another comment, then they shouldn't be there. The changes are so simple, if they can't figure it out they won't be able to figure out git.
The markdown had information on checking out to a specific branch, and I'm not sure where it went. I don't think it should be addressed here how to checkout to specific commits but should be done earlier.

I hate the solutions in the script files.

🙄

If you want to put them there I'll let you. But if you don't want them copying and pasting like you said in another comment, then they shouldn't be there. The changes are so simple, if they can't figure it out they won't be able to figure out git.

Yep, no solution files. It was simpler before and allows for easy reference because again, the point of the tutorial isn't how to debug or see if they can figure it out. We need this tutorial to be as user-friendly as possible.

The markdown had information on checking out to a specific branch, and I'm not sure where it went. I don't think it should be addressed here how to checkout to specific commits but should be done earlier.
This checkout method is going to be a little odd. Because they 'll need to go into their branch and checkout that branch, then pull request again.

We can talk tomorrow about what this breakout room is/isn't supposed to be about.

OK this PR is ready for review @GBruening.

made changes to breakout 3 after our revert/checkout experiments today
removed solution files (sorry not sorry)
fixed bug in script 5
made reviewer 2's suggestions really ugly
confirmed all solutions ran on my machine, which produced the figures below.
@swkasica, this PR's MATLAB scripts should be ready for porting. Most changes moving forward will be to breakout_3.md

post reviewer comments:

finished/original figure:

I guess gary can't review this. I'll merge this because it's at least functional and we can address revisions later.

I can review it but do that later probably, and I made this comment in the issues.
If we have the solutions in the file, it encourages them to just copy paste it or uncomment and comment things. Literally 0 investment into what they actually need to do. If they actually have to do something, they'll have a better chance of remembering it because theres more investment. The problems are not hard to fix, if you hand them everything on a plate they'll forget it all


fix title level in 02-access.md

fix title-3's level in 02-access.md. (from level-3 to level-2)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Sparkles FTSO V2

Sparkles FTSO V2

Manually added in https://github.com/TowoLabs/ftso-signal-providers/commit/fcb3a4f21a4cc2960dd5af3bf709931351c53b2d

Manually added in https://github.com/TowoLabs/ftso-signal-providers/commit/fcb3a4f21a4cc2960dd5af3bf709931351c53b2d


updated B-Harvest.toml

Updated with proper validator.toml file
sorry for the inconvenience

Update2 is correct

newly created by 0.13.0 for Thursday new genesis


Target is not recognized

Background
I'm currently putting together a ChemKED file for an ignition delay experiment and the experiment uses fuel concentration as the target, measured by a HeNe laser to determine ignition delay. It doesn't use a typical radical as the target, such as OH*, and therefore ChemKED doesn't validate the file and returns a not implemented error. Is there a way to express this in the file? Thanks in advance for your help!
Code sample, preferably able to be copy-pasted and run with no changes
common-properties:
species: &hept
- species-name: n-heptane
InChI: 1S/C7H16/c1-3-5-7-6-4-2/h3-7H2,1-2H3
ignition-type: &ign
target: *hept #ChemKED does not recognize this
type: d/dt max
Expected behavior
To successfully indicate that the target is the fuel n-heptane
Actual behavior, including any error messages
Not Implemented Error
PyKED/ChemKED version: 4.1, Python version: 3.6.7, OS version: Windows 10 Home, version 1803

Hi! Specifying the fuel as a target is not permitted at the moment. However, this could be something we support. I'd encourage you to make a pull request to add fuel and/or *fuel as options here: https://github.com/pr-omethe-us/PyKED/blob/master/pyked/schemas/ignition_delay_schema.yaml#L9 It also doesn't make sense for the criteria to be the d/dt max for the fuel, since the fuel concentration should never be increasing. You may need to edit/add a new method to specify how ignition is determined a few lines below that too.


Allow notifications

This is a feature request that might be out of scope for the project, but it would be nice if I could POST some data to the Willow box and have it show up on the screen. This would be ideal for notifications and things like that, since it's already sitting on my desk.

Hi!
This is WELL within scope and on the roadmap - the ability to do something as simple as alarms and reminders relies on such functionality.
Broadly speaking, the current plan is to use websockets that support events in conjunction with either a Willow Application Server or a platform (Home Assistant, others) component that emulates WAS.
We may also support things such as generic HTTP post, MQTT, etc for such notifications but that has the potential to get very hairy very quickly and we would need to be very careful in how we approach such mechanisms.

Hmm, why might POST/MQTT get hairy? If you have a way to display notifications, you don't care how they came in, they all go into the same queue, no?

In our drive for performance, high quality audio, etc we are already pushing the fundamental physical capabilities of the ESP S3 the ESP BOX is based on. A $50 ready-made device comes with some compromises and those are rooted in being based on a microcontroller with extremely limited CPU, RAM, flash, etc.
Every single socket/thread/queue/task/etc needs to be carefully accounted for to only not crash the device, but also not interfere/compete with the primary goal of extremely accurate and reliable high quality wake-word and speech recognition.
We also want to ensure a long support lifetime - we have many planned features, users continually suggest more, and there's the unknown of what the road may bring in X years. We do and will continue to operate under tight tolerances and it would be very easy to make Willow unusable by haphazardly dropping in additional functionality.
Again, I'm not saying no, I'm saying it would need to be as carefully considered as everything else we have today has been so that we can continue to provide the high quality user experience we do today.

Ahh, yes, you mean from a performance perspective. I work with servers so I forget that, with embedded, the stuff Willow does is already basically magic.

Hah, yes. The ESP32 S3 has two 240 MHz cores with 512kb of IRAM and 8MB of additional PSRAM. 16MB of flash.
My machines otherwise have 512GB of RAM - needless to say very different situation!

@skorokithakis since you're running Willow 0.3.0-rc, did you also update WAS to use the :rc container image tag? This will give you notification support in a new tab named "Applications" in WAS.

@stintel I did, it works great, thank you.


[Backstage AI Assistant] Is this plugin already compatible with the new backend system?

I noticed that in the documentation the examples are in the old standard.
https://roadie.io/backstage/plugins/ai-assistant-rag-ai/
Feature Suggestion
If it is already compatible with the new backend system, we can update the documentation.
And if you still have to use the legacy form, how would the implementation be?

Maybe @Irma12 has some updates?


yarnConfigHook fails on Darwin, works on Linux if HOME is not writable

Describe the bug
Error is:
Executing yarnConfigHook
yarn config v1.22.22
warning Skipping preferred cache folder "/tmp/home/Library/Caches/Yarn" because it is not writable.
warning Selected the next writable cache folder in the list, will be "/private/tmp/nix-build-yao-pkg-6.1.1.drv-0/.yarn-cache-352".
error Error: EACCES: permission denied, open '/tmp/home/.yarnrc'
info Visit https://yarnpkg.com/en/docs/cli/config for documentation about this command.

Steps To Reproduce

Try to build e.g. #362297

I have seen this in lots of packages.
Expected behavior
Should just work.
Screenshots
Additional context
Metadata

Notify maintainers
none listed

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Have a few packages that I can't build right now due to this. Created a PR that resolved it for me locally if you want to test it out.


Convert process can still encounter decoding errors

Describe the bug
The issue at hand is similar to https://github.com/marimo-team/marimo/issues/540.
It concerns the conversion of jupyter notebook files in utf-8 format to marimo notebooks.
marimo convert my_notebook.ipynb > my_app.py
marimo convert my_notebook.ipynb -o my_app.py
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "<my_path>.venv\Scripts\marimo.exe_main.py", line 8, in
sys.exit(main())
~~~~^^
File "<my_path>.venv\Lib\site-packages\click\core.py", line 1157, in call
return self.main(*args, **kwargs)
~~~~~~~~~^^^^^^^^^^^^^^^^^
File "<my_path>.venv\Lib\site-packages\click\core.py", line 1078, in main
rv = self.invoke(ctx)
File "<my_path>.venv\Lib\site-packages\click\core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
File "<my_path>.venv\Lib\site-packages\click\core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "<my_path>.venv\Lib\site-packages\click\core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "<my_path>.venv\Lib\site-packages\marimo_cli\convert\commands.py", line 68, in convert
f.write(notebook)
~~~~~~~^^^^^^^^^^
File "C:\Users\mlohove\AppData\Roaming\uv\python\cpython-3.13.0-windows-x86_64-none\Lib\encodings\cp1252.py", line 19, in encode
return codecs.charmap_encode(input,self.errors,encoding_table)[0]
~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
UnicodeEncodeError: 'charmap' codec can't encode characters in position 395-397: character maps to
I was able to trace the error to
if output:
        # Make dirs if needed
        maybe_make_dirs(output)
        with open(output, "w") as f:
            f.write(notebook)
        echo(f"Converted notebook saved to {output}")
    else:
        echo(notebook)

in marimo/_cli/convert/commands.py
Replacing
with open(output, "w") as f:
with
open(output, "w", encoding="utf-8") as f:
Allowed me to convert my notebook.
marimo convert my_notebook.ipynb -o my_app.py
Converted notebook saved to my_app.py
Environment
{
"marimo": "0.9.28",
"OS": "Windows",
"OS Version": "11",
"Processor": "Intel64 Family 6 Model 142 Stepping 10, GenuineIntel",
"Python Version": "3.13.0",
"Binaries": {
"Browser": "131.0.6778.86",
"Node": "v18.16.0"
},
"Dependencies": {
"click": "8.1.7",
"docutils": "0.21.2",
"itsdangerous": "2.2.0",
"jedi": "0.19.1",
"markdown": "3.7",
"narwhals": "1.15.1",
"packaging": "24.1",
"psutil": "6.0.0",
"pygments": "2.18.0",
"pymdown-extensions": "10.12",
"pyyaml": "6.0.2",
"ruff": "0.8.1",
"starlette": "0.41.3",
"tomlkit": "0.13.2",
"typing-extensions": "4.12.2",
"uvicorn": "0.32.1",
"websockets": "14.1"
},
"Optional Dependencies": {
"pandas": "2.2.3",
"pyarrow": "18.1.0"
}
}
Code to reproduce
No response

thanks for the issue and fix! i will put up a PR for your suggested fix


Modifier -> SwiftUI -> Lists: listRowSeparator(_:edges:)

Doc: https://developer.apple.com/documentation/swiftui/view/listrowseparator(_:edges:)

[ ] Swift implementation
[ ] Elixir implementation

This modifier is already implemented.


Re-build the virsorter container, using one hosted in quay by our team.

Added 2 more cwl test for virsorter to cover this case and to avoid regressions in the future.
We need to port the tests to nextflow.. not sure how yet.
This PR will fix https://github.com/EBI-Metagenomics/emg-viral-pipeline/issues/93

Very cool! It looks like no test for nextflow, but cwl probably enough :)

Yes, we don't have unit tests for the nextflow code (yet). It was easier to improve the cwl tests for that tool

@hoelzer I'm merging this one as we need. I'll tag you in the dev -> master PR, we will add other bits to that one.


Example of using an Indicator - e.g. Candlesticks with EMA

Hi @richardeschloss - would you be kind enough to give an example on how to include an Indicator (e.g. ema) in a Highstocks chart? i.e. perhaps a similar example for a scenario like this?
Thanks!

Hi, here's a working example for Nuxt 2. Basically, import the indicators library and use it: (it's important to use the correct highcharts constructor for things to work)
https://stackblitz.com/edit/nuxt-starter-cvmkva?file=components/StockChart.vue
Integrating this directly into the nuxt-highcharts component factory is planned for the Nuxt3 compatible module (since Nuxt3 provides the much faster dev and user experience). That working branch is here:
https://github.com/richardeschloss/nuxt-highcharts/tree/feat/v3-indicators
https://github.com/richardeschloss/nuxt-highcharts/blob/83c6566262dfc4c58945c37e9524364143bdcc63/lib/components.js#L34

I just pushed feat/v3 with this support merged in. Using it with Nuxt3 is even easier...in that one just use it without any extra coding or imports.

npm i nuxt-highcharts@next # requires Nuxt3

Then just use highstock component:
<highstock :options="chartOpts" />

https://github.com/richardeschloss/nuxt-highcharts/blob/feat/v3/components/charts/StockChart.vue  (working example for Nuxt3 version)

Thanks @richardeschloss - much appreciated!


mldesigner: error: invalid choice: 'execute' (choose from 'generate')

Package Name: azure-ai-ml
Package Version: 0.1.0b6 , mldesigner: 0.1.0b6
Operating System: ubuntu
Python Version: 3.8.12
Describe the bug
When launching a job it promptly fails with an mldesigner error in the std_log.txt, reading
"
usage: mldesigner [-h] {generate} ...
mldesigner: error: invalid choice: 'execute' (choose from 'generate')
"
I was last able to launch the same pipelines this past Friday with no issue

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Storage:0.17655122,Service Bus:0.13826701,Machine Learning:0.061742257'

Hi @palmerss thanks for the feedback, we'll investigate asap. @azureml-github

Update on my part, reverting mldesigner back to  0.1.0b3 seems to allow me to launch pipelines again, however this also restricts me on the usage of optional arguments that mldesigner 0.1.0b6 addresses

Hi @palmerss, we have repro-ed this issue and currently working on a fix release. Sorry for the inconvenience casued, currently you can use this private build to mitigate:
pip install mldesigner==72212755 --extra-index-url=https://azuremlsdktestpypi.azureedge.net/sdk-cli-v2/

@0mza987 I'll give this a shot and report back within the next day

@0mza987 I'm unable to use the conda.yml with my environment, whenever try to launch the version of my environment that uses the conda.yml I get an error when launching that reads along the lines of
"Environment <environment_name> with version  is already registered and cannot be changed"
However I did double check that the docker image for my job is updated with both
mldesigner==0.1.0b6 AND azure-ai-ml==0.1.0b6
I am now getting a different error shortly launching in the std_log.txt that looks to be in
mldesigner/_utils.py line 223, in list_2_dict
`
key, value = item.split("=")
ValueError: too many values to unpack (expected 2)
`
keep in mind I'm using the python azure-ai-ml package and not the cli tool

Oh and of course let me know if I should just open this up as a separate issue and close this one out, because this may seem different enough.

@palmerss  can I get a look at your component command of the failed component? This error looks like the command is not in correct format which should be like "--inputs a=1 b=2", if you are trying to use optional parameter, make sure you pass nothing at all instead of passing an empty string when consuming the component.

And can you please show us the interface of your component definition and how you consume it like below example?

@0mza987 Sorry for the delay in response, have been pretty busy, before getting these screenshots for you I went ahead and tried updating azure-ai-ml to 0.1.0b7 with mldesigner at version 0.1.0b6, still am getting the same error, as mentioned above, reverting mldesigner back to 0.1.0b3 allows me to launch jobs for the time being, but doesn't allow me to make use of optional inputs
pipeline

Command Component

This issue has been resolved with the latest version of azure-ai-ml and mldesigner, as of the date of this comment, not sure what version it was actually fixed.


Xstream upgraded to v1.4.17 to fix CVE issues

Pull Request type

[X ] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Changes in this PR
xstream libs are upgraded to v1.4.17
Describe the new behavior from this PR, and why it's needed
Issue #2389
as per this link https://x-stream.github.io/changes.html, there are several CVE vulnerabilities in v14.13 which are fixed in latest v1.4.17.

Please resolve conflicts.

As per https://x-stream.github.io/changes.html, v1.4.18 has been released which addresses a few more CVEs. Please consider upgrading to this version.

I dont see xstream dependency in the latest main anymore.  We can close the bug then


Error:   LazyPRMstar: There are no valid initial states!

Hi,I encountered the same problem as # 4, but his solution does not apply to me.
I did not notice anything around the starting point, so I had to create a new problem and seek your help.
[ INFO] [1684482995.882166198]: planFromCurrentRobotPoseW
Grid map does not have "traversability" layer. Assuming traversable.
Invalidated 0 vertices
Invalidated 0/0 edges
[ INFO] [1684482996.122158127]: Get robot poseing
[ INFO] [1684482996.122209804]: getCurrentRobotPose
[ INFO] [1684482996.122242194]: pose_robot finished
Invalidated 0 vertices
Invalidated 0/0 edges
Initial start state not valid. Trying to sample around it.
Epic fail, dude
Warning: LazyPRMstar: Skipping invalid start state (invalid state)
at line 248 in /home/xin/下载/ompl-1.4.2/src/ompl/base/src/Planner.cpp
Debug:   LazyPRMstar: Discarded start state Compound state [
RealVectorState [-1.81222 -0.475486 -0.475]
SO3State [0 0 0 1]
]

Think you for your reply.I think i need to tell you same situation about my robot.
I have only one lidar on my robot back.So this lead to it can not see the environment beneath its feet，is lead my question?Can you help solve  this issue?

That would explain the issue.
You either need to teleoperate your robot until it stands on parts of the elevation map it has seen, or you set the unknown_space_untraversable parameter to false. This will make the planner assume that unknown space is traversable. It will then use image painting to fill these parts of the elevation map. This will reduce safety, though, so make sure that all obstacles are always seen by your lidar (especially negative obstacles, like holes or cliffs).

hello, I have set  unknown_space_untraversable parameter to false as you said, but I got this result.
The planner can plan in the previous second, but it still reports an error in the next second.
This is my terminal information.I don't know what is wrong.
[ INFO] [1685078106.207487167]: planFromCurrentRobotPoseW
[ WARN] [1685078106.208595293]: No new map received since last planning call.
Cleaned up for 810ms
Added 9 new milestones.
Planned 51 paths between 9 pairs and removed 43 edges
Invalidated 2 vertices
Invalidated 6/8866 edges
[ INFO] [1685078106.241065132]: Get robot poseing
[ INFO] [1685078106.241126606]: getCurrentRobotPose
[ INFO] [1685078106.241162343]: pose_robot finished
Invalidated 0 vertices
Invalidated 0/8860 edges
Initial start state not valid. Trying to sample around it.
x=0 y=0
Found valid start state offset by 1.96 0.313
Start position 1.96 0.313
Cleaned up for 42.1ms
Added 1 new milestones.
Planned 1 paths between 1 pairs and removed 0 edges
Info:    LazyPRMstar: Starting planning with 5519 states already in datastructure
Info:    LazyPRMstar: Created 0 states
Info:    No solution found after 0.038302 seconds
[ INFO] [1685078107.207644632]: planFromCurrentRobotPoseW
[ WARN] [1685078107.208408393]: No new map received since last planning call.
Cleaned up for 916ms
Added 1 new milestones.
Planned 14 paths between 1 pairs and removed 14 edges
Invalidated 3 vertices
Invalidated 7/8872 edges
[ INFO] [1685078107.280268269]: Get robot poseing
[ INFO] [1685078107.280333875]: getCurrentRobotPose
[ INFO] [1685078107.280405121]: pose_robot finished
Invalidated 0 vertices
Invalidated 0/8865 edges
Initial start state not valid. Trying to sample around it.
x=0 y=0
Epic fail, dude
Cleaned up for 36.3ms
Added 1 new milestones.
Planned 2 paths between 1 pairs and removed 2 edges
Warning: LazyPRMstar: Skipping invalid start state (invalid state)
at line 248 in /home/xin/下载/ompl-1.4.2/src/ompl/base/src/Planner.cpp
Debug:   LazyPRMstar: Discarded start state Compound state [
RealVectorState [-0.527231 -1.86758 -0.475]
SO3State [0 0 0 1]
]
Error:   LazyPRMstar: There are no valid initial states!
at line 513 in /home/xin/art_planner/src/art_planner/src/planners/lazy_prm_star_min_update.cpp
Info:    No solution found after 0.006294 seconds

And sometime "there are no solutions "or assume "There are no valid initial states!"

OK,it works now.Thank you very much.

And i have a question : does D the motion cost node  needs to be used with PRM_ Motion_ Cost used ,and other method does not need cost?
What is the difference between them?
Thank you for your work.

Happy to hear that it works.
The biggest difference is that prm_motion_cost uses a learned motion cost, while lazy_prm_star_min_update uses a simpler shortest path cost. As you say, only prm_motion_cost needs the motion cost node.
For details, please check the paper.

I have another question.I would like to know how you enable robots to fully explore independently in DARPA without the need to manually specify endpoints.

hello, I am sorry to ask you some other question.
I currently do not have a GPU to do landing point planning, so I would like to ask if there is a CPU only navigation algorithm. I saw a paper in your lab  about this.
M. Wermelinger, P. Fankhauser, R. Diethelm, P. Kr¨usi, R. Siegwart,
and M. Hutter, “Navigation planning for legged robots in challenging
terrain,” in IROS. IEEE, 2016, pp. 1184–1189.
I would like to know if the code for this paper is open source ？If it is, could you send it to me? My email is<EMAIL_ADDRESS>you very much.

For exploration, please check out GbPlanner.
The planner in this repo can run on CPU, without GPU, just fine. Simple change the planner/name parameter to lazy_prm_star_min_update. It's the planner published in this paper.
In fact, you were already running this planner on CPU alone as your screenshots show, so I'm confused by your request.

In lazy_prm_star_min_update the planning cost is shortest-path. It does not support a cost-map. It uses a traversability map to restrict where the robot can go, but it is not used in the optimization objective.

So if i want to use a cost map to planner with cpu ,how can i do?
Can this paper give me some help？
M. Wermelinger, P. Fankhauser, R. Diethelm, P. Kr¨usi, R. Siegwart,
and M. Hutter, “Navigation planning for legged robots in challenging
terrain,” in IROS. IEEE, 2016, pp. 1184–1189.


Added Google Analytics

Issue Reference
This PR addresses the Issue : Fixes #852
Summarize
Added Google Analytics code to templates/base.html

Also, mention the key points if any to look into while code reviews

Ahem.. who's gtag is that!

syam3526's

#852 has been closed considering the fact that google analytics will slow the site down.


Update prepare-for-upload-vhd-image.md

Corrected 1 MB to 1 Mebibytes
1 Mebibytes = 1048576 byte
so
"$vhd.Size % 1048576 " is correct
1MB is inconsistent with this statement.
It also fails to upload.
Disks in Azure must have a virtual size aligned to 1 MiB. If your VHD is a fraction of 1 MiB, you'll need to resize the disk to a multiple of 1 MiB. Disks that are fractions of a MiB cause errors when creating images from the uploaded VHD. To verify the size you can use the PowerShell Get-VHD cmdlet to show "Size", which must be a multiple of 1 MiB in Azure, and "FileSize", which will be equal to "Size" plus 512 bytes for the VHD footer.

@genlin
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


fix: the typo for youtube in readme

This PR contains the changes in Readme file for fixing the spelling of youtube

@musikalkemist, PR fixes a typo mistake, could you please review and merge


k_msgq API to peek the message at the head of the queue

Currently there are 2 primary APIs for the kernel message queues. _put() and _get().
In a few drivers, I could have avoided creating additional message queues if there was an API to read the message at the head of the queue without actually deleting the message from the queue.
For example while tracking data buffers through the driver from

application to the driver
within driver to the DMA descriptor
within driver from DMA descriptor to ISR
within driver from ISR to application callback

Since the message gets deleted from the queue upon _get(), another queue needs to be created for the different "hand-offs" within the driver.
What do people think of having a _peek() API for message queues?

if you come up with a patch I would be happy to review it

I'll give it a try. The userspace stuff is something I have to familiarize myself with before I can implement


[BUG] Incorrect preview in popup and live preview

Description

To Reproduce
$\{ x^{-1} \}$
Expected Behavior
Correct preview

I can't reproduce this:

Are there errors in the console? Can you paste your debug info here?


Update code for EDU

Title of ticket:
Description
Summarize issue
Update code for EDU to ECC.
Dependencies
Are there any dependencies?
Need to confirm that code is changing.
Need to update code in TEST AXIS.
DOD

[x] Update org code in FOI Mod

Confirmed code will be change to ECC.

@lmullane , I am moving Reporting Dashboard tasks/DOD with other existing task #3192
"Map new org code to for reporting. Combine EDU and ECC for dashboards."

As discussed, New Key Cloak group has been created to accommodate ECC code . On Prod, We will keep same ministry users in EDU Ministry Team and ECC Ministry Team. @lmullane , @m-prodan

Confirmed org code and 'Ministry Assigned To' updated in FOI Mod.
Confirmed ministry name updated in Angular app.
Closing ticket.

PROD RELEASE STEPS Taken

Created DB back up of Camunda, FOI DB

patroni-master-002-ffa-camunda_2023-01-10_15-41-16.sql.gz
patroni-master-002-foidb_2023-01-10_15-41-15.sql.gz

Backed images FormsFlowWeb and Requestmanagement api -002

oc tag forms-flow-web:prod forms-flow-web:prod-bu

oc tag request-management-api:prod-002 request-management-api:prod-002-bu

Tagging FormsFlowWeb and Request Managment API - 002

oc tag  request-management-api:test request-management-api:prod-002
oc tag  forms-flow-web:test forms-flow-web:prod


external packages has some problem

Hi ，I reinstalled my vim under the new linux computer, my go version1.11.2, the latest version of gocode, I found that the external package can not be completed, so I set g:deoplete#sources#go#source_importer, it works , but very slow, and the standard library can't be completed, but on my mac I'm using nsf/gocode and by setting autobuild true, it's very fast, I did a test.
On linux

On Mac

You can see that mdepsky/gocode is very slow on the completion of local external packages.

It is gocode issues instead of deoplete-go.
Please report it in gocode issues.


feat: port Data.List.Sublists

sublistsAux seems to be removed, and sublistsAux₁ also is the subject to be removed. I believe theorems related to those two in the statement should be removed, or note that it will be removed. I'll ask in Zulip.

Or sublistsAux₁ should not be removed, but should be moved to Sublists.lean.


[TC-177] changed psql to use URI for admin.pl in trafficops.

This fixes TC-177

can you change the title to mention admin.pl?   I want to be able to find this a year from now..

[--admuser (Postgres Admin User)] [--admpwd (Postgres Admin Password)] -- this requires a password on the cmd line -- not a good idea either,  since it ends up in the shell history.   what we talked about was having the user deal with being a user that has db admin privileges.  Don't you think that would be better?

The other thing to do is add the admin user and password as a new key in the dbconf.yml file?  Does this sound okay as an alternative?

no -- they should be able to make their own user able to connect as a db admin or create postgres user with the same.   dbconf.yml is for goose.  don't change that.   I think this aspect of admin.pl should not have changed.

okay.  I will try to do what you want when I have time.

we ended up taking a different direction for running these commands,   so this change isn't needed.  Please close.


modal does not work in vue 2.1.8

Modal shows "undefined undefined undefined" when running vue 2.1.8.
Works in 2.1.6.
Example:
https://jsfiddle.net/xmcLp8nt/2/

This is a bug in VueJs.  It is affecting the Alert component as well.  The bug is already fixed and should come out in 2.1.9.  (Guessing)
https://github.com/vuejs/vue/issues/4581

If persist the problem i'll rolback to 2.1.6 ... but not today. Happy new year 😃

If persist the problem i'll rolback to 2.1.6 ... but not today. Happy new year 😃

Issue continues into Vue 2.1.10 , issue related to vue-strap ? #41 Please let us know what the plans are to fixing this.

argh. this seems to be back because of the recent vue updates, probably just needs a new webpack build.


[Release-1.22] Unable to start secondary etcd nodes if initial cluster member is offline

Backport #4746 to release-1.22

Validated in k3s with RC and followed the steps as described in https://github.com/k3s-io/k3s/issues/4752#issuecomment-996509673. Additional tests include rebooting server 2 and server 3 first and the reboot server 1, observed the results as expected

This looks like it's due to the IP addresses changing from shutting down the nodes (default behavior in AWS). I think coredns pod was probably also crashlooping. So this looks good to me actually! I also quickly gave it a try using Digital Ocean to test this theory (which uses static IPs by default) and see the agent able to rejoin and none of the pods failing to start.


how to configure Google Tag Manager with Telescope ?

I Try to configure Google Tag Manager and i found  https://github.com/GorillaStack/meteor-iron-router-gtm for configure GTM but it was not working with FlowRouter.
Can you Please help me how we can configure GTM  with Telescope.

if(Meteor.isClient){
FlowRouter.triggers.enter( [ enterFunction ] );
FlowRouter.triggers.exit( [ exitFunction ] );
function enterFunction() {
$('body').append("<iframe src="//www.googletagmanager.com/ns.html?id=GTM-XXXX" height="0" width="0" style="display:none;visibility:hidden">")
}
function exitFunction() {
$('#gtmnoscript').remove();
$('#gtmscript').remove();
}
}
writing above code block it working for me. this is right way to handle this situation.
Thanks..

Hey, @jayeshdalwadi! Telescope has changed a lot in the past months, and there is a new React-based version, Telescope Nova (intro video).
The Legacy version is no longer supported, I'm closing this issue. I invite you to check the new version, it's awesome!
Happy hacking! 🚀
Ps: by the way, don't hesitate to join the Slack to ask any question, you are welcome! 👍


much faster block selection

Instead of walking the remote bitfield to find a fitting chunk this now once produces
bitfield = remote-blocks & !local-blocks

Then to select a block this bitfield is simply iterated (very fast, log32(n))
@pfrazee can you sanity check this for silly errors? it's v critical code

(@pfrazee tested this on irc for me with success)


APPS-700 Add release note for bugfix.

Adds missing release note for @mhrvol 's APPS-700 bugfix.

Please review and merge before the release. @jdidion  @mhrvol


A bug in the runGRASP.py

Hi Greema,
There is a bug in the runGRASP.py script

  File ~/git/GSFC-GRASP-Python-Interface/runGRASP.py:1251 in <listcomp>
    GasAbsorb = " ".join(['%14.10f' % n for n in allVals]) #Molecular depolarization ration if DP is used or Tau of LS is used

You might want to fix it before any run

@aninramesh what was the error message here? We should update the title to be a bit more specific.


Kotlinx Serialization Multiplatform Protobuf Generator

This is the tracking issue for https://github.com/marcoferrer/kroto-plus/pull/13
Originally work on this feature was blocked due to a bug with the @SerialInfo annotation in the  kotlin serialization native runtime. It looks like the bug is addressed in versions 0.10.0-eap-1 and 1.3.20-eap-52 with an expected release in 1.3.20

Development of pb-and-k has been slowed down due to the original developer having limited time. Therefore I think MP support for this project is something many would like.

Development of pb-and-k has been slowed down due to the original developer having limited time.

This is true...apologies, much of my day job has shifted away from Kotlin.

(..) It looks like the bug is addressed in versions 0.10.0-eap-1 and 1.3.20-eap-52 with an expected release in 1.3.20

We're past 1.3.20 \o/

Was there already work done on the kotlinx integration? @marcoferrer


BUG: Some reserved keywords are not identified properly

There is an issue in some reserved keywords tokens being printed.

The first row of reserved keywords print correctly (although IF, ELIF, and ELSE print weirdly sometimes). But after that, they usually aren't identified as keywords. The FOR row doesn't work at all, but the ones below sometimes work (although they don't print).

I know that it is not an issue with Lexer::iskeyword() -- the following code prints each keyword as expected:
char * setup = "print(\"Hello World\")";
Lexer lexer(&setup);
for (uint16_t i = 0; i <= DEL; i++) {
    lexemes x = (lexemes) i;
    char y[15] = "";
    char * z = (char *) y;
    uint16_t j = 0;
    while (names[x][j] != '\0') {
        *(y + j) = (names[x][j] >= 65 && names[x][j] <=90) ? (names[x][j] + 32) : (names[i][j]);
        j++;
    }
    xpd_puts(names[x]);
    xpd_putc(' ');
    xpd_puts(z);
    xpd_putc(' ');
    xpd_puts(names[lexer.iskeyword(&z)]);
    xpd_putc('\n');
}
return 0;

I have no clue why this error happens.

Resolved by not using the strcmp() function, which seems to have some unknown issues (it works when I test it directly, but fails in practice). Fixed by more manual code that I am not proud of.
I manually tested each and confirmed it all works.
After lexeme #72, undefined names are printed (OOM?). After #76, they are usually just printed blank.


Add option of using colon/semi-colon/curly brace

Thanks for working on this wonderful package!
I am wondering if you could add settings of whether to use colon/semi-colon, etc. that could override the auto-detection?
For example, I've been using stylus and would like to use colon between properties and values. Right now I have to get into the code to manually change it every time the package is updated. It'll be much better to have it in the settings instead.

It seems like you can force colons by setting "hayaku_CSS_syntax_no_colons": false in your sublime config (if you use sublime).


Add copy of Python-3.10.7/Lib as test input to fix find-omvll-config

These tests failed when running the test suite without OMVLL_PACKAGE set:

find-omvll-config-cwd.c
find-omvll-config-plugindir.c

In both cases Python scripts failed to execute, because the fake Python Libs folder didn't have the stdlib files. The same thing happens on Linux. I must have missed to run them from a new terminal last time (one that doesn't have OMVLL_PACKAGE set).
The only way I see to fix this quickly is to add the Python Lib contents as test input. This is 10M of text files, so this is not ideal. If we had more time we could check which of the files are actually used. (In the end all we run is an almost empty Python script.) Or should we download it at configuration time? Or maybe we add it as a resource to the plugin binray at some point? We will see. For the moment, I think this is acceptable. Eventually, we only every link a single Python version.
The review of the diff isn't ideal either. I don't see how to hide new files in the diff here on GitHub. The updated tests can be checked here as well:
https://github.com/open-obfuscator/o-mvll/tree/9972f155e438a6826201c279672888e55b896a0a/src/test/core/plugin

Hi @weliveindetail , I am taking a look at this, in order to merge it with https://github.com/open-obfuscator/o-mvll/pull/40
I am a bit confusing because of the following, I am able to execute the tests if the OMVLL_PYTHONPATH is set and this mentioned variable OMVLL_PACKAGE is not set, in my macOS, without doing any change on main branch
If I unset OMVLL_PYTHONPATH and comment the requirement, the tests are not working, but I believe there is no point to not have this variable set, isn't it?
Additionally if I apply you two fixes on the files, 2 out of 11 tests are working only, so still missing something.
I believe I am missing something you may have clear, can you clarify it for me? Maybe this is something that was happening but not anymore?
Thanks,
Sergi

Hi @marcobrador and @sevilS
Sorry for the delay, I was just taking a look today. It's been while, so I had to reproduce the tests myself and revisit the context.
TL;DR: Closing this is the right thing to do, but it might be worth thinking about a strategy for testing release packages. Let me still follow-up on your questions.

I believe there is no point to not have this [OMVLL_PYTHONPATH] variable set, isn't it?

This is correct, if we want to run the tests against the build tree (like in any regular CI testing). However, I think the idea at the time of this PR was to run the regression tests on the MVP package (like in release testing). I think this is where the OMVLL_PACKAGE variable originated from, but I don't find any reference to it in the sources. Did we have a ticket for this maybe?

I believe I am missing something you may have clear, can you clarify it for me? Maybe this is something that was happening but not anymore?

OMVLL_CONFIG points to the obfuscation configuration and each test passes a specific one explicitly.
OMVLL_PYTHONPATH points to the Python stdlib and for CI testing we set it explicitly as well.

IIRC omvll.yml provides exactly these two pieces of information and by setting them both upfront, we circumvent the whole process of finding omvll.yml. This is, however, a pretty critical aspect of a release test, especially since there are two distinct approaches right? From what I remember, we find omvll.yml by traversing parent directories from:
(1) CWD when testing development builds (it's somewhere in the build-tree)
(2) the plugin's directory in release builds (it's part of the MVP package)

Additionally if I apply you two fixes on the files, 2 out of 11 tests are working only, so still missing something.

So, having this reconstructed, it actually makes sense. The two tests that pass are the ones to find omvll.yml and thus cover the code for the above case (2) 🙂
PASS: O-MVLL Tests :: core/plugin/find-omvll-config-cwd.c (8 of 9)
PASS: O-MVLL Tests :: core/plugin/find-omvll-config-plugindir.c (9 of 9)

I guess back then this was "good enough" for the moment and we got derailed by higher-prio issues. Maybe we didn't get to the remaining infra for MVP package testing and this mysterious OMVLL_PACKAGE approach? Well, looking back today, it might have been worth spending at least the time to explain the context in more detail in the PR.


2.2 release missing table.update_field

Hi all - thank you for the amazing work on pyairtable!
I noticed that in the roadmap* you mentioned that the release 2.2 contains table.update_field() but I can't see it in the code (checked both 2.2.0 and 2.2.1):
https://github.com/gtalarico/pyairtable/blob/2.2.1/pyairtable/api/table.py
Probably just a typo in the milestone definition?
Thank you for the wonderful tool!

https://github.com/gtalarico/pyairtable/issues/249

Yes, fair point, the roadmap document isn't necessarily getting updated with the details of API implementation, it's more of a todo list. In this case, modifying an existing field is done through the .save() method on the schema definition objects:
>>> schema = table.schema()
>>> schema.name = "Renamed Table"
>>> schema.save()
>>> field = schema.field("Name")
>>> field.name = "Label"
>>> field.description = "The primary field on the table"
>>> field.save()

You can read more about working with metadata here: https://pyairtable.readthedocs.io/en/stable/metadata.html#modifying-existing-schema


Programming exercises: Fix image export in combination with tooltip text

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) and too complex database calls.
[x] I strictly followed the principle of data economy for all database calls.
[x] I strictly followed the server coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Changes affecting Programming Exercises

[x] High priority: I tested all changes and their related features with all corresponding user types on a test server configured with the integrated lifecycle setup (LocalVC and LocalCI).

Motivation and Context
Fixes https://github.com/ls1intum/Artemis/issues/9823
Description
With this PR, the Artemis Server now correctly extracts uploaded images from the markdown of exercises to include them in an exercise export, even when a tooltip text is specified. To achieve this, the code differentiates between two types of image URLs: those with tooltip text and those without. Using RegEx, it adjusts the corresponding substring operations based on the URL type.
Steps for Testing

Prerequisites:

1 Instructor
1 Programming Exercise

Create an exercise
Add an image to the exercise description (e.g. by dragging and dropping an image into the text area)
Add a tooltip text as described above, e.g. "ShownOnHover"
Save the exercise, and download it using the 'Download Exercise' button in the exercise's overview page
Unzip it, and look inside the files/ directory. It should contain the image

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance even for very large courses with more than 2000 students.
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance even for very large courses with more than 2000 students.

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Performance Tests

[ ] Test 1
[ ] Test 2

Summary by CodeRabbit

New Features

Enhanced handling of embedded file markdown syntax to support hover text in problem statements.

Bug Fixes

Added logging for export errors when the problem statement is null, improving error tracking.

Please unstale, I will recreate the PR in the near future


Exclude the temporary directory from scanning.

This fixes the case where the CodeQL Runner is downloaded on-the-fly to the source directory of the repository being analyzed. It gets copied to the toolcache which is outside of the working directory, but still remains in the repository directory too. We could delete it from the repository directory after copying it to the toolcache but a more robust solution seems like it would just be to ignore the temporary directory completely.
It's not ideal that people download the runner in this way because even compiled languages could take issue with unexpected files in the repository but it seems like people do so we should handle it as well as we can.
Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.

No worries! I'm glad it all works.


stdlib-list 0.8.0

Releases: https://github.com/jackmaney/python-stdlib-list/releases
License: https://github.com/jackmaney/python-stdlib-list/blob/v0.8.0/LICENSE
Requirements:

https://github.com/jackmaney/python-stdlib-list/blob/v0.8.0/setup.py
https://github.com/jackmaney/python-stdlib-list/blob/v0.8.0/requirements.txt

Actions:

Reset build number

Fix python in host and run

Add missing setuptools and wheel to host

Comment sphinx as it's not needed for runtime

Add pip check

Add license_family

Fix summary

Add description

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_40icg6vjj5/clone/recipe/meta.yaml:1: missing_python_build_tool: Python packages require a python build tool such as setuptools.
ERROR: /tmp/abs_40icg6vjj5/clone/recipe/meta.yaml:1: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_40icg6vjj5/clone/recipe/meta.yaml:1: missing_python: For pypi packages, python should be present in the host and run sections. Missing in host
INFO: /tmp/abs_40icg6vjj5/clone/recipe/meta.yaml:45: invalid_url: https://jackmaney.github.io/python-stdlib-list : URL domain redirect jackmaney.github.io ->  www.jackmaney.com
ERROR: /tmp/abs_40icg6vjj5/clone/recipe/meta.yaml:45: http_url: http://github.com/jackmaney/python-stdlib-list is not https
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_40dj56were/clone/recipe/meta.yaml:1: missing_python_build_tool: Python packages require a python build tool such as setuptools.
ERROR: /tmp/abs_40dj56were/clone/recipe/meta.yaml:1: missing_python: For pypi packages, python should be present in the host and run sections. Missing in host
ERROR: /tmp/abs_40dj56were/clone/recipe/meta.yaml:1: missing_wheel: For pypi packages, wheel should be present in the host section
INFO: /tmp/abs_40dj56were/clone/recipe/meta.yaml:45: invalid_url: https://jackmaney.github.io/python-stdlib-list : URL domain redirect jackmaney.github.io ->  www.jackmaney.com
ERROR: /tmp/abs_40dj56were/clone/recipe/meta.yaml:45: http_url: http://github.com/jackmaney/python-stdlib-list is not https
Errors were found


[BUG] Response is not disposable causing leaked TCP connections.

Describe the bug
Response<T> contains a disposable field but is not itself disposable. This is a huge problem because the object indirectly contains a HttpResponseMessage. HttpResponseMessage objects are one of the most important types to dispose because they hold on to TCP connections.
Exception or Stack Trace
N/A
To Reproduce
Try to put a Response<T> in a using.
Code Snippet
https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/core/Azure.Core/src/Response{T}.cs#L14
Expected behavior
Response<T> should be disposable so that HttpResponseMessage objects can be properly disposed.
Screenshots
N/A
Setup (please complete the following information):

OS: all
IDE : any
1.0.1

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Bug Description Added
[X] Repro Steps Added
[X] Setup information Added

The HttpResponseMessage contained in Response is guaranteed to be already disposed.

The it should not be accessible at all. If the message has been disposed then I shouldn't be able to read it.

What shouldn't be accessible?

The Response, from a Response<T> I should not be able to access the Response object, because it contains the HttpResponseMessage and the content stream, both of which are useless if the HttpResponseMessage has been disposed.

They are not useless, most of the time Response contains a buffered memory stream that is fully functional.

It still contains the HttpResponseMessage, and none of the code I saw to create a Response<T> guarantees that anything is disposed. An incorrectly written method that returns a Response<T> can very easily leak the responses. Normally you shouldn't dispose things that you return from a method, but you have to wrap the Response in a using despite returning it from the method to not write code that will cause problems running on a server.

An incorrectly written method that returns a Response can very easily leak the responses.

Agreed, but making Response disposable wouldn't fix this issue. Customers are even more likely to not dispose things.
That's exactly why we made a decision to keep Response non-disposable and handle complexity in clients instead.

Customers are even more likely to not dispose things.

If the type is disposable static analysis produces a warning/error for failing to dispose it. I think that is (sadly) ReSharper specific though. If the onus is on users of HttpPipeline to do the correct thing with Response objects then I guess its fine. It is just a very alarming experience as a dev when I put a Response<T> in a using and it doesn't work. I knew it was associated with a http response so it should be disposable to control the lifetime of the response.
Then I F12 to the implementation and see this Response field and see that it is disposable, but the parent class isn't. I then F12 on Response and see the HttpResponseMessage field and no way to dispose it from a Response<T> object. That raises a big red flag because the HttpResponseMessage is unavoidably leaked unless an already disposed instance was passed in. If the object was already disposed it should not be saved, that's just useless memory overhead and misleading when looking at the code.

@pakrym

That's exactly why we made a decision to keep Response non-disposable and handle complexity in clients instead.

Currently reviewing our dispose patterns and using in our applications since we sometimes have DisposedExceptions.
We have a pretty big throughput of blobs in our applications, does your comment means the Client ensure the response's stream are correctly disposed?
Currently, we're doing the following
var response = await blob.DownloadAsync(cancellationToken);
using (response.Value)
{
    return steamDownloader.GetValue(response.Value.Content);
}

Is this not advised?

@AlexandreArpin, you are doing the right thing here. DownloadAsync is one of the few client methods that return "live" network stream that has to be either read to the end or disposed.


broken database

Request
The problem is that in 2 sites that I have used the plugin, the database has been broken, generating problems with the menu and the content
Additional Context

Plugin Version: Version 1.0.0-beta.1
Operating System: Windows
Browser: Chrome (last update 08-09-2020)

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

@dgmdonna thanks for reporting and I'm sorry to hear about the trouble.

Did this occur after activating the plugin or with some action in particular?

Could you share your Site Health information so we can check for any conflicts?

A screenshot would be helpful too!

Before using the plug in we made a backup of the database, but it still broke the way it shows when you are logged into wordpress.
Site Health:
wp-core
version: 5.4.2
site_language: en_US
user_language: en_US
timezone: -05:00
permalink: /blog/%postname%/
https_status: true
user_registration: 0
default_comment_status: closed
multisite: false
user_count: 8
dotorg_communication: true
wp-paths-sizes
wordpress_path: /home/lttrmastrsite1/public_html
wordpress_size: 20.16 GB<PHONE_NUMBER>4 bytes)
uploads_path: /home/lttrmastrsite1/public_html/wp-content/uploads
uploads_size: 736.70 MB (772481139 bytes)
themes_path: /home/lttrmastrsite1/public_html/wp-content/themes
themes_size: 9.87 MB (10351126 bytes)
plugins_path: /home/lttrmastrsite1/public_html/wp-content/plugins
plugins_size: 89.54 MB (93885956 bytes)
database_size: 249.69 MB (261815218 bytes)
total_size: 21.22 GB<PHONE_NUMBER>3 bytes)
wp-active-theme
name: Brandon (brandon)
version: 1.1.0
author: Muffin group
author_website: http://muffingroup.com
parent_theme: none
theme_features: automatic-feed-links, post-thumbnails, menus, widgets
theme_path: /home/lttrmastrsite1/public_html/wp-content/themes/brandon
wp-themes-inactive (1)
Twenty Twenty: version: 1.4, author: the WordPress team
wp-plugins-active (21)
Accelerated Mobile Pages: version: <IP_ADDRESS>, author: Ahmed Kaludi, Mohammed Kaludi
Admin Columns: version: 4.1.6, author: AdminColumns.com
Advanced Custom Fields: version: 5.8.13, author: Elliot Condon
Akismet Anti-Spam: version: 4.1.6, author: Automattic
All In One SEO Pack Pro: version: 3.6.2, author: All in One SEO Team
All In One WP Security: version: 4.4.4, author: Tips and Tricks HQ, Peter Petreski, Ruhul, Ivy
AMP Pro Extension Manager: version: 1.0.9, author: AMPforWP Team
AMP Theme Framework: version: 1.0, author: AMPforWP Team
Brave Payments Verification: version: 1.0.4, author: Brave Software Intl
Classic Editor: version: 1.5, author: WordPress Contributors
EmbedPress: version: 2.6.0, author: WPDeveloper
HREFLANG Tags Lite: version: 1.9.6, author: DCGWS LLC
Imagify: version: 1.9.10, author: WP Media
Ninja Forms: version: <IP_ADDRESS>, author: Saturday Drive
Ninja Forms - File Uploads: version: 3.3.7, author: The WP Ninjas
Ninja Forms for AMP: version: 1.2.4, author: Ahmed Kaludi, Mohammed Kaludi
Simple 301 Redirects: version: 1.07, author: Scott Nellé
Simple Share Buttons Adder: version: 7.7.1, author: Simple Share Buttons
SSL Insecure Content Fixer: version: 2.7.2, author: WebAware
Web Stories: version: 1.0.0-beta.1, author: Google
WordPress Force HTTPS: version: 0.1.4, author: A. Kai Armstrong
wp-media
image_editor: WP_Image_Editor_Imagick
imagick_module_version: 1690
imagemagick_version: ImageMagick 6.9.10-68 Q16 x86_64 2020-04-01 https://imagemagick.org
imagick_limits:
imagick::RESOURCETYPE_AREA: 125 GB
imagick::RESOURCETYPE_DISK: 9.22337203685E+18
imagick::RESOURCETYPE_FILE: 12288
imagick::RESOURCETYPE_MAP: 125 GB
imagick::RESOURCETYPE_MEMORY: 63 GB
imagick::RESOURCETYPE_THREAD: 12
gd_version: bundled (2.1.0 compatible)
ghostscript_version: 9.25
wp-server
server_architecture: Linux 3.10.0-862.3.3.el7.x86_64 x86_64
httpd_software: Apache
php_version: 7.3.20 64bit
php_sapi: cgi-fcgi
max_input_variables: 1000
time_limit: 360
memory_limit: 1024M
max_input_time: 360
upload_max_size: 8192M
php_post_max_size: 16M
curl_version: 7.71.0 OpenSSL/1.1.1g
suhosin: false
imagick_availability: true
htaccess_extra_rules: true
wp-database
extension: mysqli
server_version: 5.7.31
client_version: mysqlnd 5.0.12-dev - 20150407 - $Id: 7cc7cc96e675f6d72e5cf0f267f48e167c2abb23 $
wp-constants
WP_HOME: https://www.looktotheright.com
WP_SITEURL: https://www.looktotheright.com
WP_CONTENT_DIR: /home/lttrmastrsite1/public_html/wp-content
WP_PLUGIN_DIR: /home/lttrmastrsite1/public_html/wp-content/plugins
WP_MAX_MEMORY_LIMIT: 256M
WP_DEBUG: false
WP_DEBUG_DISPLAY: true
WP_DEBUG_LOG: false
SCRIPT_DEBUG: false
WP_CACHE: false
CONCATENATE_SCRIPTS: undefined
COMPRESS_SCRIPTS: undefined
COMPRESS_CSS: undefined
WP_LOCAL_DEV: undefined
DB_CHARSET: utf8
DB_COLLATE: undefined
wp-filesystem
wordpress: writable
wp-content: writable
uploads: writable
plugins: writable
themes: writable
This is an image of how the site is now, even with the database restored.

@dgmdonna thanks so much for the details!
Since this is affecting your experience while logged in, I'm curious if this is a theme or plugin conflict.
Could you install the Health Check & Troubleshooting plugin, enable Troubleshooting mode (which affects your logged in user only) and let us know if the issue persists?

Sorry for the delay, now it looks like that because I restored the DB, otherwise part of the content had disappeared and in the AMP version it assigned a post to a page and there was no way to change that unless I was not editing the DB.
We are in the process of migrating to a new design, so I want to avoid this new event, because we find the proposal of the Story very interesting

@dgmdonna thank you for these additional details. I see you have several AMP extensions. I'll look into this further to see if they can be involved.

@dgmdonna just a follow up that I have not been able to replicate this issue, so it may be something specific in your environment. We've released the stable version 1.0.0 of the plugin, so feel free to try it (on a test site to be safe). Please don't hesitate to open a topic in our WordPress.org support forum if the issue should reoccur. It would be great if we can further troubleshoot to find the source.