OpenBLAS detection on Windows (and Linux) wanted

Hi,
currently, the CMake's FindBLAS package  does not detect the OpenBLAS library, which is the only one working for MS Windows:
https://ci.appveyor.com/project/bast/autocmake/build/1.0.413#L298
The previous math_libs.cmake contained working OpenBLAS detection. Would it be possible to have the corresponding openblas.cmake macro  ?
Also, we would need Linux CI service testing openblas library detection.
Is there a some flag for math-libs preference (like pick openblas instead of system_native blas or atlas blas) ? Is it possible to combine libraries (like --openblas --blas ) in case of missing routines ?

It is certainly possible and certainly the plan to have such a module.
I don't understand the question with the preference since you
then tell exactly what you want by the flags. And combining math libraries
is tricky. From the Autocmake perspective this is no problem but the code
has to then deal with multiple definitions.

Hi,
wrt combining libraries: for example, openblas does not contain the whole lapack. Therefore I would combine -openblas -lapack to supply missing lapack routines in openblas. Is it goog approach ?
Also, how about the automatic detection of MKL (ESSL)  libraries ?

I have created a forum to discuss things and will try to keep issues for real issues.
Please see http://autocmake.readthedocs.org/en/latest/general/help.html


Accessibility enhancement

Examine the level of accessibility provided in each component, and update it if necessary to comply with web standards.

I think that this is an important issue and i would like to see this implemented. Here are some observations i made, regarding keyboard control.
As far as i can see, some components, and therefore possibly entire pages, are not usable only via keyboard.
Some components are not focusable via tab. (e.g. Accordion, Navbar menu items)
Some components are focusable but not usable via keyboard. (e.g. Buttons and radio buttons cannot be pressed via enter or space)


Dependency with macros not working

Yo, I'm trying to make https://github.com/lloydmeta/enumeratum to work with bazel-deps:
  com.beachape:
    enumeratum:
      lang: scala
      version: "1.5.13"

And when trying to build, I'm getting:
error: macro implementation not found: findValues
(the most common reason for that is that you cannot use macro implementations in the same compilation run that defines them)

As per my understanding we're defaulting scala_import for any scala dependency, but there are situations where using https://github.com/bazelbuild/rules_scala/blob/master/docs/scala_macro_library.md or https://github.com/bazelbuild/rules_scala/blob/master/docs/scala_library.md would have a better fit. If my assumptions are correct, we could support this through an option kind:
  com.beachape:
    enumeratum:
      lang: scala
      kind: scala-macro
      version: "1.5.13"

take a look at:
https://github.com/bazelbuild/rules_scala/issues/445
https://github.com/bazelbuild/rules_scala/issues/366
There are subtleties with some macro set ups and open issues not resolved.
To your point: yes, many external libraries could be non-macro imported, which would run ijar on them and could result in better cache hits, but I'm skeptical of that. ijar removes the code, which can only increase cache hit rates. But when you bump a version of a jar, it is very rare that the ijar would wind up being exactly the same after that. Furthermore, bumping external dependencies is pretty rare.
So, my bias would be to simplicity here.
That said, I'm open to adding optional options if we can make it clear what problem we are solving.
Thanks for taking the time to open an issue.

I'm closing this as I agree with you design choice. And, in the above case,
importing like this does solves the issue:
  com.beachape:
    enumeratum-macros:
      lang: scala
      version: "1.5.10"
    enumeratum:
      exports:
        - "com.beachape:enumeratum-macros"
      lang: scala
      version: "1.5.13"


some kanji get dropped when converting KANJIDIC2

at least one kanji, '剝' , gets dropped when converting from kanjidic2.xml to the ZIP file. This kanji is clearly in the kanjidic2.xml file, but it's not in the converted ZIP file.

I looked into this a bit and there are some kanji that get dropped in the xml->yomidict conversion but '剝' isn't one of them. Here is the entry:
[
    "剝",
    "ハク ホク",
    "へ.ぐ へず.る む.く む.ける は.がれる は.ぐ は.げる は.がす",
    "jouyou",
    [
        "come off",
        "peel",
        "fade",
        "discolor"
    ],
    {
        "grade": "8",
        "halpern_kkd": "2105",
        "jis213": "1-15-94",
        "moro": "2049",
        "skip": "1-8-2",
        "strokes": "10",
        "ucs": "525D"
    }
],

The kanji that are skipped are because of the following checks in kanjidic.go:
func kanjidicExtractKanji(entry jmdict.KanjidicCharacter, language string) *dbKanji {
	if entry.ReadingMeaning == nil {
		return nil
	}
...
	if len(kanji.Meanings) == 0 {
		return nil
	}
...
}

which is probably fine for a J-E dictionary like this.
Can you elaborate on where it was you detected this issue (or anyone else with the same issue)?


Source folder not found when encrypting string resources

Hello,
I have a project setup to run and encrypt string resources located on a file root_project/module_name/src/main/res/values/secrets.xml. This works perfectly with:
a root_project/build.gradle that looks like
...

buildscript {
    repositories {
        jcenter()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:2.3.2'
        classpath "com.efraespada:stringobfuscatorplugin:0.5.0"
        ...
    }
}
apply plugin: com.efraespada.stringobfuscatorplugin.StringObfuscatorPlugin

and a root_project/app/build.gradle that looks like:
repositories {
    jcenter()
}

dependencies {
    compile "efraespada:androidstringobfuscator:0.5.0"
}

But in another machine, when running the same project, it seems that the plugin is not able to find the path to root_project/app/src/main/res/values. I can see this on the gradle console:
:app:backupStringResources
    source folder not found: src/main/res
    source folder not found: values
:app:encryptStringResources
    source folder not found: src/main/res
    source folder not found: values

Any idea on how to fix this? Thanks

@d-moreira it should work in different OS. Could you find the res folder on your project?

Hi @efraespada,
yes I can. The project is version controlled here on GitHub so the code is exactly the same on both machines. I'm thinking this issue may be related with some sort of setting, or something that is affecting the way the plugin finds the paths to the files... Do you have any idea?
On the same machine, on version 0.4.1, I couldn't use the plugin either. There was some error
when the plugin tried to run the AndroidStringObfuscator.jar file. This is the log:
FAILURE: Build failed with an exception.

* Where:
Build file 'path_to_proect/android-app/app/build.gradle' line: 178

* What went wrong:
Execution failed for task ':app:mergeDevelopmentDebugResources'.
> Process 'command '/Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Hope this can help you track the issue.

@d-moreira could it be a problem with Java? Type java -version on terminal, are you getting anything like this?
$ java -version
java version "1.8.0_152"

If it is not a Java problem, run this:
./gradlew lint --stacktrace

Tell me what you receive.

These are the logs on a machine with the problem and on a machine where everything is working.
failure.txt
success.txt

@d-moreira I can't see the problem 😅
Don't worry. I'll build a new version with a debug flag to print everything related with resources folder. Thanks for your patient.


Added local_hpc compatibility with Zimo's update

@mitkotak I just merged Taylor's commit which fixed flake8 errors and brought the latest v2 changes to this branch. Could you resolve the conflicts that caused with both of your PRs?


Allow deleting of commands.

e.g.
del app['foo']

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (848ce6c) 95.65% compared to head (bb3d799) 95.53%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #91      +/-   ##
==========================================
- Coverage   95.65%   95.53%   -0.12%
==========================================
  Files          16       16
  Lines        1702     1704       +2
  Branches      429      429
==========================================
  Hits         1628     1628
- Misses         36       37       +1
- Partials       38       39       +1

Flag
Coverage Δ

unittests
95.48% <100.00%> (-0.06%)
:arrow_down:

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

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


Add /api/v1/targets API

I've also restructured how the Sampler and Web component are created to allow for better extensibility in the future. This was needed to be able to pass the scrape.Manager to the Web -> API.
Some rough edges that I'm happy to follow up in the future.

Really awesome! :+1:


custom font

Hello,
i can't save the svg using the custom font, it don't work neither in the example.
Regards.

Duplicate of #24.


Error at startup

Describe the bug
Mod in and of itself entirely unusable in its current due to !!! error reporting on startup.
I installed the mod following the install instructions on the nexus mod page and after having installed it through the unity mod manager, I then launched the game and I see the mod manager window come up and immediately see both red light and !!! under status section.
To Reproduce
Steps to reproduce the behavior:

Install Toy Box 1.3.7
Open the mod manager window at any point after launching the game, whether in main menu or after loading a save/starting a new game
See red warning light and !!! error
See detailed error for further report on the actual issue

Expected behavior
Access to the mod settings and capabilities in its full functionality from within the mod manager window.
Save Files
To help us reproduce your issue, please attach a named save file
Please zip it or GitHub will not accept it.
No save file necessary for this issue.
Screenshots
If applicable, add screenshots to help explain your problem.
Settings
Please attach your settings.xml file from the mod folder
It appears I don't have such a file, not sure I'm able to provide something that doesn't exist.
Version Info:

ToyBox Version: 1.3.7
Windows 10
Version 1.01c

Additional context
From the player.log file I saw this about the problem:
[ToyBox] Version '1.3.7'. Loading.
Fallback handler could not load library E:/GOG Games/Pathfinder Wrath of the Righteous/Wrath_Data/Mono/data-000001DED2937070.dll
[ToyBox] [Error] Error trying to call 'ToyBox.Main.Load'.
[ToyBox] [Exception] TargetInvocationException - Exception has been thrown by the target of an invocation.
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> HarmonyLib.HarmonyException: Patching exception in method null ---> System.ArgumentException: Undefined target method for patch method static System.Void ToyBox.classes.MonkeyPatchin.BagOfPatches.SummonUnitsAfterArmyBattle_Patch::Prefix(Kingmaker.Armies.TacticalCombat.TacticalCombatResults& results)
at HarmonyLib.PatchClassProcessor.PatchWithAttributes (System.Reflection.MethodBase& lastOriginal) [0x00047] in <88b084d277f344aa9c2904c640976875>:0
at HarmonyLib.PatchClassProcessor.Patch () [0x00059] in <88b084d277f344aa9c2904c640976875>:0
--- End of inner exception stack trace ---
at ToyBox.Main.Load (UnityModManagerNet.UnityModManager+ModEntry modEntry) [0x000d9] in :0
at (wrapper managed-to-native) System.Reflection.MonoMethod.InternalInvoke(System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00032] in :0
--- End of inner exception stack trace ---
at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00048] in :0
at System.Reflection.MethodBase.Invoke (System.Object obj, System.Object[] parameters) [0x00000] in :0
at UnityModManagerNet.UnityModManager+ModEntry.Invoke (System.String namespaceClassnameMethodname, System.Object& result, System.Object[] param, System.Type[] types) [0x00011] in :0
[ToyBox] Not loaded.

Why are you still on game version 1.01c, 1.0.4d.02 is the current version.

I have the same issue and my game is even older, v1.0.0 (ha!)
I think settings.xml should appear in the toybox's folder after you've tried to activate the mod in the game. (I have mine, but I think it'd be more appropriate for you to upload, not me.)
I don't know how far back can one rollback. There doesn't seem to be former releases available here (nor NexusMod). If you installed the mod manually, go to
UnityModManager\Pathfinder Second Adventure
and you should be able to find the zipped files of prior versions. V1.2.6 still works.

Because the game changes the internal structure, the most recent patch only works with the "current" (give or take) version of the game. There is an update on NexusMods that has the version that broke and specifies that you need version 1.02g beyond that point.

@ArcaneTrixter Kudos for the tip! Hmm, I skimmed through a bit on Posts and the readme but couldn't find where such specification is... Doesn't matter! In the pinned post it does say
Important: Make sure you are on the latest version of the game 1.04d
BTW, the latest ToyBox that I can run is V1.3.0. Anything beyond will result in errors when starting up, for @vathrielrose 's reference.

Depends on what you want for features. There are quite a lot of new features that aren't available in older versions, and the recommendation is to update your  game.


unhide disable-downsampling flag in compactor

Signed-off-by: yeya24<EMAIL_ADDRESS>

[] CHANGELOG entry if change is relevant to the end user.

fix #1377
Do I have to remove the debug prefix of this flag?
Changes

Verification

Updated :smile: PTAL @bwplotka  @GiedriusS

Thank you! LGTM.


Add menuMaxHeight for DropdownButton

To satisfy our usage, I added menuMaxHeight. By default, the height of the menu does not change.
Some people want to control it by padding, some want to set the maximum height, some want to control it by height percentage, but if only the maximum height limit is increased, other needs can be calculated by menuMaxHeight.
close #23865
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Had to fix some trailing whitespace issues and a merge conflict. Should be good to go again as long as the bots are happy!

thx

It's really good to see this problem resolved!

In which D version of flutter will this feature appear?

In which version of flutter will this feature appear?

I see available for use in flutter 2.2.1+

while you were on this, why menuMaxWidth wasn't added? now with Desktop and Web support stable, I believe this is. a must. Some times we have designs that required a dropdown to take a big part of the width (almost by default since desktop is a wider aspect ration than mobile) and the menu that pops is then too big, you have almost no space left to cancel the menu selection.


[circt] Print CIRCT errors to stderr, not stdout

When CIRCT (MFC) hits an error, print the CIRCT error to stderr instead of stdout.  This is pragmatically done to work around an issue with mdoc where, when it fails, it only prints whatever is on stdout.  More basically, printing errors to stderr probably makes sense.
No test is included for this as tests of stdout/stderr are notoriously difficult to do in a thread-safe way that is compatible with ScalaTest.
Signed-off-by: Schuyler Eldridge<EMAIL_ADDRESS>

I'll hold off on merging this until I can verify that this is what mdoc wants... This gets the message to display when building the doc, but not to show up in the mdoc output. Digging...

what's the mdoc modifiers on the section you have? If it's :silent it won't show the output anyway... if it's :crash it expects an exception to be thrown and will show only the stack trace. You probably want :passthrough if you want to see the stderr/stdout output in your mdoc (if you aren't putting your dramaticError into an actual thrown exception). I am not sure if we often do :crash:passthrough but you could try that...? Or change this PR to put your message into the thrown exception.

As far as I can tell, there is no way to do this with how mdoc works unless maybe using a PreModifier. The PostModifers only give you access to what happens after the fact and usage of :crash seems to be wrapping certain statements with a doc.crash object so that this is specially handled. Basically, we're in a really weird place where mdoc seems to expect one of two use cases:

Watch for an exception.
Post-process the output.

For this to work as is, we need both.
Jack had a suggestion that we could instead use an exception here instead of printing to stderr. This would fix it and would likely be cleaner. The exceptions that come up here don't suffer from the problem of Chisel runtime exceptions that are buried in the stack trace. Anything CIRCT related would show up right at the top.
I'll rework this to change to CIRCT flavored exceptions with verbose text packed in their message.
WDYT?

Having the exception message with all the useful info sounds good to me.


saltpack.org makes it difficult to understand how to decrypt messages

My friend send me a message on Slack with a BEGIN KEYBASE SALTPACK ENCRYPTED MESSAGE prelude. I typed this phrase into Google, landed on saltpack.org and have been trying to figure out how to decrypt the message.
I've scanned the sidebar, read the intro, read the homepage, read https://saltpack.org/signing-format, read https://saltpack.org/implementations. I ran go get github.com/keybase/saltpack and expected to find a saltpack binary on my $PATH - I could then call saltpack -h and maybe learn about decryption options, but that also failed. I'd rather not write a Go main function to decrypt the message.
I expected to find something like this:
Decrypting messages with Saltpack
To decrypt messages, save the encrypted message to a file, then run keybase decrypt foo.bar or call saltpack decrypt file.name
or whatever other instructions I need.
Or a list item in the sidebar that says "decrypting messages", or something.

The real issue here is how to deal with key management, which as you might guess, is thorny...

I'm not so sure; I had (and have) a valid key, I just had no clue how to decrypt the message.

We can make an issue for this; it seems reasonable, but it's probably not going to get much attention in the short term. In the mean time, can you try the python version?

(or the full keybase binary)

The instruction I needed was keybase decrypt --infile <file>. I guess it would be nice if this instruction was more prominent or had a heading somewhere on the page.

I can't find any way to verify a Saltpack message online.
The first Google result is this issue https://www.google.co.uk/search?q=saltpack+decode
I just want a box people can paste a signed message into, which will then show the contents and verify the signer.

Updating the PGP verification box into a PGP/Saltpack verification box is one of our TODO's, yes. (In the other direction, it should be possible to encrypt anonymous messages from the browser too. Unfortunately, not being able to decrypt them in the browser is kind of inherently confusing, and we'll need to figure out how to present that to people gracefully.)

I can't find any way to verify a Saltpack message online.
Well, first you need to install keybase. Then in your terminal:
keybase verify -m "copy-paste your saltpack encrypted message"
If sender message is unknown you will get some error message like this

ERROR Verification failed: For user blablabla : sender key revoked
Then you need to force decryption with
keybase verify -f -m "saltpack encrypted message"

My friend send me a message on Slack with a BEGIN KEYBASE SALTPACK ENCRYPTED MESSAGE prelude. I typed this phrase into Google, landed on saltpack.org and have been trying to figure out how to decrypt the message.
I've scanned the sidebar, read the intro, read the homepage, read https://saltpack.org/signing-format, read https://saltpack.org/implementations. I ran go get github.com/keybase/saltpack and expected to find a saltpack binary on my $PATH - I could then call saltpack -h and maybe learn about decryption options, but that also failed. I'd rather not write a Go main function to decrypt the message.
I expected to find something like this:
Decrypting messages with Saltpack
To decrypt messages, save the encrypted message to a file, then run keybase decrypt foo.bar or call saltpack decrypt file.name
or whatever other instructions I need.
Or a list item in the sidebar that says "decrypting messages", or something.

@oconnor663


[WIP] Add new components and props to documentation

Pretty much ready I think, just waiting for the new release and then I just need to update package.json.

@prateekbh Can we get a new release?

@cmonteiro128 just published a new release

will merge this after sometime as it needs more changes

Right, still had to add the latest changes to the Textfield component, but I'll make a new pull request.


'VisionTransformer' object has no attribute 'dist_token'

When I fully follow the README file, when I run the py file, I report the following error“'VisionTransformer' object has no attribute 'dist_token' ”, how can I solve this.as follows:
Traceback (most recent call last):
File "main.py", line 102, in
main()
File "main.py", line 42, in main
model = CountingAnything(CFG)
File "/home/hao/桌面/LearningToCountAnything/models/CountingAnything.py", line 47, in init
self.backbone = ViTExtractor(vit_config)
File "/home/hao/桌面/LearningToCountAnything/models/backbone_vit.py", line 29, in init
self.create_from_base_model(
File "/home/hao/桌面/LearningToCountAnything/models/backbone_vit.py", line 47, in create_from_base_model
self.dist_token = base_model.dist_token
File "/root/anaconda3/envs/LearningToCountAnything/lib/python3.8/site-packages/torch/nn/modules/module.py", line 947, in getattr
raise AttributeError("'{}' object has no attribute '{}'".format(
AttributeError: 'VisionTransformer' object has no attribute 'dist_token'

I was not able to replicate this error.
We believe this may occur if using an incorrect version of timm, and that it can be fixed by removing the dist_token from the timm method as it is no longer needed.

Found the exact same problem. Maybe it would be interesting to update the environment.yml so it includes the correct version of timm, and also other libreries needed for the execution that are not initially installed such as opencv, scipy and sklearn.
Also, could not get over of this problem so as for today I still cannot run the --config example_training step

My apologies that you haven't been able to run it today, I have just updated the backbone_vit.py to remove the dist_token.
It should work now (and works on my clean env from the yml), let me know if you have any more problems.

another problem that appears when trying to run it is "no module named torch._six". I was able to overcome it by modifying the async_dataloader.py of pytorch lighning bolts so it doesnt use it. Updating plbolts might also solve that problem. I dont know why it happens to me though, if you tested it with clean yml


mesh check failed

The mesh check failed for a few examples as I tried just now. airfoil_2d, elbow and pump_volute are failed at checkMesh.
elbow: ***Number of edges not aligned with or perpendicular to non-empty directions: 40110

That's because the examples don't fully define every patch. BlockMesh sets defaultFaces (type empty) to block sides where patches are not specifically defined and checkMesh sees that as an 'error'. If you use mesh.set_default_patch() or manually add set_patch() on relevant block faces, this message will go away.

Thanks !! I changed the type of defaultFaces to patch in constant/polyMesh/boundary in case then the checkMesh was success.
naming the patch (say, for example,  "chained.set_outer_patch('pipeWall')" for the case of elbow.py) is the best way, I think. Great work Nejc. I'll keep exploring this !!


agm-map is not centered by [latitude] and [longitude]

Issue description
I am trying to have a map with a marker in the center of map.
<agm-map [latitude]="lat" [longitude]="lng" [zoom]="zoom" [scrollwheel]="false"
(mapReady)="onMapReady($event)">
I put the marker in the position of center of map (use the same value of 'lat' and 'lng'). But the marker is not shown at the center of map. It's slightly below the center.

I set the map height to fit to the height of the container (height: 100vh;), so the map is not a square. Not sure if that causes the problem.
how to get the correct center of map?
Thank you very much in advance for your help.

it seems the problem is caused by setting the height of map.


settings

So the minimap button has disapeared and theres no interface settings to turn them on.

You should have an option for Minimap Icon under General > Global.


ListItem: verify uif-type binding responds to changes

Verify that the uif-type binding is responding to changes on the bound scope. May just need to add a demo & test case.
types to monitor changes:

[ ] list type (grid / list)
[ ] list selection type (single / multiple)
[ ] list item rendering type (item / item with image / item with icon)

References #358

Switching to bug... three different type options within this directive are not monitored for changes when bound to scope.


A UDP-based SIP session is ended with TCP BYE

Hello Arsen,
A customer informed us about this bug. And here is the SIP messaging log coming from the customer:
sip.log
at line 86 you can see Via: SIP/2.0/TCP <IP_ADDRESS>:9060;branch=z9hG4bKX3m3QZ4QD64gH, in the BYE message that is sent by UniMRCP
although SIP session had been started with an INVITE having "Via: SIP/2.0/UDP <IP_ADDRESS>:5060;branch=z9hG4bK49796F5"

Hello Bayram,
I have checked the provided SIP message exchange and noticed that not all the SIP header fields are present in the original SIP INVITE. It is unlikely that Cisco-SIPGateway sends such a malformed INVITE message. Where is the SIP Contact header field?
Sofia-SIP sends a BYE message to the address/transport specified in Contact, which is in accordance with the specification.


How can stop  files retry without kill process?

Version:6.2.2
Operating System:Linux
When data with incorrect topic name try to send to kafka thru logstash, it will retry again and nerver give up, how can I stop this file retry without modify config or kill process and keep this files down?

I don't believe that there is a means today. However, see https://github.com/logstash-plugins/logstash-output-kafka/issues/167. Once the dead letter queue is enabled for this output, then it should be able stop the retries.
I am going to close this issue please re-open if I am misunderstanding your question.  Also please add any commentary for the dead letter queue requirements for this plugin over at https://github.com/logstash-plugins/logstash-output-kafka/issues/167.


Error from RVEA algorithm when the problem has no feasible solutions

APDSurvival in the RVEA algorithm appears to have an issue with not setting the nadir point when there are no feasible solutions.
On problems where there are no feasible solutions, the below error is produced as self.nadir is None:
  File "C:\Users\dis-2\anaconda3\envs\tuning\lib\site-packages\skopt\optimizer\gp.py", line 259, in gp_minimize
    return base_minimize(
  File "C:\Users\dis-2\anaconda3\envs\tuning\lib\site-packages\skopt\optimizer\base.py", line 299, in base_minimize
    next_y = func(next_x)
  File "C:\Users\dis-2\PycharmProjects\tuning_project\tuner.py", line 135, in objective_function
    res = minimize(problem,
  File "C:\Users\dis-2\anaconda3\envs\tuning\lib\site-packages\pymoo\optimize.py", line 67, in minimize
    res = algorithm.run()
  File "C:\Users\dis-2\anaconda3\envs\tuning\lib\site-packages\pymoo\core\algorithm.py", line 141, in run
    self.next()
  File "C:\Users\dis-2\anaconda3\envs\tuning\lib\site-packages\pymoo\core\algorithm.py", line 162, in next
    self.advance(infills=infills)
  File "C:\Users\dis-2\anaconda3\envs\tuning\lib\site-packages\pymoo\core\algorithm.py", line 228, in advance
    val = self._advance(infills=infills, **kwargs)
  File "C:\Users\dis-2\anaconda3\envs\tuning\lib\site-packages\pymoo\algorithms\moo\rvea.py", line 95, in _advance
    self.survival.adapt()
  File "C:\Users\dis-2\anaconda3\envs\tuning\lib\site-packages\pymoo\algorithms\moo\rvea.py", line 137, in adapt
    self.V = calc_V(calc_V(self.ref_dirs) * (self.nadir - self.ideal))
TypeError: unsupported operand type(s) for -: 'NoneType' and 'float'

The error originates in this method of the APDSurvival class:
    def adapt(self):
        self.V = calc_V(calc_V(self.ref_dirs) * (self.nadir - self.ideal))
        self.gamma = calc_gamma(self.V)

The nadir variable is set to None in the APDSurvival constructor. It appears to require the _do method to be run to assign a point to the variable. However, in the base Survival class, line 42, the _do method is not being run if there are no feasible solutions:
            if len(feas) == 0:
                survivors = Population()
            else:
                survivors = self._do(problem, pop[feas], *args, n_survive=min(len(feas), n_survive), **kwargs)

This prevents the nadir variable from being assigned a point when there are no feasible solutions, leaving it as None, and therefore causing the error seen above.
This error can be circumvented by not filtering out the infeasible solutions, i.e. by setting self.filter_infeasible to False.
To reproduce this error, you may use the following code, running RVEA on the MW1 problem:
from pymoo.algorithms.moo.rvea import RVEA
from pymoo.optimize import minimize
from pymoo.problems import get_problem
from pymoo.util.ref_dirs import get_reference_directions

problem = get_problem("mw1")
ref_dirs = get_reference_directions("das-dennis", problem.n_obj, n_partitions=12)

algorithm = RVEA(ref_dirs)

res = minimize(problem,
               algorithm,
               termination=('n_gen', 400),
               seed=1,
               verbose=False)

Can you check if the commit above fixes it!
Thanks for reporting this issue here.

Yes, that has fixed it.
Thanks for the quick turn around.


Integrate a Logger-Class with a namespace, maybe PSR3-Logger?

You use a "Logger"-Class without any namespace. This is very bad style for an external library and will break many projects (as it does with mine).

The Logger is from log4php; we haven't change anything at that source code, just copied it into the project. Unfortunately Log4php is still not using namespaces, so a update wouldn't fix it.
The library itself uses the FACTFinder\Util\LoggerInterface - always with namespace notation. Log4php is included into the project by default using the FACTFinder\Util\Log4PhpLogger implementation. If you have a different logger, you can create a similar LoggerInterface implementation and simply delete log4php.

Hi @rudibatt why do you close the ticket? Is a plan, to deliver a more robust library such a bad idea?

I closed it, because there is no solution here. The log4php logger can't be fixed as we won't change that code and if log4php shouldn't be used, this can be replaced in a project.
...or did I miss understand it wrong and you suggest to replace log4php per default?

I just checked PSR3-Logger and this looks very similar to our LoggerInterface and as far as I could see it does not provide a default logger; it's also just that LoggerInterface. So I don't get this issue..

Thx for your reply, but the real problem is, that you break the typical depency injection style with your logger class. For example FACTFinder\Adapter\Search will be injected with type-hinted objects , but the logger will be provided by class name and loaded internally.
(This behaviour leads to even more problems, if you try to load this factfincder lib with for example symfony dependency injection, you need to take additional steps.)
The missing namespaces makes matters worse. But i did not want to discuss DI styles till now - you can not be responsible for every possible DI-Framework there is - but using not secure namespaces is a responsibility of you.
PS: PSR3 was a suggestion for the refactoring of the used logger mechanics

Thanks for your explanation.
 Would a replacement of this kind of injection solve the problem?

+1
Maybe you could work with an optional setter-Injection for a logger. And an optional getter could check the logger property internally and lazy loads the default logger per default, if there was not one injected before.

I could try my best aswell. (without psr3 firstly)

Is there something new here? Or should i try my best?

Seems like do not have a maintainer here anymore... :(
So I guess you can go for it.


Git fetch. Access denied without enter login/password

VSCode Version: 1.39.0-insider
OS Version: Linux Ubuntu 18.04.3 LTS

Steps to Reproduce:

F1 -> Git: Fetch

Does this issue occur when all extensions are disabled?: Yes
When the Git: Fetch command is invoked, VSCode does not request a login / password for the repository, although the Push / Pull commands request a login / password. Then the command ends with the error Git: Missing or invalid credentials.
Output log:

If I use the terminal for the git fetch command then git requests for a login / password

Duplicate
#72613
#72615


Fixes Config/Revision update ordering.

Description
This PR puts a Revision update before a Config update in the update plan function of the plugin. This will help to avoid issues that prevent working with new config parameters, which are added in new charm revisions.
Fixes:
https://github.com/juju/terraform-provider-juju/issues/389
Type of change

Logic changes in resources (the API interaction with Juju has been changed)
(WIP) Change in tests (one or several tests have been changed)

Environment

Juju controller version:
3.3.1

Terraform version:
1.7.3

QA steps
Manual QA steps should be done to test this PR.
juju boostrap lxd lxd
juju add-model test

Apply plan:
terraform {
  required_providers {
    juju = {
      source  = "juju/juju"
      version = "0.11.1"
    }
  }
}
provider "juju" {}

resource "juju_application" "githubrunner" {
  name  = "github-runner"
  model = "test"

  charm {
    name     = "github-runner"
    revision = 8
    channel  = "latest/edge"
  }

  units = 1
}

Then update plan to
terraform {
  required_providers {
    juju = {
      source  = "juju/juju"
      version = "0.11.1"
    }
  }
}
provider "juju" {}

resource "juju_application" "githubrunner" {
  name  = "github-runner"
  model = "test"

  charm {
    name     = "github-runner"
    revision = 10
    channel  = "latest/edge"
  }

  config = {
    group = "mygroup"
   }

  units = 1
}

@anvial If you've rebased this week, some 2.9 tests will fail because juju 2.9.47 hasn't been released yet (see #405 ). Failures against juju 3.x should be investigated.

@anvial, the plans in the qa steps should either create the model, or make it a data source if you run juju add-model ahead. While the given plan will work, it's not a good TF form as it doesn't reference the model.

But I thought that I create a model but this resource definition in plan:
resource "juju_model" "this" {
  name = "{{.ModelName}}"
}
...

Where modelName is generated for every test.

But I thought that I create a model but this resource definition in plan:

@anvial I'm speaking of the QA steps in the PR description, not the ACC test you've written for the resource_application_test.go file.

But I thought that I create a model but this resource definition in plan:

@anvial I'm speaking of the QA steps in the PR description, not the ACC test you've written for the resource_application_test.go file.

Agreed,
Now fixed in the QA steps description also, but it looks like it should be fixed in both places because of the chosen charm for the test: "github-runner".


Consider optimizing readme for the marketplace

There's a bunch of repeated information when you view the extension on the marketplace or via VS Code:

This could look like this instead:

Hi! I have updated the ReadMe to conform to the requirements as outlined in the comment above. Let me know if it looks good, or any more changes are to be made.
Additionally, I feel the sub-headers "How to get started?" could be modified to the more concise "Getting Started" and "Extension" could be renamed to "Issues" as the content pertains to issues and I feel extensions sounds generic for a sub-header.
Let me know your thoughts on these, and I can add these changes into this PR too, if required!
Thanks!


Dependencies

After 2 hours of fixing different import errors, i have removed some specific versions from server requirements.
And my server container has run.
Follow lines has no versions:
Flask
Flask-Cors
flask-migrate
Flask-SQLAlchemy
This project is still relevant these days !!!

Here is the good requirements.txt:
flask==2.2.2
flask-sqlalchemy==3.0.2
Flask-Cors==3.0.10
flask-migrate==2.7.0
gunicorn==20.1.0
psycopg2-binary==2.8.6
Werkzeug==2.2.2

I just removed all versions in requirements.txt... then working as expected.
Flask Flask-Cors flask-migrate Flask-SQLAlchemy gunicorn psycopg2-binary Werkzeug

Verified!


Widget is not updated when task become "current tasks"

The widget should be updated if the start date or the due date of a task has passed.
The widget should determine the next upcoming start date or due date and set an alarm.

I don't know if I should create a new issue or reuse this one, but I have this exact problem on my Nexus 5 and Nexus 7 (Android 5.1, stock).
I get notifications for start and due date but the widget only updates on data change (new/edit/delete) but not when the start date or due date of a task passes.
It's the same on the default Google Now Launcher and Action Launcher 3.
Killing the Launcher will (of cause) also update the widget.

@wweich we'll look into that again.

I can still reproduce this issue, im my case on LineageOS 16.0-20200309 (Android 9) using a Honor 5X.
In addition, a task in the widget does not change its color to red when it becomes due, unless an update is triggered e.g. due to synchronization or by manually scrolling down and up through the list of tasks. At the same time, notifications work as expected.
I looked into the code and noticed that the TaskListWidgetUpdaterService only listens for ACTION_TIMEZONE_CHANGED and ACTION_TIME_CHANGED, whereas the notifications rely on ACTION_BROADCAST_TASK_DUE and ACTION_BROADCAST_TASK_STARTING. (According to an answer on StackOverflow ACTION_TIMEZONE_CHANGED and ACTION_TIME_CHANGED "are triggered only when the user manually changes time or date, and they are not automatically triggered by the system".)
Therefore I changed ActionService to also update all widgets when receiving ACTION_BROADCAST_TASK_DUE or ACTION_BROADCAST_TASK_STARTING. For me this change resolves the issue. I'll provide a pull request soon.

should be fixed now.


Attempt to connect to first correct port on dragging in visual shader

Currently, the user should manually connect them (after dragging). This fix will allow connecting sampler(or transform) secondary ports on dragging:

Thanks!


dynamo integration (replace backend data storage s3 with dynamodb)

DRAFT PR
Replace our data backend with dynamodb to remove s3 dependency mostly (after this PR only preview pages are stored in s3, and they probably should be in s3 going forward)
The changes are mostly:

include terraform resource to manage new data tables
entity class with pynamodb to define these tables in python
updated model.py logic to use dynamodb instead of s3
some minor updates to other part of code base

cleaning up git history and merge conflict


Couldn't load url

Hi; new to contributing to GitHub.
Everything was going fine with pageres with the three test pages I tried, but then I tried the following site- jalopnik.com- and it attempted to do it, creating a temp output file, but then said "Couldn't load url: http://jalopnik.com" so I went to the site in a browser, and noticed it's https, so I tried using https://www.jalopnik.com and it returned the same error.   This site is not the ultimate target, but I was merely trying it to see pageres in action again, but I'm glad I tried it before loading a batch of URLs into the tool.
Thoughts?
Using a daily build of Ubuntu 17.10 from a few days ago, in a VM.
Also, as an aside, is there anyway, I can load a txt file containing a list of URLs to pageres to grab screens from?
Thank you,
-Ryan

Have you tried another URL? This one is failing for me as well, no idea why but seems like a PhantomJS issue.

You can also use pageres-cli and use CLI commands to read and pass the data to pageres.

I tried your suggestion:
$ pageres < urls.txt
However, it did nothing. See below.  That file had 2 URLs in it.

As for another URL to try, I tried the parent company of Jalopnik, Kinja.com and that couldn't load either.  Whatever they're doing, it's causing failure with this tool.  That said, prob safe to assume that any site powered with Kinja will not work.... lifehacker.com doesn't work and it's powered by Kinja.
Thankfully, Kinja-based sites are not my goal.  The goal is to load a txt file with numerous URLs and have that processed.  I currently have a 180MB txt file filled with URLs.  Yes, MB. Just looking to grab screenshots of homepages so I can use them as initial profile/cover pics.
I don't mind splitting the file up, but I just want to be able to load "a" file into pageres to be processed.
thanks for help.
-ryan

I started trying part of the list of URLs, literally by copy pasting like 4 into the command line, and since one URL couldn't load, it broke the operation and the other 3 were not rendered.   I tried this with another batch and I deleted each URL it didn't like one by one til I only had 2 URLs that it was able to render, so 50% success rate.
A site from that list it couldn't load is manpower.com .  I found that the other site that didn't load actually couldn't load in a browser, so I think the site is dead or just a bad URL.
Anyway, that said, I need to find a way to have pageres skip errors and carry on, using txt files as input and if it could then use a batch of txt files, great, if not, no problem; I'll just have to manually start it again for the next file, for all 35 of them.


I hope the notebook can hide output, thanks

Feature: Notebook Editor, Interactive Window, Python Editor cells

Description
Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer, @joyceerhl

Thanks for the feature request. We actually have this already on our backlog:
https://github.com/microsoft/vscode-python/issues/7380


Added ability to specify shield style in opts

@scottdelly If you break the line in the code into 2, then I think we can get the test to pass. I will be glad to merge it!

@marcellodesales I've split the line :)

Coverage increased (+0.07%) to 92.174% when pulling c5aba4d4f7422d6a08ab852ca6505d99ec4074b2 on scottdelly:master into e4b60c1529e352521c14f08ead8a555ff1289740 on intuit:master.

@scottdelly Awesome! Thanks a lot! I will plan a release!


Add details for E1525

I've created a schematic and taken high resolution front and back pictures of the E1525 module.

I cherry-picked the commits and added them to the repository. I did resize the images a bit, otherwise Git would be very slow.
Thanks for the contribution.

I cherry-picked the commits and added them to the repository. I did resize the images a bit, otherwise Git would be very slow.
Thanks for the contribution.


Problems with sound and video (with impress-audio.js)

Hello
I have the following problems if someone could help me would be very nice. It only  must work with Chrome, Firefox and others we (I) can do later.

The video in the first slide does not start automatically. it used to work at the moment I don't know why it doesn't work anymore or onyl sometimes.

When the video starts automatically, the sound of the second slide is also played on the first slide and then on the second slide again

The console shows the following error
impress-audio.js:36 Uncaught TypeError: $el.parent is not a function
at impressObj.goto (impress-audio.js:36:20)
at impress.js:4478:17
at impress.js:1240:24

The link to the page would be
User: help
Passwd: ThankYou
Link to video error
ThankYou Rob
PS the content is created from a DB . generating it may take a while

Hello
Sorry for the late response, I currently have barely any time to work on projects other than a few select ones.
Have you tried using older versions of impress.js as well? I will add this to my ToDo list for beginning of December and I will look into this.

@Kuhlig
Is impress-audio.js even an officially supported plugin? Because I don't recall ever seeing any code in this or the extras repositories. After a quick search I've found this: https://github.com/danielsimons1/impress-audio
Is this the thing you've been using?
On another note, I'll think about creating an official plugin for this with the refactor as well, so any suggestions would be highly appreciated.

I was using this one ... https://github.com/danielsimons1/impress-audio ... thanks for your help curob

Hi
Unfortunately I didn't get past the password prompt

@Kuhlig We can't get past the pw prompt. Can you fix this please?


Fix example dependency version

Type of Change
feature or bug fix or documentation or others
API changed or not
Description
detail description
JIRA ticket: xxx
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

extension test: https://inteltf-jenk.sh.intel.com/job/ITREX-1.4-release-test/7/


Theme download does not respect directory setting in config

I currently have a folder structure of:
Client/
-- Theme1/
-- Theme2/
-- config.yml
Inside my config i have a directory set up for Theme1/ and running theme watch from the client directory works as expected. A file changed inside the Theme1 folder is updated on the store. Magic.
But when I run theme download from the same place it pulls down the theme into the current Client/ directory, not Theme1/.

@stewartknapman thanks! I will look into it.

What version of themekit are you using?

I cannot reproduce this on version 0.5.3 the most current version.

@tanema I'm also on version 0.5.3 I installed it fresh today.

Hey @stewartknapman The problem is that the directory config is not read in from the config file. I will fix this soon. You can workaround this by passing in the --directory flag.

@tanema Thanks for the update


Handle persistence of generic base classes

Library can export generic class from which user code then can derive or instantiate with specific arguments.

https://github.com/microsoft/python-language-server/pull/1459 actually implements this


Help About Methods

"CPP2IL failed to recover any usable IL for this method" how i could fix that i use GameAssembly.dll and just-give-me-dlls-asap-dammit

It means what the message says, there's no fix, maybe in the future the IL regeneration will impro

is there any way to dump specific method i really need one

I don't think you can with Cpp2IL currently if it couldn't recover, however can use reverse engineering tools like IDA/Ghidra/Binja/whatever, but I'm not guiding you through that


Error formatting numbered lists with code snippets

Given the following input:
# A B C

1. Test
    ```bash
    python -m pip install foo
    ```

2. Another
   For example:

    ```yaml
    repos:
      - repo: https://github.com/psf/black
        rev: v24.4
        hooks:
        - id: black
    ```

I get the following error:
The formatted Markdown renders to different HTML than the input Markdown. This
is likely a bug in mdformat. Please create an issue report here, including the
input Markdown: https://github.com/executablebooks/mdformat/issues

This is with the latest release from pypi.

Should now be resolved with v3.0.1


SQLite doesn't support foreign keys

I tried to execute following migration
        Create.Table("folder")
            .WithColumn("id").AsGuid().NotNullable().PrimaryKey()
            .WithColumn("name").AsString().NotNullable()
            .WithColumn("path").AsString().NotNullable();

        Create.Table("file")
            .WithColumn("id").AsGuid().NotNullable().PrimaryKey()
            .WithColumn("path").AsString().NotNullable().Unique("uq_file_path")
            .WithColumn("folder_id").AsGuid().NotNullable().ForeignKey("fk_file_folder", "folder", "id");

and I became following messages
[+] CreateTable folder
CREATE TABLE 'folder' ('id' UNIQUEIDENTIFIER NOT NULL, 'name' TEXT NOT NULL, 'path' TEXT NOT NULL, CONSTRAINT 'PK_folder' PRIMARY KEY ('id'))
=> 0s
[+] CreateTable file
CREATE TABLE 'file' ('id' UNIQUEIDENTIFIER NOT NULL, 'path' TEXT NOT NULL, 'folder_id' UNIQUEIDENTIFIER NOT NULL, CONSTRAINT 'PK_file' PRIMARY KEY ('id'))
=> 0s
[+] CreateIndex file (path)
CREATE UNIQUE INDEX 'uq_file_path' ON 'file' ('path' ASC)
=> 0s
[+] CreateForeignKey fk_file_folder file(folder_id) folder(id)
No SQL statement executed.
=> 0,001s
The foreign key 'fk_file_folder' wasn't added. I read that there is a lot of issue with altering the SQLite table. It isn't simply supported. I didn't want to alter the table. I want to create it with foreign keys. Is it possible to do it in a single step?

I have implemented the ability to add foreign key to sqlite tables when creating the table.
The only change is in SqliteGenerator.cs (https://github.com/ha11owed/fluentmigrator/blob/master/src/FluentMigrator.Runner/Generators/SQLite/SQLiteGenerator.cs)
I was unable to create a real unit test, but I tested with one of our migrations and seems to work fine:
return Create.Table(tableName)
.WithColumn("Id").AsInt64().PrimaryKey("PK_" + tableName).Identity()
.WithColumn("CreatedById").AsInt64().NotNullable().ForeignKey("FK_" + tableName.Replace("", "") + "Users_Created", "Users", "Id").OnDeleteOrUpdate(Rule.None)
.WithColumn("UpdatedById").AsInt64().NotNullable().ForeignKey("FK" + tableName.Replace("", "") + "_Users_Updated", "Users", "Id").OnDeleteOrUpdate(Rule.None)
.WithColumn("CreatedAt").AsDateTime().NotNullable()
.WithColumn("UpdatedAt").AsDateTime().NotNullable()
If anyone is interested in creating unit tests, I can commit my code, but I can't seem to manage to follow the testing methods used until now.

@ha11owed
I am interested in adding foreign key in sqlite.

Can I help you in doing unit tests ?

Hi, yes of course, I will clean the code up a bit and try to get it pushed. The problem with the tests is that they test more or less the internal methods of fluent migrator, and I don't quite know how that maps to the changes I have done

Fixed by #638


Default chunks=None creates unusable array

When using dask[array], most commands to create array objects claim to require a chunks= kwarg in the docs. However, a dask array object is successfully created with the default chunks=None. Then it fails when .compute() is called, with a less than informative traceback.
Minimal example:
import dask.array as da

has_chunks = da.random.random(size=(100, 100), chunks=(100, 100))
no_chunks = da.random.random(size=(100, 100))  # chunks left at default (None), succeeds

has_chunks.mean(axis=0).compute()  # succeeds
no_chunks.mean(axis=0).compute()  # fails - traceback not particularly informative (see below)

Unless some future functionality is intended to automatically set chunks if chunks=None, I'd propose this workflow should fail earlier at the point chunks is not provided to be consistent with the docs. Alternatively, the error when .compute() is called on an array with chunks=None could be more informative.
Traceback:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-22-f80a4f9b0fcd> in <module>()
      1 nparr = np.random.random(size=(100, 100))
      2 darr = da.ones((100, 100))
----> 3 darr.mean(axis=0).compute()

/src/anaconda/lib/python2.7/site-packages/dask/array/core.pyc in compute(self, **kwargs)
    721     @wraps(compute)
    722     def compute(self, **kwargs):
--> 723         result, = compute(self, **kwargs)
    724         return result
    725

/src/anaconda/lib/python2.7/site-packages/dask/array/core.pyc in compute(*args, **kwargs)
    514
    515     results2 = tuple(concatenate3(x) if arg.shape else unpack_singleton(x)
--> 516                      for x, arg in zip(results, args))
    517     return results2
    518

/src/anaconda/lib/python2.7/site-packages/dask/array/core.pyc in <genexpr>((x, arg))
    514
    515     results2 = tuple(concatenate3(x) if arg.shape else unpack_singleton(x)
--> 516                      for x, arg in zip(results, args))
    517     return results2
    518

/src/anaconda/lib/python2.7/site-packages/dask/array/core.pyc in concatenate3(arrays)
   2035     """
   2036     arrays = concrete(arrays)
-> 2037     ndim = ndimlist(arrays)
   2038     if not ndim:
   2039         return arrays

/src/anaconda/lib/python2.7/site-packages/dask/array/core.pyc in ndimlist(seq)
   2014         return 0
   2015     else:
-> 2016         return 1 + ndimlist(seq[0])
   2017
   2018

IndexError: list index out of range

Oh yes, indeed.  I can see how that would trip up almost any incoming user.

This definitely didn't just happen to me...

On it.

Since it appears dask_arr.chunks can dynamically be changed after creation, the best approach is probably to fail on computation.

Fixed in https://github.com/ContinuumIO/dask/pull/416
Regarding chunks changing after creation, I don't think I've ever seen this happen.  I also have a hard time thinking of a situation in which it wouldn't be catastrophic.  In my mind chunks is immutable.  This isn't at all enforced but I have yet to encounter a person who has tried to change it.  I suppose we could hook this up into a property that calls rechunk, but I'd rather wait until the person comes along who trips on this.

Thanks, @mrocklin!
Re: chunks changing after creation I just tried to do exactly that - the chunks property accepts any tuple then fails in a different, nasty way on .compute().

Sounds like that person just came along.  Changing to property + private variable

Fixed in https://github.com/mrocklin/dask/commit/b1cbc7278717726c165687d479d459320e0d42b1


WIP: Refactor resources to to fulfill index and show

This PR aims to solve that currently the Resources are only for Forms. It implements the logic to retrieve lists of Resources and display dynamic table in the frontend as well as dynamic show pages.
Now we have implement 3 components for each field. Index, Details and Form.

[x] New ResourcesIndex Vue component
[x] New ResourceDetails Vue component
[x]  Implemented a basic set of index and detail fields
[x] Refactored all the logic from a Form class to a Resource class.
[x]  New controllers for the index and show handling

Now the Resource expects an Eloquent model. In the first instance I thought it would be nice to be independent of Eloquent, but it gives us so many advantages such as pagination, and relation and so on.


feat: show users for each organization

Fix: #119
Signed-off-by: wasabi<EMAIL_ADDRESS>

demo link: http://test.rescld.cn
admin,123

Replaced by: https://github.com/casbin/casdoor/pull/121


Fix login

GitHub just changed their OAuth redirect, breaking our post-login URL cleanup flow. This PR:

Fixes URL cleanup to remove all query params
Skips the window reload on most modern browsers by using history API
Moves the "hidden" redirect action (still there for older browsers) to a visible action

I don't understand, what is the difference between the case where we need to redirect and where we don't?

@jonnor if browser has History API available, we can remove the stuff from URL without redirect. If browser doesn't, we need to redirect.


Embedding two screenshots after a failed scenario

Hi,
I am using Ruby + Cucumber + prettyFace (no PageObject) for testing with two browsers at the same time (two different users logged in to the same webpage and interacting - one user does some action, the second one verifies it, etc). When the scenario fails, I iterate through the list of users and take screenshot of their browser. So far so good (I end up with two screenshots).
But the problem is to embed them in the prettyFace report. I only see link for the screenshot that was embedded last. Seems to me that the first embed is overridden by the second one. My goal is to embed both of them so I can have better idea where the user interaction failed. Is there a way around it or only one screenshot can be embedded to the  report for each failed scenario?

Included in the main branch now.


If you could change Backbone at will, how would you?

Sooner or later, there will be a Backbone 2.0. Major releases are disruptive for users, so I prefer to make them as infrequently as possible. But when we do one anyway, we might as well use it to its full potential.
This is a "wild ideas" ticket, where you can suggest any backwards-incompatible changes that you would like to see in a future release of Backbone. Please keep it brief, but do explain why you would like to see the change(s) that you suggest.
You are also welcome to highlight anything that you want to stay the same. Again, please with reasons.
Some rules of the game:

State your reason for wanting the change (or non-change)! It does not need to be a "good" or "smart" reason.
All ideas are valid, bearing in mind that there are always tradeoffs. Please keep it very civil and do not enter competition with other suggesters over whose idea should "win".
The fact that an idea is mentioned here does not say anything about its likelyhood to make it into Backbone, even if that idea seems popular. In the end, the decisions are made by the people who do the work, with me and @jashkenas having the final word because we are publishing the package and we need to assure its quality.

The following prior tickets are somewhat related: #4274, #4245, #3560.

What to keep: It seems that backbone is the only MVC-Framework for JS that is still updated, so I would not change anything about that paradigm.
What to keep/Improve: Use of ESModules without introducing the Backbone global or a build tool #4274
What to improve: Not a huge problem for me, but using modern  JavaScript (not cutting edge, just anything that works since a couple of years) and, with that, make underscore optional? Many functional-style methods has been added to JavaScript arrays by now, so maybe this is sufficient? Similarly Map and Set might be helpful in some occasions. Reasons: Size and code-readability for people not familiar with underscore.

☝ I endorse these suggested changes for a Backbone 2.0.

So far, we have one response, which only suggests evolutionary changes, rather than anything radical. Would it be safe to conclude that Backbone users just want the library to be updated with recent JavaScript and otherwise stay mostly the same? If anyone thinks otherwise, please don't hesitate to respond.

Not radical, either but:

A standard way to manage child views might be great
I already pointed out that I would love a code update to ESM, ES6. In a similar vain: I wonder if code readability/learning/extending could be a focus in an update? That is not limited to code itself but also to documentation. It would not be a strong departure (though I find the event handling code difficult to understand sometimes) and might entail some potential compromizes where ease-of-understanding might be favored over performance, as well as architectural documentation (something between the line comments and the current documentation)

@jdittrich

A standard way to manage child views might be great

A possible radical way to implement this, would be to unify Backbone.View with the custom elements standard. In that case, child views are managed by declaring them in the template HTML. Child views managed by the parent itself are in the custom element's shadow DOM, while child views inserted from the outside would go in a <slot>. I have an idea for a total redesign of the library that involves this change and others (still preserving MVC and loose coupling through events), but I've been holding off from sharing that idea because it would be a lot of work and I'm not sure people are waiting for it.
A possible non-radical way would be to absorb the core functionality from backbone-fractal into Backbone itself. That would probably amount to introducing a new CompositeView class. Such a feature could be added without a 2.0 release.
Yet another option is to just leave it to external libraries, as is currently the case. Marionette provides its own way to compose views, so how to do it in Backbone itself would probably be somewhat controversial.

I already pointed out that I would love a code update to ESM, ES6. In a similar vain: I wonder if code readability/learning/extending could be a focus in an update? That is not limited to code itself but also to documentation. It would not be a strong departure (though I find the event handling code difficult to understand sometimes) and might entail some potential compromizes where ease-of-understanding might be favored over performance, as well as architectural documentation (something between the line comments and the current documentation)

Could you elaborate a bit on how you would like to see this intermediate documentation? In any case, pure documentation changes are never breaking, so this could be done before a 2.0 update already.
Perhaps related: I have been wanting to deepen the indentation, which might already improve the readability. Such a change would best go together with a modularization, because that keeps all obfuscating changes in one batch.

intermediate documentation

…for me would usually involve

What is already in "getting started"
And then, for the larger code modules how to read and understand the respective code. For event handling (since that is the part that I know best) something like: What are the contracts/interfaces for an __Api-Function/Reducer, how is this._events structured, which "private" attributes are added to an object that uses the Events mixin (These are all focussed on the current way events work)

JSDoc might help with the contracts/interfaces?

Basically, something that helps understanding how different parts of the code work together and how to read the code.

composite views

No strong opinion, for me the fractal integration sounds most meaningful.

@jdittrich I created a new issue (#4291) for your point about code understanding. It is a very good point and I think it does not need to (and perhaps should not) wait until the 2.0 release.

I haven't written any Backbone applications in ages, so take this with the appropriate amount of salt.
Change:

Good support for view composition. I liked Marionette back when I was using Backbone, but it looks to me like Marionette moved on from Backbone and then work stopped.
Some way to determine what inputs a model or view expects just by reading the model or view definition. My experience reading Backbone application code was that the "everything is a bag of attributes" approach meant that the source of truth for what a class needed was often several files away in one direction or the other. React propTypes is one example of solving this problem without static types.
Drop the underscore and jQuery dependencies unless they still provide significant value on modern browsers.

Keep:

MVC.
Relatively light weight.
Usable without bundling. (Shipping Backbone as an ES module is probably fine these days.)

Remove JQuery
Add life cycle handling, eg i may have thousands of view coming and going during the day, of which i can not be confident that i have the best approach with regards to memory management.
Keep it very similar with regards to not imposing a style.  Our implementation is probably unique to us and is only possible because backbone lets you build in your own way

Backbone has saved us a lot of time and give us a clear starting point.  Greatly appreciate your work!

I would push back against adding view functionality..  Marionette's team at one point intended on dropping backbone as a strict dependency as extending the view to allow for other renders was difficult.  A big ask had been dropping jQuery, and that meant making backbone optional.  Making that easier would be good.  In order to do it now it feels like a hack or just large copy-pastes from the backbone source.  You can generally see those modifications here: https://github.com/marionettejs/backbone.marionette/pull/3679
Beyond that ESM would be great..  not sure if we've overcome the breaking issues of switching to class as far as how that changes patterns overall.

I would push back against adding view functionality..

For utmost clarity: which added functionality are you referring to here?

absorb the core functionality from backbone-fractal in particular, but even the shadow DOM / custom elements.. backbone views should allow for that as a possibility without adding that as a required opinion.

@paulfalgout absorbing backbone-fractal would mean adding a new class (CompositeView) without changing anything about the original View base class. Would you still object to that?
The custom element thing is not so much a feature addition but a radical total redesign of the library. So far, I don't have the impression that a lot of people would want that.

It adds opinion to the library handled by other 3rd party libraries that now may be expected to support the new stuff.  It would make more sense to me if a significant portion of backbone users had already adopted it, but historically a light View layer opinion was characteristic of Backbone, and I'm not certain how at this point in time embedding a particular solution is any better than just saying "if you need to do X, here are a few options"

We had a big codebase with Backbone.js, Grunt for building, BEM for scoping CSS and plates for the view rendering. The latter was probably the biggest pain point.
We rewrote that application to ES6 + react + redux many years ago now. I still think we've got performance regressions from the rewrite, not in small part to React being hard to grasp even though the initial DSL seems small and simple - but rewrites, hooks and memos will eventually come get you.
I haven't used Backbone.js since our refactor, but have done quite a lot of work with web components. And I can't stop to think how Backbone.js combined with ES modules, template literals and web components would be such a delightful stack now!

@jgonggrijp: Our company has been heavily using Backbone for well over a decade, but (like most devs & companies) we no longer choose Backbone for new projects and we have been slowly converting large codebases to React. React's size, complexity and conceptual surface area are frightening and make it the polar opposite of Backbone ("smaller ... less conceptual surface area ... read and understand the source in an afternoon"). But React (and most recent UI frameworks) address a major pain point with Backbone: seamless view reactivity.
Backbone views are reactive (they typically re-render in response to model changes), but this is not seamless in two ways:

Coarse-grained Reactivity: When a Backbone View re-renders, it blows away anything stateful that the user is doing -- where their cursor is in a text field, the current scroll position of scrollable content, unsaved text, a drop-down menu being open, etc. Technically, Backbone is render-agnostic, so you could avoid these issues by rendering with a vdom (or using vdom-less lib that maps data directly to the DOM, as lit-html and solid.js do). While the Backbone docs explicitly say you can do this, it isn't exactly encouraged or common.

View Composition: Backbone doesn't provide any help with view composition, so the view either destroys & recreates child views on every re-render, or it has custom code to track and re-use child views. Sometimes this is trivial, but in a number of common situations, it is painful. One example is when you have child views that map to models in a collection. When one or more models are inserted or removed from the collection, the view needs to insert or remove child views at corresponding locations in the DOM.

The above shortcomings aren't that important when most changes are initiated by the current user, but we write applications for teams, so there are often incoming changes from other members of the team. These trigger re-renders, which either result in bugs or requires a good bit of code workaround to manage. In contrast, React & most recent UI frameworks handle this seamlessly (though I'll be the first to admit that these frameworks bring a boatload of other problems).
I do appreciate how Backbone is rendering-agnostic. Here are a couple possible ways to address the above issues while remaining render-agnostic:

In the docs, we could highlight one or more rendering layers that support fine-grained reactivity and view composition, and provide examples of how to use these with Backbone (or, if it requires more code than can comfortably fit in an example, link to it). lit-html supports fine-grained reactivity (without a vdom, so +1) and is fairly lightweight. But for composition, I think it requires that your View be a web component (I would prefer a solution that supports non-web component views on an equal footing with web component views).
We could write our own rendering layer (Backbone.Reactive?), perhaps even in a separate repo, that does fine-grained reactivity and composition. This would be optional -- it would be highlighted and mentioned in the docs, without discouraging or deprecating the use of other renderers (especially for situations when all you need is a simple _.template()).

Less important requests:

Drop jQuery in favor of fetch() and native DOM
Convert sync/fetch/save/destroy to async functions


feat: Allow sharing queues between tile layers.

Although you could sort of share the fetch queue between tile layers, it wouldn't really share properly without a custom needed function.  This explicitly supports sharing a fetch queue between tile layers.
This allows, for instance, multiple layers that access the same server to have a single fetch queue so that the number of requests against the server is limited (usually to 6) rather than that number multiplied by the number of layers.
This also adds an additional reference parameter to tile labels which prevents there from being a tile string collision between layers.
The multiframe tutorial now uses this.

@aashish24 This is intended to be the first of a few improvements that will allow better buffering of multiple tile sources.  Follow on work will include an explicit A/B buffered tile layer.


Updating the min agent version - There is a agent side change done fo…

…r showing the warning message in case machine group is used in task

:shipit:

ok then it looks fine


Updates for new Developer APIs for Health Records in Exporter 3.0

Do we have a parallel set of APIs for study developers that restrict those developers to studies they have access to through their organization? Is that possible with the v3 scheduler?

Do we have a parallel set of APIs for study developers that restrict those developers to studies they have access to through their organization? Is that possible with the v3 scheduler? I believe that would even take priority over the developer role going forward.

These APIs are used by Participants to get their own health records. Do we have a use case for study developers getting health records? If so, it should probably be limited to test_users only, and we should file a separate Jira for this.

Do we have a parallel set of APIs for study developers that restrict those developers to studies they have access to through their organization? Is that possible with the v3 scheduler? I believe that would even take priority over the developer role going forward.

These APIs are used by Participants to get their own health records. Do we have a use case for study developers getting health records? If so, it should probably be limited to test_users only, and we should file a separate Jira for this.

Great, no problems then.


Element remounted when prop 'static' changes

Hi there,
I am toggling the static attribute from a button on the element. The new static value is saved to the database (Firebase in my case) and gets pulled from there by a listener to the Redux state.
Normally, changes of the size or position position of the element do not trigger a complete remount. But changing the staticvalue triggers a remount every time. So first componentWillUnmount and then componentDidMount get executed every time. This leads to problems with an external non-react library I am using for connecting the different grid elements with arrows (jsPlumb).
Any idea how to solve this or why the remount is even happening?

This is an unfortunate side effect of how Draggable & Resizable work - because they wrap the GridItem, the tree changes if the item is static (no longer wrapped) and the component remounts.
I am hoping to fix this in a new major by exposing <GridItem> for direct use.

Alright, so I guess for now there is not much I can do against this behaviour.

Has anyone figured out a workaround for this? I need to set static on the smallest breakpoint, but I can't afford a remount.  Any idea how I could work around the remounting?


kVK_ANSI_ virtual keycodes don't match output in Dvorak or non-QWERTY international keyboard layouts

Cool and useful framework! This is just a heads up, not a request for an immediate fix or anything, about an issue I have some experience with having written some keyboard apps and being a Dvorak typist.
The kVK_ANSI_A virtual keycodes reference physical key locations on a standard QWERTY hardware keyboard. If the typist is using a different software keyboard layout, the key codes will no longer match the output letter specified in the name.
So tables like:
case .o: return UInt32(kVK_ANSI_O)
...
case UInt32(kVK_ANSI_O): self = .o

will only work for typists using a U.S. QWERTY keyboard.

Those constants with "ANSI" in the name are labeled according to the key position on an ANSI-standard US keyboard. For example, kVK_ANSI_A indicates the virtual keycode for the key with the letter 'A' in the US keyboard layout. Other keyboard layouts may have the 'A' key label on a different physical key; in this case, pressing 'A' will generate a different virtual keycode.

I wrote a bit about this on the Stack Overflow question here:
https://stackoverflow.com/questions/3202629/where-can-i-find-a-list-of-mac-virtual-key-codes/16125341#16125341
Some strategies for layout-agnostic conversion of keycodes to output are here:
https://stackoverflow.com/questions/1918841/how-to-convert-ascii-character-to-cgkeycode

I am developing a macOS desktop app that is localized in several languages. Is this issue going to affect all non-US users if I were to use HotKey?

@pkamb good call. I had no idea. I'll try to implement this when I have time.
@rgbworld I'd wager non-US keyboards would have trouble with this.

Curious to know whether any work was done on this?
I used HotKey and a user using Swiss-French keyboard complained about shortcut keys not working. Upon some investigation(by changing my keyboard layout in System Preferences, etc.), I figured that this is the issue.


Fix: Use defaultName when it exists to correctly stub async components

When mounting a component that has components defined with defineAsyncComponent the result of type.name is AsyncComponentWrapper. The defaultName contains the filename which should get precedence over the type.name to make stubbing work correctly.

@cexbrayat I just ran yarn run lint in the project in my branch but that has 100+ warnings. Seemed to also be indicated here: https://github.com/vuejs/vue-test-utils-next/runs/4445921989?check_suite_focus=true. Is that something I need to fix because I did not change anything in those files :)

@Evertvdw It looks like it fails in files you updated (tests/components/ScriptSetupWithChildren.vue). Run yarn lint:fix and you should be good to go. If you have other errors, make sure you have up-to-date dependencies, as it should not be the case?

Probably some git glitch, I ran the lint:fix and git changes said 105 changed files, but when comparing there were no differences for 104 out of 105. Fixed the one error that was there though :)


Bad link in "docs/git-config"

About in the middle of the page at http://git-scm.com/docs/git-config, there is a bad link called diff-config.txt.  Looks like another instance of #425?

more here:
http://git-scm.com/docs/git-merge
mergetools-merge.txt
etc

Bump. I noticed that http://git-scm.com/docs/git-config was missing some information at https://www.kernel.org/pub/software/scm/git/docs/git-config.html and I think this probably explains it.

Hey, is still an issue for anyone? If not I'd move to close.

Made a quick check - seems ok.


added MS Teams

Please refrain from pushing multiple features in a single PR. Keep MS Teams in this PR and create a separate one for Web Share API 🙏🏻

Sorry for the inconvenience. I don't have time to properly PR, etc.
MS Teams and WebShare API are ready on my fork. You can implement/merge them properly if you want.
Thanks!

Okay, I'll have a look and merge them separately. I hope I can get this out before the end of the week.

No need to rush. Thanks!

Merged in #66 in #67. Will release on npm shortly after

Released in v2.2.0 🎉


Mod Options Icon

Adds support for mod icons.

It only fails because I add an option that does not exist in ultimate-type-defs repo.

Closed because this has been implemented in c3ffad8713647bb5f566329ba4de3fd5e58d2fad


Uses a better hash function for Text

According to https://softwareengineering.stackexchange.com/a/145633 djb2 does reasonably well, certainly a lot better than what we have now.
Is this a breaking change? People might have stored these hashes in stable memory. Do we need to add a disclaimer that people shouldn't do that, or are we doomed to never improve our hash functions in base?

I'm not (yet) worried about this breaking anything either. Storing hashes in stable mem should probably be discouraged anyway (which doesn't mean that folks won't do it).

Great, thanks!

Though mentioning it in the changelog can't harm.


problem getting device id in callback

When I want to issue a serie of reads to several devices, the callback gives no way of telling which device is delivering result.
What to do?
var Client = require("owfs").Client;
var con = new Client('localhost');

var dev = [
        {'id':'/10.243EE400080086/temperature', 'topic':'/lolland/stuen'},
        {'id':'/10.8E6521010800E3/temperature', 'topic':'/lolland/vaerksted'},
        {'id':'/10.FD622101080073/temperature', 'topic':'/lolland/orangeri'},
        {'id':'/10.AFCE3503080058/temperature', 'topic':'/lolland/depot'},
]

for (i in dev) {
        con.read(dev[i].id, function(err, result) {
               console.log(dev[i].id,dev[i].topic, result);  //  this will not work!
        });
}

Hi,
You say "this will not work" - does it not work?
Or do you just think it won't won't?
I think it should work - in JavaScript dev[i].topic should be in scope.
It probably doesn't help, because it is a bit complex but this is the code I wrote that uses the owfs.js library:
https://github.com/njh/node-red-contrib-owfs/blob/master/owfs.js
nick.

I havn't tested it but something like this should work:
function lookupTemp(hostname, path) {

    var Client = require('owfs').Client
    var client = new Client(hostname, 4304)

    client.read(path, function (err, result) {
        if (err) {
            console.log(err)
        } else {
            console.log("path: " + path)
            console.log("result: " + result)
        }
    })

}

No it does not work!. It works for one device, but when issuing read for several devices, the path in the callback will be the last used path - for all events. The path must be delivered as an argument  in the callback to identify the path.

Would it help you if I created a working example?

See this example:
`var Client = require('owfs').Client
var client = new Client('localhost', 4304)
var dev = [
{'id':'/10.243EE400080086/temperature', 'topic':'/lolland/stuen'},
{'id':'/10.8E6521010800E3/temperature', 'topic':'/lolland/vaerksted'},
{'id':'/10.FD622101080073/temperature', 'topic':'/lolland/orangeri'},
{'id':'/10.AFCE3503080058/temperature', 'topic':'/lolland/depot',},
]
function lookupTemp(path) {
client.read(path, function (err, result) {
if (err) {
console.log(err)
} else {
console.log("path: " + path)
console.log("result: " + result)
}
})
}
for (i in dev) {
lookupTemp(dev[i].id);
}
The result is (correct!);path: /10.243EE400080086/temperature
result: 19.1875
path: /10.FD622101080073/temperature
result: 16.375
path: /10.8E6521010800E3/temperature
result: 16.5625
path: /10.AFCE3503080058/temperature
result: 19.875`
Then changing the code like this:
`var Client = require('owfs').Client
var client = new Client('localhost', 4304)
var dev = [
{'id':'/10.243EE400080086/temperature', 'topic':'/lolland/stuen'},
{'id':'/10.8E6521010800E3/temperature', 'topic':'/lolland/vaerksted'},
{'id':'/10.FD622101080073/temperature', 'topic':'/lolland/orangeri'},
{'id':'/10.AFCE3503080058/temperature', 'topic':'/lolland/depot',},
]
for (i in dev) {
var path = dev[i].id;
client.read(path, function (err, result) {
if (err) {
console.log(err)
} else {
console.log("path: " + path)
console.log("result: " + result)
}
})
}
`
The result is (I expected a duplication of the old result):
path: /10.AFCE3503080058/temperature result: 19.1875 path: /10.AFCE3503080058/temperature result: 16.625 path: /10.AFCE3503080058/temperature result: 19.875 path: /10.AFCE3503080058/temperature result: 16.25
Results are correct, buit path is duplicated.
I understand that this is async code. This is why you cannot count on the initial values in the callback.


Making the compass command configurable.

This makes it possible to not only run "compass compile", but also e.g. "compass watch".

When will this be merged in?

Closed via https://github.com/appleboy/gulp-compass/pull/96


Newbie - Atom not appearing in the Start Menu

I  recently downloaded and installed Atom.  It opened after it finished downloading.  However, I can't find it in the start menu, program list, and I can't make a shortcut to the desktop.  I can't even find a folder with a shortcut or program.  I just find the download file.
Is this normal?  Anyone have a solution?

Thanks for reaching out!
The setup program will install Atom to the local user account (i.e. under %LocalAppData%), you will then add the atom and apm commands to your PATH and create shortcuts on the desktop and in the start menu. As detailed in the installing Atom in Windows section of the Atom Flight Manual.
Because we treat our issues list as the Atom team's backlog, we close issues that are questions since they don't represent a task needing to be completed. For most questions about Atom there are a lot of options, check out:

The Atom Flight Manual
The Atom FAQ on Discuss
Discuss, the official Atom and Electron message board
Join the Atom Slack team

On Discuss and in the Atom Slack team, there are a bunch of helpful community members that should be willing to point you in the right direction.

Please make sure that you used AtomSetup.exe and not AtomSetup.msi (or their -x64 equivalents).


Adds HookLogger

This is the first pass for the HookLogger.
I used an event type payload. Hooks and events aren't the same but this works just fine.
Sample code:
add_action( 'init', function() {
    ray()->showHooks();

    get_option( 'home' );

    ray()->stopShowingHooks();
} );

Ray output:

Is it possible to change the label for the event type payload?

Very nice, could you also create a PR to the docs?

Very nice, could you also create a PR to the docs?

I'll review it in detail, and probably merge it, later tonight.

I'll review it in detail, and probably merge it, later tonight.

Very nice, could you also create a PR to the docs?

I will send PR for docs tomorrow morning.

Very nice, could you also create a PR to the docs?

I will send PR for docs tomorrow morning.

Thanks!

Thanks!


add rustfmt.toml to configure format

Closes https://github.com/deislabs/containerd-wasm-shims/issues/156
@radu-matei this overlaps with your PR #63

Could you please give it a review? @devigned @kate-goldenring


[table] column ordering slow

Describe the bug
The table updates “realtime” while the user is dragging a column and it can cause sluggish and buggy behaviour because of all the processing that is going on per render. See the attached gif

It is also possible to end up in a state where the columns position keep jumping back and fourth.

With the additional complexity/processing needed for table column resize this behaviour gets worse when hasResize is true.
To Reproduce

Choose a table story with hasColumnSelection:true
Click the column reorder button
Drag a column slowly across the as shown in the GIFs

Expected behavior
The best in my opinion would be to update the table on drop instead of on drag when rearranging the columns.
Environment/versions:

OS: MacOS
Browser: chrome

Looking into this, I believe the minimal code change for this is just one row. Changing
 hover(props, monitor) { .... to  drop(props, monitor) { .... in file ColumnHeaderSelect.js will solve this issue.
But we probably want to add some kind of visual feedback that indicates where the drop zones are. I assume that is well supported by react-dnd so that should also be easy.

@lukefirth @JordanWSmith15 is updating after drop ok for UX. Do we need to design some visual aspect to allow users to know reorder is happening?

I am fine with a "state of done" if we follow up with the enhancements soon.

:tada: This issue has been resolved in version 2.59.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


function not found getCoordinateRGBDepthImage

function not found getCoordinateRGBDepthImage

lookup for any update?

Got the same error.
I achieved what I needed by using this example: [https://github.com/ThomasLengeling/KinectPV2/tree/master/KinectPV2/examples/SkeletonColor]


Week 2 post on k8s volumes, mounts, and claims

Closing in favor of https://github.com/Azure/Cloud-Native/pull/89


Export UntrustedTransaction and fix documentation

UntrustedTransaction should be exported but haven't.

@ethereumproject/etcdev-team Please review.


Additional features: Go templating in messages and titles & dispatch alerts with error-message for misconfigured alerts or faulty templates

Hi!
I extended the bridge by adding support for Go templating in titles and messages for further customization. To be able to use alert values I also implemented a method to parse them from the valueString-object, sent by grafana.
As adding these templates to existing alerts can be difficult while getting familiar with the syntax, I also added a flag, so that errors   that occur during the translation to gotify, will be dispatched with additional information. This is helpful to debug the alert and furthermore that the alert will not be missed by the user.
If you have any questions or advice, feel free to comment or contact me.

Thank you for the review and great feedback!!!
I addressed every point and hope that I resolved them well. If you have any further questions feel free to ask!

@DRuggeri, how does @xRy-42's changes look? This looks like a great enhancement. Thanks.

Yikes, @seifer44 and @xRy-42 - I somehow missed the updates way back at the end of August! Really sorry for that.
Love the updates/changes - I'll merge and build now!


Update README.md

Just updating the root readme, feel free to add anything else on that may seem important.

LGTM. I like the nice instructions :+1:


Optimize the expression converter for better scan planning in iceberg

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] others

Which issues of this PR fixes ：

Fixes
Problem Summary(Required) ：

cherry-pick  #2946 #2792 to branch-2.1

run starrocks_fe_unittest_coverage

run starrocks_fe_unittest_coverage


Solarized Light Theme

Dash Bootstrap Templates is great! Would it be possible to incorporate the Solarized Light Theme?
Hexadecimal color codes are provided here:
Solarized Webpage.

I'm glad you like Dash Bootstrap Templates :slightly_smiling_face:
The Solarized LIght Theme looks like a nice theme, but the templates in this library are created from Bootstrap stylesheets.
You can find information on creating a figure template based on color codes in the Plotly docs here:  https://plotly.com/python/templates/


Creating an unsigned image with imgtool requires using the "sign" subcommand

Creating an unsigned image with imgtool requires using the "sign" subcommand. Either a new subcmd needs to be added, or the "sign" subcommand needs to renamed to "create".

Using create seems like a good idea!

When you "sign" without using a key, it is still putting a sha hash at the end. I'm not sure if create really captures this either, but I'm not sure what the best word would be.
If we did add another command, it should probably just be an alias, since sign does really make sense for the cases where we sign the image.
What do you think of #251 as a solution to this?

"What do you think of #251 as a solution to this?"
Review feedback given in #251.

how about a flag like -n (no sign, hash only)?

I don't think it would be helpful to have a flag to indicate no signature.  We just have to make the description clear enough. Any reviews on #251 would be appreciated.


[AVM Module Issue]: Log analytics workspace not enabled by default

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
(Optional) Module Version
No response
(Optional) Correlation Id
No response
Description
Diagnostic settings is not required by default. This is required to comply with https://azure.github.io/Azure-Proactive-Resiliency-Library-v2/azure-resources/Network/azureFirewalls/#monitor-azure-firewall-metrics
variable "diagnostic_settings" {
type = map(object({
name                                     = optional(string, null)
log_categories                           = optional(set(string), [])
log_groups                               = optional(set(string), ["allLogs"])
metric_categories                        = optional(set(string), ["AllMetrics"])
log_analytics_destination_type           = optional(string, "Dedicated")
workspace_resource_id                    = optional(string, null)
storage_account_resource_id              = optional(string, null)
event_hub_authorization_rule_resource_id = optional(string, null)
event_hub_name                           = optional(string, null)
marketplace_partner_resource_id          = optional(string, null)
}))
default  = {}
nullable = false

@maurojasmsft Diagnostics settings is set to false for nullable. I will create an example to show setting up monitoring for the Azure firewall. That should meet the requirements. The Diagnostic Settings variable is also part of the templates from the AVM team so following that guidance.


get data from json

I have a big menu, and i don't want to send everything in the html.
So, my idea is this:
i have a menu, and if I click on a item, it sends an get request to the server, that answers with JSON data, what is displayed in the next sub-menu.

i see that it is asked before, so i close this issue.


Cannot login with upper case letters in email

We should do case insensitive email matching

We seem to be lowercasing the email addresses when creating an account but not when logging in
create an account with a<EMAIL_ADDRESS>try to log in -<EMAIL_ADDRESS>doesn't work but<EMAIL_ADDRESS>does

RFC 5321 (2.4) says emails are<EMAIL_ADDRESS>some people prefer to use mixed case for their emails and especially their usernames
UX-wise we're better off being case-insensitive, especially regarding usernames, but we must never edit the user's information, so the .findOne calls we use should match case-insensitive.

I've taken out all lowercasing and marked the places this needs changing with // #664

also see #1100

so is this fixed? @gmarinov


SVG Export instead of PNG?

As the title says, is there an option to export the final image as SVG instead of PNG?
Thanks

+1

I want the same.


Implement the Project Member Endpoint

https://github.com/remsfal/remsfal-backend/blob/8631892d64abbc405a272e3152b144f220ee259c/remsfal-service/src/main/java/de/remsfal/service/boundary/ProjectResource.java#L88
https://github.com/remsfal/remsfal-backend/blob/8631892d64abbc405a272e3152b144f220ee259c/remsfal-service/src/main/java/de/remsfal/service/boundary/ProjectResource.java#L97
https://github.com/remsfal/remsfal-backend/blob/8631892d64abbc405a272e3152b144f220ee259c/remsfal-service/src/main/java/de/remsfal/service/boundary/ProjectResource.java#L106
https://github.com/remsfal/remsfal-backend/blob/8631892d64abbc405a272e3152b144f220ee259c/remsfal-service/src/main/java/de/remsfal/service/boundary/ProjectResource.java#L115

Guys, I have put together a project but there is no class ImmutableProjectJson.class Can you put it on?

@alexeyleping The ImmutableProjectJson.class is a generated class by Immutables. Hier you can find further information about Immutables: https://immutables.github.io/
If you are using IntelliJ, the follwoing information may be usefull for you: https://immutables.github.io/apt.html

Thanks, great advice, everything worked out

https://github.com/remsfal/remsfal-backend/pull/158


support also partial cubemap/fallback images: backgroundColor for missing face(s)

Similarly to the recently added support for missing tiles in multires panoramas,
this PR adds support for missing faces in cubemap (which includes fallback) panoramas.
Any areas for which no data is available are shown with the configurable backgroundColor.
I updated the documentation of backgroundColor accordingly.
This PR also includes a couple of minor improvements of related code.
This is in a sense a continuation of PR #563.

I've continued this branch with a major extension of utils/multires/generate.py to cylindrical input and partial panoramas.
The tile and fallback face files will only be produced when they actually contain image data.
For those files produced, the background color used past the edges of a partial pano can be defined.
I also added some generally useful options. Some options are just passed through in the .json file to the viewer.
Added options regarding partial panoramas:

-b --backgroundcolor: RGB triple of values [0, 1] defining background color shown past the edges of a partial panorama (defaults to "[0.0, 0.0, 0.0]")
-B --avoidbackground: viewer should limit view to avoid showing background, so using --backgroundcolor is not needed
-H --haov: horizontal angle of view (defaults to 360.0 for full panorama)
-V --vaov: vertical angle of view (defaults to 180.0 for full panorama)

Added options regarding cylindrical input:

-C --cylindrical: input projection is cylindrical (default is equirectangular)
-e --horizon: offset of the horizon in pixels (negative if above middle, defaults to 0)

Added general options:

-F --hfov: starting horizontal field of view (defaults to 100.0)
-O --voffset: starting pitch position (defaults to 0.0)
-f --fallbacksize: fallback tile size in pixels (defaults to 1024)
-a --autoload: automatically load panorama in viewer
-G --gpu: perform image remapping by nona on the GPU
-d --debug: debug mode (print status info and keep intermediate files

Thanks for your swift feedback;
I've just done all requested changes.

Great. Thanks!