Switch to tensorflow in setup.py/install_requires

Given that tensorflow is the new default for keras, it would make sense to switch the install_requires field as well.
As far as I can tell, theano doesn't seem to be a requirement when installing keras.

I forgot that tensorflow is not available from pypi.
This would complicate the setup.py script quite a bit , considering 2.7/3.5/linux/mac variations for the binary url.
Feel free to close the PR, if you don't want to introduce such complications.


Revisited Monad

This PR is - eeehh - a little bit bigger than expected...
Pro-tip: Never ever try to re-design monadic stuff throughout the whole code-base.

Code-reviews are welcome! I will leave this PR open for a few hours...

The MonadTests are deactivated now. I created a MonadTestMixin, which also extends MonadLaws. We need to solve #1028 in order to enable default interface method unit tests. There are libs out there to achieve this. Maybe junit-lambda (SNAPSHOT).
I pulled the 'dirty' flatMap code into a new method Monad.unit. This might come handy later when a Monad needs to cleanup system resources.
The Type conversion stuff now resides in Value. It should not be within the algebra layer.
You can ignore the changes within Tuple*. Tuple.transform and .map will be re-visited tomorrow, see #1025, #1027.
I started to fix some generic bounds in method signatures on my way through the monadic code. This should be done for other method signatures also. See #1029
I added BiFunctor which has bimap(Function, Function), implemented now by Validation, Map, Either, Either.LeftProjection, Either.RightProjection
I re-arranged some methods in alphabetic order (yes, I know - never re-format code in a PR...)
...

Fixes #1018 "Keep Monad clean, fix flatMap"
Fixes #1021 "When reduce* returns null reduce*Option must return Some(null) instead of None"
Fixes #1022 "Unify Either.bimap(Function,Funtion) vs. Functor2.map(Function, Function)"
Towards #1029 "Fix bounds of generic argument parameters in methods"

Current coverage is 94.56%

Merging #1031 into master will decrease coverage by -1.36% as of 0afa75f

@@            master   #1031   diff @@
======================================
  Files           76      75     -1
  Stmts         9314    9360    +46
  Branches      1732    1760    +28
  Methods          0       0
======================================
- Hit           8934    8851    -83
- Partial        113     125    +12
- Missed         267     384   +117

Review entire Coverage Diff as of 0afa75f

Powered by Codecov. Updated on successful CI builds.

Note: The Monad tests are temporarily deactivated. This probably is the reason for the decreased code coverage.

This probably is the reason for the decreased code coverage.

Probably?! Realy?! :)

@ruslansennov I try to be diplomatic :smile: - they will be re-activated soon.
I had to deactivate them because Value is no Monad any more. This is because our new Validation (which will extend Value soon) is an Applicative.
I will pull these things now to go on with the other issues. If you have any remarks, please comment on the code anyway...


Cleanup SPIs

This PR is to cleanup SPIs, starting with DatabaseService, and impls so that we can make them public with clear and simple contracts. E.g. this.release() instead of monkey-patched instances.

looks like the tests time out, what could be the reason?

Any chance to get this done?
As said last week when we reviewed: the longer we wait the harder it gets → 3 conflicts already today

Nicht notwendig für die release, aber halt bald... da es unsere APIs betrifft

Now, as we've all the other PRs in, would be great to merge that next as it affects Database Service Provider Interfaces / APIs (= SPIs)

If you enable auto merge it will trigger the tests again and if they pass will merge.


Bug on Dropdown.componentWillUnmount()

Simple bug on every component dependants on Dropdown (FormSelect, Select, etc...)
it happens since the last update
Error: Cannot read property '_bound' of undefined
TypeError: Cannot read property '_bound' of undefined
    at EventStack._unlisten (modules.js:47055)
    at EventStack.unsub (modules.js:47083)
    at Dropdown.componentWillUnmount (modules.js:48677)
    at modules.js:20397
    at measureLifeCyclePerf (modules.js:20064)
    at ReactCompositeComponentWrapper.unmountComponent (modules.js:20396)
    at Object.unmountComponent (modules.js:13040)
    at ReactCompositeComponentWrapper.unmountComponent (modules.js:20406)
    at Object.unmountComponent (modules.js:13040)

anybody has the same issue ?

I have not, but this may be related to #2117.  It is being released now as 0.74.2.  Let's see if it fixes it.

@MeXaaR please try the latest release, 0.74.2, and let us know if you're still seeing this bug.

Thanks guys, it work fine now
Great fix

@levithomason  I'm still seeing this on 0.74.2 and 0.75.1. Please see the trace below:
Error: Cannot read property '_bound' of undefined
TypeError: Cannot read property '_bound' of undefined
    at EventTarget._unlisten (http://localhost:3000/packages/modules.js?hash=0051c69b4aad0f180713cf5acfcf13ef81b7be36:91254:18)
    at EventTarget.unsub (http://localhost:3000/packages/modules.js?hash=0051c69b4aad0f180713cf5acfcf13ef81b7be36:91282:11)
    at EventStack.unsub (http://localhost:3000/packages/modules.js?hash=0051c69b4aad0f180713cf5acfcf13ef81b7be36:91125:19)
    at Dropdown.componentWillUnmount (http://localhost:3000/packages/modules.js?hash=0051c69b4aad0f180713cf5acfcf13ef81b7be36:99245:23)
    at http://localhost:3000/packages/modules.js?hash=0051c69b4aad0f180713cf5acfcf13ef81b7be36:39978:25
    at measureLifeCyclePerf (http://localhost:3000/packages/modules.js?hash=0051c69b4aad0f180713cf5acfcf13ef81b7be36:39645:12)
    at ReactCompositeComponentWrapper.unmountComponent (http://localhost:3000/packages/modules.js?hash=0051c69b4aad0f180713cf5acfcf13ef81b7be36:39977:11)
    at Object.unmountComponent (http://localhost:3000/packages/modules.js?hash=0051c69b4aad0f180713cf5acfcf13ef81b7be36:32715:22)
    at Object.obj.(anonymous function) [as unmountComponent] (chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/backend.js:8598:20)

As a workaround I disabled componentWillMount:
import {
  Dropdown as BaseDropdown,
} from 'semantic-ui-react';

class Dropdown extends BaseDropdown {
  componentWillUnmount() {
    // don't call the buggy code
    // See datails at https://github.com/Semantic-Org/Semantic-UI-React/issues/2120
  }
}

Please fill our issue template, we need the reproduction case and its example on the codesandbox. We will reopen the issue once we will have it.

+1
@layershifter can you work out the issue from the stack trace she provided? Seems like undefined reference kind of thing...

@mhagmajer Make codepen on sandbox and describe how we can reproduce it.

What browser is this happening in?  I'm unable to reproduce the issue.  We also have passing tests that remove handlers multiple times.  This covers removing undefined handlers.

can you work out the issue from the stack trace she provided? Seems like undefined reference kind of thing...

Not in this case, the error occurs in removeEventListener which is a browser function, not our code.  The thing is, this browser method also handles undefined handlers without throwing:

Without a reproduction case, we cannot tell why this is happening.  The proposed PR will only prevent the above workflows, removing an undefined handler.  However, as I've shown, this is not the problem.
Please submit a reproduction and we can help.


Add documentation for changing org. name and URL slug

https://app.intercom.io/a/apps/yzor8gyw/respond/inbox/conversation/11384277041

Followed up in https://github.com/appsignal/appsignal-docs/issues/165. Closing.


rekit cli not work as docs said

rekit create hello --clean does not create a clean app, rekit version is v2.0.3.

rekit add component home/hello -u hello does not add the route.

I check the source and known that rekit cli send command with url_path param to the rekit-core, but rekit-core is using urlPath as the route param.
Change addCmd.addArgument(['--url-path', '-u'], ...) to addCmd.addArgument(['--urlPath', '-u'], ...) in rekit.js fix the bug.

Thanks @wenris for reporting this. --clean command has not been supported yet. The second issue has been fixed.

--clean will be tracked by #24 , close this one.


add extra step for building pyauditor source distribution

This PR adds an extra step to build the pyauditor source distribution.

Codecov Report

Merging #665 (26a8b55) into main (3f6e423) will decrease coverage by 0.24%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #665      +/-   ##
==========================================
- Coverage   63.64%   63.41%   -0.24%
==========================================
  Files          45       44       -1
  Lines        5675     5270     -405
  Branches       89        0      -89
==========================================
- Hits         3612     3342     -270
+ Misses       2063     1928     -135

Components
Coverage Δ

apel_plugin
∅ <ø> (∅)

auditor
83.95% <ø> (ø)

slurm_collector
65.26% <ø> (ø)

slurm_epilog_collector
0.00% <ø> (ø)

htcondor_collector
∅ <ø> (∅)

priority_plugin
35.57% <ø> (ø)


iOS 9 status bar style "lightContent" not working

Now that the status bar is visible on iOS 9 I recognized that the status bar style set by the  view controller is ignored. My code is pretty simple:
`
override var prefersStatusBarHidden: Bool {
return false
}
override var preferredStatusBarStyle: UIStatusBarStyle {
return isDarkMode ? .lightContent : .default
}
`
The status bar style is switching on iOS 10 but not on iOS 9 (where it is always default).

Take a look at release v1.0.9.
It seems that the only way to fix this is configure status bar of GDPerformanceVIew root view controller. So now you can use something like this - 'GDPerformanceMonitor.sharedInstance.configureStatusBarAppearance(prefersStatusBarHidden: false, preferredStatusBarStyle: UIStatusBarStyle.lightContent)'


iwyu: missing includes in xla

Hi @selfisekai Can you please review this PR? Thank you!

rebased on top of current master, lgtm

Hi @ tdanyluk , Can you please review this PR?Thank you!

Hi @ tdanyluk , Can you please review this PR?Thank you!

Hi @tdanyluk , Can you please review this PR?Thank you!


[flutter_tools] Move _flutterGit to globals

As stated, moves _flutterGit from flutter_tools/lib/src/version.dart to flutter_tools/lib/src/globals_null_migrated.dart.
~Should only be merged after #79372 lands.~
~More context at https://github.com/flutter/flutter/pull/79372#discussion_r605062034.~
Edit: #79372 was reverted due to the change breaking the plugins repo: #83373
cc @christopherfujino should this require any tests?
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 signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

Google testing failures are Dart build failures to unrelated files that appear to be due to a bad merge internally. This PR should be good to go.

@christopherfujino since #73972 was reverted, we may as well merge this without it.

@christopherfujino since #79372 was reverted, we may as well merge this without it.

Gold has detected about 5 new digest(s) on patchset 1.
View them at https://flutter-gold.skia.org/cl/github/79568


Allow TriageView to be built standalone.

This allows TriageView to be built from dev or from BN_API_BUILD_EXAMPLES.

This was addressed in a previous commit: b8af211f9a3647687c7e45c0fd04b0768df87ffe


Initiate async task before test

How can I complete some async task before starting test
test('setup', function (t) {
  //complete some async task here then do the test. In my case test are getting executed before the async task is done.
});
test('GET /hello', function (t) {
  request(app)
    .get('/hello')
    .query({name:'asfd'})
    .expect(200)
    .expect('Content-Type', /json/)
    .end(function (err, res) {
      t.end();
    });
});

Use t.plan()
On Tue, Jun 20, 2017, 15:20 mustafamamun<EMAIL_ADDRESS>wrote:

How can I complete some async task before starting test
test('setup', function (t) {
//complete some async task here then do the test. In my case test are
getting executed before the async task is done.
});
test('GET /hello', function (t) {
request(app)
.get('/hello')
.query({name:'asfd'})
.expect(200)
.expect('Content-Type', /json/)
.end(function (err, res) {
t.end();
});
});
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/substack/tape/issues/373, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWlehB0ZJZjS_2GfiE-MRVouPCNX8q9ks5sF8ckgaJpZM4N_kuJ
.

Is there anything like before in mocha?

@mustafamamun no, there's definitely not, but since it's just JS, you can do this:
test('blah', function (t) {
  t.plan(1);
  someBeforeLogic();
  somethingAsync(function (error, result) {
    if (error) {
      t.fail(error);
    } else {
      t.ok(result);
    }
  });
});


Use sipa’s crypto library

Use this library.

As much as I appreciate libsecp256k1, I think that haskoin should feature a pure haskell implementation. The support for libsecp256k1 can still be added. Using a compilation flag for instance.

Given that you have already done most of the work, my suggestion would be to have a puresecp256k1 library with the exact same interface.

There is a problem with cryptographic code: you should probably never attempt to write your own unless you know exactly what you are doing. Our cryptography was implemented in a very naive way. It does not run in constant time, and we have not done any analysis to make sure it will stand against common cryptographic attacks. We are not cryptographers. On top of that, the code is extremely slow.
If somebody would step in and write a cryptographic library in Haskell for secp256k1 that is both fast and resilient to common attacks, we would definitely adopt it in preference of the C secp256k1 library. So far, the only implementations of secp256k1 that I think are generally OK from a security standpoint are OpenSSL and secp256k1. I don't know if Crypto++ implements this curve, but if it did I would also trust it, although linking C++ from Haskell is probably not as straightforward as it is to link C.
For these reasons, we are pretty much stuck with secp256k1. That a cryptographic library is pure does not mean that it is necessarily better. Cryptographic code has its own issues that go well beyond referential transparency and side-effect-free computation, and Haskell's lazy evaluation may end up hurting the safety of cryptographic code more than helping. Cryptography is better done using languages that can describe what the hardware will do precisely, such as C or assembly.
Also, in x86_64 secp256k1 uses assembly code optimizations that make it run very fast.

Isn't this done now?

Indeed, we can close this issue now.


chore: Fixes fat container welcome message

Description
Fixes grammar of fat container welcome message
Type of change

Bug fix

How Has This Been Tested?
It has been tested locally
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@sharat87 can you take a look at this


Enums

As @tonnenpinguin suggested, I have added the enums.
I don't know if we should split the drills between the frisbee and fitness.
Maybe this difference should more appears on the drillpage.js and the Drilllistpage.js

@guillaumemoby
You have three things to do before we can merge this PR:

Update your branch with master. When you create a new branch, always create it when you are on master and this one is up to date
Resolve conflicts. You can see when you do your PR that you have conflicts with master. It means we cannot merge it. To resolve conflicts, from enums branch, you need to do a git pull origin master to get last commits from master. Then, resolve conflicts, commit these changes and update your branch on Github
Isolate your last commit from the previous ones

I can help you or you ask to @sallesma

The easiest way might be to create a new branch from master and cherry pick the commit with your changes. Then submit the new branch and open a new PR.

I ve apply the change that you have suggest but I can't commit anymore because of the test.

@Martin est ce que je dois encore enlever " favoritesDrill: []," sachant que dans master il est aussi présent ?


Remove alert to Hipchat

Coverage remained the same at 80.717% when pulling 5657e6e95d4567f11c4cf9a093ba362416a3ded5 on feature/remove-hipchat-alert into 62cc1519d203ba313bd74b5c2aae3f050377a48b on develop.


Timeout error in mutePublishedTrack

Describe the bug
I am trying to mute audio of remote participant  using mutePublishedTrack but i everytime i am getting timout eror
Expected behavior
It should return success or error if anything goes wrong on livekit server
Device Info:

Java Version: [e.g. Java 11]
LiveKit Server SDK version: [e.g. 0.4.0]

Additional context
NA

@ShubhamJuIT please include snippet of the call you are making. are you using LiveKit Cloud?

Here is my code snippet. Yes i am using livekit cloud.
    public LivekitModels.TrackInfo mutePublishedTrack(LivekitRoom.MuteRoomTrackRequest request) {         Call<LivekitModels.TrackInfo> call = liveKitClient.mutePublishedTrack(request.getRoom(), request.getIdentity(), request.getTrackSid(), request.getMuted());         try {             Response<LivekitModels.TrackInfo> response = call.execute();             LivekitModels.TrackInfo rooms = response.body();             return rooms;         } catch (IOException e) {             log.error("Exception in muting participant", e);             return null;         }     }

Here is my code snipppet. Yes i am using livekit cloud.
public LivekitModels.TrackInfo mutePublishedTrack(LivekitRoom.MuteRoomTrackRequest request) {
    Call<LivekitModels.TrackInfo> call = liveKitClient.mutePublishedTrack(request.getRoom(), request.getIdentity(), request.getTrackSid(), request.getMuted());
    try {
        Response<LivekitModels.TrackInfo> response = call.execute();
        LivekitModels.TrackInfo rooms = response.body();
        return rooms;
    } catch (IOException e) {
        log.error("Exception in muting participant", e);
        return null;
    }
}

@ShubhamJuIT this seems like either a connectivity issue or a livekit cloud issue. Closing since it can't be reproduced locally.


Allow URL filter parts to be joined by either "OR" or "AND".

A proposed extension to UrlFilter model, based on one idea from issue 33.

Any feedback on this one?

I would like please to ask about this as well, such as whether there may be any changes needed for it. Thank you!


Fix codecov

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.90%. Comparing base (f9c1844) to head (3162c9f).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #75   +/-   ##
=======================================
  Coverage   97.90%   97.90%
=======================================
  Files           7        7
  Lines         287      287
=======================================
  Hits          281      281
  Misses          6        6

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


[Bug]: Last bit of a label is truncated on alignment change

Is there an existing issue for this?

[X] I have searched the existing issues

Description
This bug exists for all widgets that have a Label property.
When we change the Alignment of the label to the right and apply Italics to the Label Font Style, the last bit of the label gets truncated.
Even after increasing the width, the label does not get displayed properly.
https://www.loom.com/share/d714fa3cd85949e18aa6bb480e785228
Steps To Reproduce

D&D any widget that has a Label property e.g. Input widget.
In the property pane right align the label and add italics to it.

Expected: The label should be displayed completely and no part of it should be truncated.
Public Sample App
No response
Version
Cloud

@shadabbuchh @somangshu Let's close this bug as we have removed side label + alignment from the input widget?


NFDiv-2953 Welsh Translation for NFDiv-2914

https://tools.hmcts.net/jira/browse/NFDIV-2953

NFDiv-2953 Withdrawn


Fix pip install if torch isn't already installed

Hello !
Currently if you are working on a fresh new environment without torch installed, you have issues when installing this package:
$ pip install git+https://github.com/parlance/ctcdecode.git
Collecting git+https://github.com/parlance/ctcdecode.git
  Cloning https://github.com/parlance/ctcdecode.git to /tmp/pip-req-build-tu1xf42c
  Running command git clone --filter=blob:none -q https://github.com/parlance/ctcdecode.git /tmp/pip-req-build-tu1xf42c
  Resolved https://github.com/parlance/ctcdecode.git to commit c90ad94a0b19554f80804fb7812f2a1447a34a70
  Running command git submodule update --init --recursive -q
  Preparing metadata (setup.py) ... error
  ERROR: Command errored out with exit status 1:
   command: /afs/melodis.com/home/ahoorelbeke/venv/bin/python -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-tu1xf42c/setup.py'"'"'; __file__='"'"'/tmp/pip-req-build-tu1xf42c/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-wn60pvg7
       cwd: /tmp/pip-req-build-tu1xf42c/
  Complete output (5 lines):
  Traceback (most recent call last):
    File "<string>", line 1, in <module>
    File "/tmp/pip-req-build-tu1xf42c/setup.py", line 10, in <module>
      from torch.utils.cpp_extension import BuildExtension, CppExtension, include_paths
  ModuleNotFoundError: No module named 'torch'
  ----------------------------------------
WARNING: Discarding git+https://github.com/parlance/ctcdecode.git. Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

This PR suggests different improvements:

pip install in a single line, useful to add this package in a requirements.txt for instance
add torch to install_requires in setup.py so that torch is installed after the pip install
add torch to requires in pyproject.toml so that torch is present before calling the setup.py and we avoid the error above

You can test this by doing:
pip install git+https://github.com/anth2o/ctcdecode.git

Can you list the dependencies like torch version, c++ and others
Also I am facing same error in the windows. Will it be different for linux based ?


Current node not updated when processing HideEntity regex (c#)?

Hey,
In DataHandler.cs, there is:
match = Regexes.ActionHideEntityRegex.Match(data);
if(match.Success)
{
	var rawEntity = match.Groups[1].Value;
	var tagName = match.Groups[2].Value;
	var value = match.Groups[3].Value;
	var entity = Helper.ParseEntity(rawEntity, state);
	var zone = Helper.ParseTag(tagName, value);
	var hideEntity = new HideEntity {Entity = entity, Zone = zone.Value, TimeStamp = timestamp};
	if(state.Node.Type == typeof(Game))
		((Game)state.Node.Object).Data.Add(hideEntity);
	else if(state.Node.Type == typeof(Action))
		((Action)state.Node.Object).Data.Add(hideEntity);
	else
		throw new Exception("Invalid node: " + state.Node.Type);
	return;
}

Contrary to all the other processes, there is no call to state.UpdateCurrentNode(typeof(Game), typeof(Action)); before checking the type of the current node. Is this intentional?

HideEntity can not have child elements, so I think that was intentional, yes. Does that behaviour result in any problems?

Closing bugs from deprecated c# interface.


add optional details to fridays

pizza type
games
maximum number of guests
etc.

details added in v0.9.0


Some discussion

... to continue the discussion at stackexchange
I dont really get your question. Of corse the noun will change with respect to the case,
Nominativ | der Abschnitt
Genitiv      | des Abschnitts / Abschnittes
Dativ         | dem Abschnitt
Akkusativ  | den Abschnitt
so it is not invariable with respect to the case. But this is so obvious, that i dont think that this was youre question...

I was thinking of the particular situation where no article is used
(form = prep+noun). I was unsure whether the whole could depend on the
case, because I was only thinking about genitive and couldn't find a
meaningful example; but I think I have one now, with plural dative: one
could say “in Abschnitten 3 und 4”, right? If so, this means the code
has to know which case is used (nom/akk/dat/gen) even if the form is
prep+noun—which is what I wanted to know.


feat(parser): report class properties that are both definite and optional

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5181  👈
#5180
#5179
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite


Shared Compendium

I have been looking online and countless sources say you simply go to settings and select the shared compendium you wish to import say "Items" into so that when you much "Items" it will update the shared compendium verses a world specific one. So far this has not worked and not sure if it is a bug, a change, or a procedural problem on my side.
When I go into the settings all optioned only show the world specific "[World] ddb [Compendium Name]", Tables and Journals are the only ones that give other options.
What I am attempting to do:

Create a shareable module which would have tables referencing the imported from DDB Items compendium
This would allow me to access/update info from DDB in all worlds.

If this is not a feature yet, adding the option to allow munched compendiums created during import to be shareable verses world specific by default

It is an option, make sure you have created compendiums of the right type, and that they are unlocked.


Improve Import of changed catalogs

If you import a changed catalog which is part of another catalog, the differences should be somehow visible.

I actually don't think we addressed this yet (or if we want to). It reads that an import should show which other catalogs are affected. @leucoryx can you clarify, if you still remember what this issue was about?


[WFLY-4882] Ensure the ObjectList attribute is initialized with an em…

…pty list when its enclosing element is present but empty.
This is being done so that in the security manager subsystem we are able to differentiate between an empty and an absent maximum permission set.
Link to the upstream Jira:
https://issues.jboss.org/browse/WFLY-4882

You are right, Tomaz. I should be handling this via AttributeDefinition.


Mousing over Editor MenuButton causes incorrect selection

Godot version
v4.0.beta.custom_build [11d74d606]
System information
Windows 10, Vulkan, i9900KF, RTX 2070S
Issue description
When interacting with the Editor MenuButtons, Scene, Project, Debug, Editor, Help, the mouse position is incorrectly detected. Menu items are selected while the mouse is visibly on top of other options, or while it is not within the menu whatsoever. Nested dropdowns open on the wrong side as well.
This also affects other uses of Option and MenuButton, like in the File Dock.
Steps to reproduce
https://user-images.githubusercontent.com/44248915/199045201-67e15a56-1708-4069-9601-6f6ae722ebb3.mp4
My editor Theme settings are as follows:

Minimal reproduction project
No response

Possibly introduced in #67903.
Do I assume correctly, that you have Single Window Mode Off?

Possibly introduced in #67903. Do I assume correctly, that you have Single Window Mode Off?

All settings are at default, so yes that is off.


Add nightly builds

The nightly build is currently identical to the CI build, but without the publish step.
This also adds workflow dispatch options to both the nightly and CI builds. Because this repository depends crucially on qutip itself, it may regularly be useful to manually trigger a build (e.g. to update after a QuTiP bug fix or similar.
Fixes #41.

Looks good to me! :) Happy to merge.


HTML support needed

Excellent work I would say. Supporting msg format from JS is a must!
Supporting html is very important. Since I could not reopen https://github.com/ykarpovich/msg.reader/issues/1 and I do not know if closed issues are monitored I am opening a new issue now. The demo page fails to parse the msg in full when html is included. Is it difficult to enhance the library to support that? Perhaps the author can share some ideas?
In the meantime here is a workaround:

Use msgconvert linux command to go from msg to eml:

sudo apt install -y  libemail-outlook-message-perl
cd /tmp
msgconvert test\ with\ html\ content.msg # creates  test\ with\ html\ content.eml

Use https://github.com/nodemailer/mailparser to get the information from the eml, for example:

git clone https://github.com/nodemailer/mailparser.git
npm install
cd mailparser/examples
node extractInfoFromEml.js /tmp/test\ with\ html\ content.eml

Below is the code for extractInfoFromEml.js

/* eslint no-console:0 */

'use strict';

const util = require('util');
const fs = require('fs');
const simpleParser = require('../lib/simple-parser.js');
const args = process.argv.slice(2);
const filePath = args[0];

let input = fs.createReadStream(filePath);

simpleParser(input)
    .then(mail => {
        console.log(util.inspect(mail, false, 22));
    })
    .catch(err => {
        console.log(err);
    });

Best regards, - Nestor

HTML support would be fantastic :-)

@0sander See PR #7 for a fix affecting HTML embedded as binary.

is this fixed now that #7 is included ?

my body html is appearing like -
bodyHTML: "���9buT"
any ideas?

@visgotti same here. bodyHTML seems to not parse correctly. In my case it only reads 16 bytes from the corresponding block. The same would happen if #7 wasn't applied so that is not it.
maybe @ykarpovich has some idea where that goes wrong ?
Example mail with some attachments: https://drive.google.com/file/d/1Qt1I0w1TTP6H-Z4ZrGnbWcBoehEgEHtS/view?usp=sharing
...
bodyHTML: "yz�buT.��(	B"
...

Try changing this line https://github.com/ykarpovich/msg.reader/blob/master/msg.reader.js#L396
From:
    if (fieldName) {

To:
    if (fieldName && !fields[fieldName]) {

There's something weird with the full HTML being extracted and then getting overwritten as the parser continues traversing through the .msg file's data structure.

@ashsearle I did try that just now. No change. The change only triggers on the body field anyway not on the bodyHtml field.

At this moment the issue still exists. In some cases .msg contains valid body as HTML, but it's really rare.
Looks like HTML body message stored as RTF body attachment (or something like that).
Any way need to investigate how to retrieve HTML body in 100% cases. Based on the specification it doesn't work for now

@ykarpovich sounds likely it is RTF (Outlook should use HTML as default but maybe they convert it internally depending on some use cases). will try to read the pure data as RTF

Any news on this one and RTF ?

Excellent work I would say. Supporting msg format from JS is a must!
Supporting html is very important. Since I could not reopen #1 and I do not know if closed issues are monitored I am opening a new issue now. The demo page fails to parse the msg in full when html is included. Is it difficult to enhance the library to support that? Perhaps the author can share some ideas?
In the meantime here is a workaround:

Use msgconvert linux command to go from msg to eml:

sudo apt install -y  libemail-outlook-message-perl
cd /tmp
msgconvert test\ with\ html\ content.msg # creates  test\ with\ html\ content.eml

Use https://github.com/nodemailer/mailparser to get the information from the eml, for example:

git clone https://github.com/nodemailer/mailparser.git
npm install
cd mailparser/examples
node extractInfoFromEml.js /tmp/test\ with\ html\ content.eml

Below is the code for extractInfoFromEml.js

/* eslint no-console:0 */

'use strict';

const util = require('util');
const fs = require('fs');
const simpleParser = require('../lib/simple-parser.js');
const args = process.argv.slice(2);
const filePath = args[0];

let input = fs.createReadStream(filePath);

simpleParser(input)
    .then(mail => {
        console.log(util.inspect(mail, false, 22));
    })
    .catch(err => {
        console.log(err);
    });

Best regards, - Nestor

how to use these commands in code. kindly help @
nestoru

At this moment the issue still exists. In some cases .msg contains valid body as HTML, but it's really rare.
Looks like HTML body message stored as RTF body attachment (or something like that).
Any way need to investigate how to retrieve HTML body in 100% cases. Based on the specification it doesn't work for now

In my case I am getting the file locally on server from a folder & it did't give me the html body, but if I parse it through the example given by you it shows the content over there...I have attached the image of the response after parsing on the server side  & image of the example as well.
what I am doing wrong kindly need a little help. lots of appericating in advance. thanks
serverside

example

how to get the image having "cid" link src from the html body. any idea?

how to get the image having "cid" link src from the html body. any idea?

For that I added the following line to the NAME_MAPPING section:
'3701': 'embeddedImage',

About RTF encoded HTML: I was curious so I looked at all extracted information that do not get a name and so are dropped.
I stumbled upon 1009 and called it bodyRTF.
After fighting hours to get npm packages to work that are meant to run using node.js (I need a browser solution), I used the following code for stripping the RTF annotation:
https://stackoverflow.com/a/188877/3989858
I did have to modify it a little to make it work, and I still have to fiddle with embedded images, but it looks promising so far.
I could provide this as an angular project if someone is interested.

ide this as an angular project if someone is interest

@FROGGS , do you have a fork of this project with your changes/fixes?


Add new primary key typing everywhere to ensure primary keys index nu…

…mber | string properties

Codecov Report

Merging #11 (235b60c) into feature/better-primary-key-typing (ca66e5b) will not change coverage.
The diff coverage is 86.74%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                        Coverage Diff                         @@
##           feature/better-primary-key-typing      #11   +/-   ##
==================================================================
  Coverage                              94.76%   94.76%
==================================================================
  Files                                     62       62
  Lines                                    994      994
  Branches                                 230      230
==================================================================
  Hits                                     942      942
  Misses                                    52       52

Impacted Files
Coverage Δ

packages/db/src/events/Middleware.ts
100.00% <ø> (ø)

packages/db/src/core/count.ts
89.47% <50.00%> (ø)

packages/db/src/query/limit/index.ts
78.94% <50.00%> (ø)

packages/db/src/core/updateWhere.ts
95.45% <66.66%> (ø)

packages/db/src/core/clear.ts
92.85% <75.00%> (ø)

packages/db/src/core/insert.ts
90.00% <75.00%> (ø)

packages/db/src/core/remove.ts
88.88% <75.00%> (ø)

packages/db/src/core/removeWhere.ts
90.00% <75.00%> (ø)

packages/db/src/core/update.ts
90.00% <75.00%> (ø)

packages/db/src/core/upsert.ts
90.00% <75.00%> (ø)

... and 21 more

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


[BUG] Spark Pickle produces incorrect key labels for GenericRowWithSchema

Describe the bug
https://github.com/NVIDIA/spark-rapids/pull/2169 worked around https://issues.apache.org/jira/browse/SPARK-35108
We need to retest since this is supposed to be fixed with https://issues.apache.org/jira/browse/SPARK-34545  and put a test case back in for the fixed spark versions.

I'm not sure we need a test case for this. It is a Spark bug not a bug in the Rapids Accelerator. The issue was exposed completely by accident because I happened to pick a test that caused the issue.  The work around was to modify the test to still be 100% valid for what we are testing in RAPIDS, but not expose the bug in Spark. In fact the new test has more coverage, not less. So if we add in a test we would have to make it conditional on Spark 3.0.3 and above, 3.1.2 and above and 3.2.0 and above. I personally don't think it is worth it.

I did manually verify the fix in Spark too.


Increase CLIENT_TIMEOUT_MS for initial /sync

Increase the client timeout, failed to do an initial sync for me on iOS simulator (1020 rooms) when using the SDK in a custom app.
Pull Request Checklist

[x] Pull request is based on the develop branch
[x] Pull request contains a changelog file in ./changelog.d. See https://github.com/matrix-org/matrix-ios-sdk/blob/develop/CONTRIBUTING.md#changelog
[x] Pull request includes a sign off

Thanks for that info, that's good to know!


v0.5.8

required to make a release
we can publish now or wait for #120

added #120 to the changelog

bors r+

Published: https://crates.io/crates/cortex-m/0.5.8


Update law

Check this repository to see if it is still up to date

Interview works and word doc (motion) appears sufficient


Does this library still maintained and will it continue to support React 0.14 and 15 ?

+1

Anybody?

I found this, perhaps it is the best replacement?
https://github.com/buzzer13/react-ondemand-loader

It works just fine with React 0.14 and 15

How do you use Mixins in the newer React versions?


Ensure that filter settings are presisted via toggle buttons

Closes #315 and Closes #338
This fixes the transaction filter toggle buttons not causing settings to be persisted when leaving the Account's page.

Thank you!


[Bug][tcgc] isFixed is wrong when calling getSdkUnion

Describe the bug
When we adopt to the latest tcgc our two cases failed and we found that

Sometimes the getSdkUnion could return correct isFixed value like Case 2 but sometimes it can't like Case 1
Also for union of EnumA | EnumB tcgc would flatten all underlying enums into top level

TCGC are supposed to provide consistant isFixed information, otherwise it would be a big issue for JS because RLC didn't adopt tcgc but Modular does. So it would introduce type inconsistancy between them and finally fails the compile process.
Also I think current behavior for prompting members to top level is debetable. It would lose the name info which may be useful for emitters.
Case 1: Nullable enum
enum Color {
  Color1: 1,
  Color2: 2
}
model Test {
  color: Color | null;
}
op read(@body body: Test): void;

Case 2: EnumeA | EnumB
 enum LR {
  left,
  right,
}
enum UD {
  up,
  down,
}

model Test {
  color: LR | UD;
}
op read(@body body: Test): void;

TCGC get this info from two places, first is isFixed (code) from @fixed decorator, second is the open property in UnionEnum type (code). I think current problem should mainly from losing information for hierarchy union.

@MaryGao, in JS, do we have a workaround for this issue?

isFixed
I don't think we have a quick workaround here, currently we just fallback to tcgc dev.17 version.

Here we are only calling what typespec and typespec-azure are providing us, like @tadelesh said. Talked with @lmazuel and for case 1, it is by design that isFixed is still returning False. The tsp team will change the behavior of isFixed once everyone has moved to adopting "union with base type is extensible enum"

Both your cases are by design and work as expected:
enum Color {
  Color1: 1,
  Color2: 2
}

is an extensible enum. We do have plan to switch this to fxed enum, when we can confirm all emitters are treating the union correctly
Flattening enum values is by design as well.
@MaryGao I'm worried to be honest of what "workaround" you would do here, since both are by designed. I'm closing this issue, and we can confirm this off-line.

The inconsistency is come from the flatten. TCGC uses getUnionAsEnum to see if a union is an enum, and this function does the judgement from the flattened elements view because TypeSpec type system, UnionEnum type could has child union or enum element. Since client type system could only has union or enum type, I made a fix to change back to union if union has union or enum variants, regardless of it is an enum with flatten view.

JS has workaround for the inconsistency of whether is open for flattened union as enum.


Search for "teams" is not working properly.

Describe the bug
I'm trying to create a new repo using https://repos.opensource.microsoft.com/orgs/microsoft/new and it requires a "admin team", so i created a new team here: https://github.com/orgs/microsoft/teams/ada-team but the find team dialog is not finding it:
When I search for "ada" it finds a million substring matches which also makes it impossible to find.  Can you fix the search box so it works on "whole word" or support double quotes searches like "ada team" that finds exact literal case insensitive match?
Deployment environment
This bug report is intended for issues relating to the project, and not a particular instance of the
management portal. If you are looking to file a bug report about the app itself, please instead
locate the links for support in the footer of every page of the app that you are using.
To Reproduce
See above.
Expected behavior
Search box needs to work better.
Screenshots

Additional context
Add any other context about the problem here.

Thanks for the feedback, Chris.
Any chance you used the "refresh" button on the right at any time, or just the search experience?

Yes I was clicking refresh over and over, but I think I know what happened, the team was showing up here and it seems whatever teams are prepopulated there do not show up in the "Other teams..." list.

But it does show up today, so is there just a very long delay in updating this list?

Ah, yeah, that must be it. On the GitHub side it can take over 20m to get the list of teams updated in the Microsoft org when someone requests a refresh.

would help to perhaps say that somewhere on the site so I can not fret, grab a coffee and come back to it :-)

It's my understanding this issue in the Microsoft-internal instance has been fixed.


Let's make ESPT available using the tag, without signing up / admin account

We don't need to advertize ESPT yet, and various improvements need doing, but making it accessible, though requiring the espt:1 tag, would make testing it easier.

This now seems to be the case.


Cannot supply message from github actions CI

https://github.com/buildkite/test-collector-python/blob/main/src/buildkite_test_collector/collector/run_env.py#L57 hard-codes message=None when the collector detects it's running inside github actions.
It's pretty easy to offer the commit message to the collector; export GITHUB_MESSAGE="$(git log -1 --pretty=%B | head -1)" and that's what I tried - then I tried BUILDKITE_ANALYTICS_MESSAGEandBUILDKITE_MESSAGE` before ending up reading the code :)
Would you welcome a PR that allows the =None parameters to be overridden by env-vars?
Even if I'm forced to use github actions, I'd love to use all the features of test analytics.

Hi there. I'd for sure welcome a PR that changes this.

@jimsynz #32 is ready for your reviewing pleasure :)
(Please let me know what kind of coverage you might like; I kinda figured __get_env was already covered, but am totally open to adding things as you direct)

@jimsynz thanks for the merge! Do you have an ETA for a release that this will appear in?

@gchan any chance you can cut a release?

I've published v0.1.9 of the collector with your update @petemounce. Thanks for your contribution and please let us know if you find any issues with this build!
cc @jimsynz


[Bug]:How can I correctly locate the ssg build error

Version
system： macos
npmPackages：
"react-photo-view": "^1.2.6",
"rspress": "^1.35.1",
"rspress-plugin-file-tree": "^0.4.0",
"rspress-plugin-live2d": "^0.3.0",
"rspress-plugin-reading-time": "^0.3.0",
Details
How can I correctly locate the ssg build error, the stack information provided by the console can't be debugged

I tried a global search for window, it's not in the source code, it exists in a third party dependency, but need more log information to help me locate it, can you help me?
Reproduce link
xxxx
Reproduce Steps

npm run build

set the config below in rspress.config.ts

defineConfig({
	builderConfig: {
		tools: {
			rspack: {
				optimization: {
					moduleId: 'named'
				}
			}
		}
	}
})

DEBUG=rsbuild rspress build

click the error capture stack, and jump to the main.cjs
investigate which 3rd library does not support SSR

thanks for your trying Rspress,
this is a common question, and we may need to optimize the Failed to load SSG bundle error message

It's working fine.


⚠ Spryr has degraded performance

In cde9bf2, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5130 ms

Resolved: Spryr performance has improved in ebee49d after 30 minutes.


Why is the connection not guaranteed to be valid?

About this code snippet:
https://github.com/xetorthio/jedis/blob/72e5b1c6184047ba746de0e28de2b788adc898b9/src/main/java/redis/clients/jedis/JedisSlotBasedConnectionHandler.java#L64-L81

Why is the connection not guaranteed to be valid?
How to obtain a valid connection?

Jedis version:
2.9.3
Java version:
1.8.151

Why is the connection not guaranteed to be valid?
Cause slots might moved.

How to obtain a valid connection?
You can't since it's a distributed system, you can test the connection after you get and if the server doesn't return "MOVED" you are ok


goroutine error when starting a stream

This configuration has been working for many months and was working until a week ago. I think updating Threadfin may have broken it but I'm not 100% positive on that as there may have been other minor updates my NAS received.
2024/11/17 01:40:02 [Threadfin] XEPG:                   Ready to use
2024/11/17 01:41:06 [Threadfin] Buffer:                 true [ffmpeg]
2024/11/17 01:41:06 [Threadfin] Channel Name:           UK: Premier Sports 1
2024/11/17 01:41:06 [Threadfin] Client User-Agent:      Lavf/LIBAVFORMAT_VERSION
2024/11/17 01:41:07 http: panic serving <IP_ADDRESS>:46848: interface conversion: interface {} is nil, not string
goroutine 91 [running]:
net/http.(*conn).serve.func1()
/usr/local/go/src/net/http/server.go:1825 +0xbf
panic({0x83e020, 0xc00007d800})
/usr/local/go/src/runtime/panic.go:844 +0x258
threadfin/src.bufferingStream({0xc001778b40, 0x14}, {0xc001449490, 0x63}, {0x0, 0x0}, {0x0, 0x0}, {0x0, 0x0}, ...)
/src/src/buffer.go:183 +0x1e65
threadfin/src.Stream({0x9f6790, 0xc0025c60e0}, 0xc000c00000)
/src/src/webserver.go:185 +0x938
net/http.HandlerFunc.ServeHTTP(0x7fa494726698?, {0x9f6790?, 0xc0025c60e0?}, 0x40efe5?)
/usr/local/go/src/net/http/server.go:2084 +0x2f
net/http.(*ServeMux).ServeHTTP(0x0?, {0x9f6790, 0xc0025c60e0}, 0xc000c00000)
/usr/local/go/src/net/http/server.go:2462 +0x149
net/http.serverHandler.ServeHTTP({0xc002d38390?}, {0x9f6790, 0xc0025c60e0}, 0xc000c00000)
/usr/local/go/src/net/http/server.go:2916 +0x43b
net/http.(*conn).serve(0xc0025d2000, {0x9f6b38, 0xc003d2a030})
/usr/local/go/src/net/http/server.go:1966 +0x5d7
created by net/http.(*Server).Serve
/usr/local/go/src/net/http/server.go:3071 +0x4db
QNAP Docker
[Threadfin] Version:                1.2 Build: 15
[Threadfin] Database Version:       0.5.0
[Threadfin] System IP Addresses:    IPv4: 2 | IPv6: 1
I am attempting to stream through Plex DVR when I receive this error. It seems that the Threadfin WEB UI becomes completely unresponsive once I get this error.
Thanks!

Open and save your playlists. The buffer has not been saved for your playlist

Thank you, that worked. Sorry, I haven't come across this one before. Cheers


[WIP] Handle deployment settings in a new collection in the database

Closes #704

Added a model setting with name, value and access_level (public or private)
Added a setting table to the admin dashboard that display every setting set in the db
Added a way to update these settings

Should we have a default value for certain parameters defined in the code? or
Have an empty field in the admin dashboard for parameters not yet defined? or
Just enter them manually in the database the first time?

pre-commit.ci autofix

pre-commit.ci autofix

pre-commit.ci autofix

I think much like #604, there's enough to tidy up here (and stuff that could subtly break deployments) that we should hold off until after the 0.4 release to merge this.

As discussed during the datalab meeting and for future work:

[] Should we remove the "Add a setting" button?
[] Should the settings each have an entry in db or a collection by "acces_level" that includes all relative settings?
[] Do we need to be able to delete a setting or just modify it?
[] What settings should we put in the admin dashboard for the future? (only LOGO_URL and HOMEPAGE_URL work for now)


mone in generator

Hi,
In the gan training code, since you have already used gen_cost.backward(mone) for generator, why are you still doing gen_cost = -gen_cost
i.e, should it not be

just gen_cost.backward(mone)
or
gen_cost.backward() and then gen_cost = gen_cost*-1
Please correct me if I am wrong.

I think
gen_cost.backward(mone) is changing grad to -1, it wont modify gen_cost


Missing fonts in OS X build

Downloaded the pre-built binary from:
https://github.com/yakyak/yakyak/releases/tag/v1.3.2

I verified that the css folder inside the YakYak.app package does not contain the files being referenced. Perhaps it's just a build/compile error.
Building the app from source doesn't contain this issue.

This has been fixed in next release.  See https://github.com/yakyak/yakyak/releases/tag/v1.3.3
(and dont worry, if you stick with 1.3.2, font isnt missing, just woff2 files missing, but you still have woff files working) .


Deduplicate corpora when user has multiple roles

What went wrong?
Right now, a corpus may appear multiple times in the list if a user has multiple roles.
What did you expect to happen?
Each corpus should only appear once.
Screenshot

Where did you find the bug?

[X] https://ianalyzer.hum.uu.nl
[ ] https://peopleandparliament.hum.uu.nl
[ ] https://peace.sites.uu.nl
[ ] a server hosted elsewhere (i.e. not by the research software lab)
[ ] a local server

Version
No response
Steps to reproduce
Log in with a user that has multiple roles.

closed by #1702


scjss-verify-setup doesn't exist

I'm trying to run the basic example per your documentation, and everything works fine when I use the command jss start.
When I try to run jss start:connected (after running jss setup of course), I get the following error:
 jss start:connected
> yarn start:connected
yarn run v1.5.1
$ scjss-verify-setup && jss clean && node -r babel-register ./build/start.js --env.content=connected
env: node\r: No such file or directory
error An unexpected error occurred: "Command failed.
Exit code: 127
Command: sh
Arguments: -c scjss-verify-setup && jss clean && node -r babel-register ./build/start.js --env.content=connected
Directory: /Applications/XAMPP/xamppfiles/htdocs/my-first-jss-app
Output:
".

I also tried running via NPM with npm run start:connected, and get the following:
npm run start:connected

><EMAIL_ADDRESS>start:connected /Applications/XAMPP/xamppfiles/htdocs/my-first-jss-app
> scjss-verify-setup && jss clean && node -r babel-register ./build/start.js --env.content=connected

env: node\r: No such file or directory
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>start:connected: `scjss-verify-setup && jss clean && node -r babel-register ./build/start.js --env.content=connected`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start:connected script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/fratran/.npm/_logs/2018-04-19T15_01_42_319Z-debug.log

It appears that the command scjss-verify-setup is failing. I came to this conclusion by removing the scjss-verify-setup portion of the start:connected script, and the script executes but the resulting window shows:

Page not found

Note that I'm attempting to run connected mode, trying to pull data from a sitecore instance on azure cloud and rendering it locally - not sure if that setup has anything to do with it.

@sowhatdoido are you on a Mac? this looks similar to #28 based on the node\r error.


Button color icon different from the real color

Hi,
I implement the first xml button example and the icon color into the button is different (gray) to the real icon color (white). I tried using a black icon but with the same result.
Thanks,

yes. what is the xml attribute that disables overriding the original color of the icon?


Spotipy can no longer find devices

So I've had some code running for weeks and it's been able to find my chromecasts/google homes pretty well, but last night it seems to have issues finding any chromecasts/google homes, I'm wondering if a Spotify update might have caused it. Seems like the web interface doesn't list google homes/chromecasts any more, it just lists "Google Cast" and you select your speaker inside of there.
Here's my code I'm using:
# Query spotify for active devices
devices_available = client.devices()

# Match active spotify devices with the spotify controller's device id
for device in devices_available["devices"]:
    if device["id"] == sp.device:
        spotify_device_id = device["id"]
        break

if not spotify_device_id:
    print('No device with id "{}" known by Spotify'.format(sp.device))
    print("Known devices: {}".format(devices_available["devices"]))
    sys.exit(1)

Here's the output:
No device with id "4862a56929200af9e9485aae060e7724" known by Spotify
Known devices: []

Looks like it can't find anything at all even if the google home is assigned to the spotify controller.

I have the same problem. I can see my devices using the spotify web api console, but spotipy.Spotify.devices() is empty.

Best answer I can find is that the token security needed for this to work now is "user-read-playback-state", which sounds like a new thing from previous, based on this Spotify help chat: https://community.spotify.com/t5/Spotify-for-Developers/get-users-available-devices-return-empty-even-if-I-am-opening/td-p/5172688
Looks like there's a bunch of Spotify tickets around playing on Chromecast. I had this working for weeks, would like to know if there's anything I can do to help with this. https://github.com/spotify/web-api/issues/787

Same issue for me since mars 22.. what happened with spotify?

This is a spotify problem

@fondberg do you now a way to fix this? I already renewed my sp_dc and sp_key cookies without any effect.

Yes and no. Head over to spotcast where I will reveal it soon :-)

Since there is nothing we can do about this on the spotipy side, I'm closing this


Refactor(ES6): ModuleDependencyError.test.js

What kind of change does this PR introduce?
Refactor, ES5 => ES6
Did you add tests for your changes?
All existing tests pass
If relevant, link to documentation update:
N/A
Summary
Refactor ModuleDependencyError.test.js
Does this PR introduce a breaking change?
No

Thanks


Preprocessor: A lot of unnecessary actions

Now our preprocessor goes deep into the conditional structure and processes instructions inside it, even if the conditional block has the false state, I think it needs to be redone so that it does not go deep into the conditional structure if it is false.

Alright, I'm on it.


[C2847497] Failed to import from JLCPCB

I tried to import the part number C2847497 from JLCPCB, but it failed. Here's the error I got:
Failed to generate snippet from JLCPCB part: duplicate key value violates unique constraint "package_name_key"

Could be an issue in fetchEasyEDAComponent or convertRawEasyEdaToTs
https://jlcpcb.com/parts/componentSearch?searchTxt=MIMXRT1062DVL6B

@techmannih i already mentioned this on discord, but this is because it already exists in your account, so it's not a bug in how we convert the easyeda files, it's just a bad error message


Rename files to match GitHub username

Seems the script that updates the leaderboard linked to the wrong asun. This should fix it.

Yeah, names are resolved based on the launcher script name. Thx for the update!


Implement CLI interface and argument parser for the benchmark

Support implementation of the parametrized run with type of a run, config file, and outputs.
Write tests first.

#2 #3


Only compute max once.

This speeds up page loads. (3X for me)

Here's a screenshot and summary of Chrome's timeline for PCSK9 on my machine.

1200 ms: the server's first response comes through.
1600 ms: synchronous network requests have completed.
1900 ms: create_new_data begins.
8100 ms: create_new_data finishes. 97% of it's time is spent in Zo.max from d3.js.
9300 ms: DOMContentLoaded fires.

 Been meaning to optimize, and this was SUPER helpful, thanks!

Thanks for merging this!
Check out Timeline in Chrome Devtools.  It's magical.


Array.slice() creates returns array who's type is incorrect

Godot version
v4.0.beta1.official [20d667284]
System information
Ubuntu 20.04.5 LTS
Issue description
The following code reports an error:
"Cannot convert argument 1 from array to array"
func test_slice(a:Array[int]):
	print(a)

func testtest():
	var a:Array[int] = [1, 2, 3, 4, 5]
	print(typeof(a))
	print(typeof(a.slice(1,2)))
	test_slice(a.slice(1,2))

Both typeof() commands return 28.
It should have accepted the slice as a valid parameter value.
It is worth noting that it is possible to store the slice in a typed variable and then pass that variable to the function. This works as expected.
Steps to reproduce

Create a typed array.
Create a function with a parameter of the same type as in step 1.
Try to pass the array to the function.

Minimal reproduction project
No response

I can confirm it on Linux Mint 20 , Godot 4.0 beta 2

Kinda a duplicate of #63892 but I think we can leave it open as it's technically a little different.
#63892 is about Array[T]'s methods returning Variant instead of T.
This issue is about Array[T]'s methods returning Array instead of Array[T].

Unlike #63892 it's not a deeply rooted issue and can be fortunately fixed with a simple assignment.


Update version for the next release (v2.3.0)

This version makes this package compatible with Meilisearch v0.29.0 :tada:
Check out the changelog of Meilisearch v0.29.0 for more information on the changes.
🚀 Enhancements
This change may not affect you, but in any case, you should check your search queries if you want to keep the same behavior from v0.28.

The NOT filter keyword does not have an implicitly EXIST operator anymore. Check out for more information: https://github.com/meilisearch/meilisearch/issues/2486

Stupid question but why we update a major and not a minor?

To draw their attention to changes in the filter behavior! The other SDK's changes their minor as well and consider that change as a breaking change. Nonetheless, we did not want to update docs-searchbar.js to a new major for a change that is not breaking the code.

To draw their attention to changes in the filter behavior!

Good point!

we did not want to update docs-searchbar.js to a new major for a change that is not breaking the code.

But it will be updated to a major version here.

But it will be updated to a major version here.

I think it is a minor.

I think it is a minor.

OH yes!

bors merge

Thanks again for contributing with Meilisearch :heart:
If you are participating in Hacktoberfest, and you would like to receive a small gift from Meilisearch too, don't forget to fill this form.


Implement the fix_power_op_line_length preview style

Summary
Implements the fix_power_op_line_length preview style.
The main change is to add a space around the power operator if the binary expression splits over multiple lines:
x = (
    long
    ** 3
)

# instead of
x = (
    long
    **3
)

Closes https://github.com/astral-sh/ruff/issues/8938
Test Plan
See deleted snapshot file. The similarity index remains unchanged.

Current dependencies on/for this PR:

main

PR #8947   👈

This stack of pull requests is managed by Graphite.


Convert to megaparsec

Here is Megaparsec conversion of the parser.  The test cases that test error messages will still need to be updated, but all the other tests seem to pass now.

Can one of the admins verify this patch?

Jenkins: add to whitelist

Apparently the comments parser needs to be tuned to fully support C-style comments:
see src/test/resources/test-comment-5.kore and several unit tests, including https://github.com/kframework/kore/blob/master/src/main/haskell/language-kore/test/parser/Data/Kore/Parser/LexemeTest.hs#L50

Rebased and tests fixed in #92


Potential skinning issue with glb files

Godot version
4.1.3
System information
Windows 11 - Godot 4.1.3 - Forward+
Issue description
I was experimenting with using Blender's bendy bones for animation by copying the transform of the segments to standard bones, but the exported animation shows up weirdly in Godot.  Here's what the animation at a certain frame should look like

And here's the result from Godot:

The same issue doesn't happen when I export to FBX and use FBX2GLTF and with other GLTF viewers (I tried this and this), so this is most likely an edge-case in gltf-s godot doesn't handle properly.
Steps to reproduce

Open the attached project
Make sure "test" has "Editable children" enabled
Select "AnimationPlayer" and set "Current Animation" to "ArmatureAction"

Minimal reproduction project
SkinningIssue.zip

Thats so odd @lyuma someone finally found an error case

He was caused by LOD

Disabling LODs does solve the issue, but should that be on by default in this case?

I suggest testing https://github.com/godotengine/godot/pull/84384 locally and see if it resolves the issue on your end.

Disabling LODs does solve the issue, but should that be on by default in this case?

Yes, LOD isn't supposed to break meshes like this :slightly_smiling_face:

The lod operator doesn't account for per vertex bone groups.
tl;dr We only match the closest normals in the vertex attributes.
The layout of inputs are:
size_t new_index_count = SurfaceTool::simplify_with_attrib_func(
	(unsigned int *)new_indices.ptrw(),
	(const uint32_t *)merged_indices_ptr, index_count,
	merged_vertices_f32.ptr(), merged_vertex_count,
	sizeof(float) * 3, // Vertex stride
	merged_normals_f32.ptr(),
	sizeof(float) * 3, // Attribute stride
	normal_weights, 3,
	index_target,
	max_mesh_error,
	simplify_options,
	&mesh_error);

Probably want the notion that bone groups that are close are kept together, but each different bone group has very different values numerically. Like bone group 0 for a "head" is numerically different from bone group 8 for the "neck" even though the vertex groups are connected.

Doing LOD on skinned meshes is a fun topic, I'm sure there will be research into this.
Because whether weighted polys can be reduced depends on the poses expected!  :grin:
At the extreme an animated skeleton with little movement can be decimated as usual. Whereas if just one arm moves a lot, only those polys need be prevented from decimation more.
Simplest scheme is just to not LOD any verts in polys that contain weights. That will preserve the joints but will limit how far poly reduction can take place.
In theory if you have a bunch of poses, you can calculate the max error metrics for use in decimation from these (by software skinning each pose maybe).

I was investigating using Smooth Skinning Decomposition with Rigid Bones. This algorithm extracts Linear Blend Skinning (LBS) from example meshes and can be used for:

Converting animated mesh sequences to LBS.
Solving skinning weights from shapes and skeleton poses.
Solving bone transformations for a mesh animation given skinning weights.

https://github.com/electronicarts/dem-bones
However, it doesn't directly relate to solving this problem of modify our simplification algorithm to better preserve vertices for an animated mesh.

The result from the image doesn't really make sense to me, but I'm not sure what the mesh that the simplifier is being fed here is - eg I'm not sure I understand what the deformation structure here is, maybe a lot of vertices are in the same position and only are moved by the final bone deformation. I think the mesh importer skins vertices before simplification but that code could be getting incorrect (default?) transforms in this case maybe.
If the initial bone transforms here are identity and the expectation is that the mesh would be in tact after applying different transforms at runtime, then one possible solution is to extract a single bone id for the dominant bone per vertex (largest weight), and at the minimum use it during vertex merging, and also maybe pass it as an extra attribute with some weight to the simplifier. The update to meshoptimizer makes it so that the attribute count is fully dynamic so there won't be a need to patch the simplifier source code anymore (right now the max attribute count is baked into the library source).
There's research on skinning aware simplification but it's fundamentally a difficult problem to solve in a reasonable amount of time. But it also feels like the image here is more wrong than anything I've seen with normal skinning (I've tested simplifier mostly with pre-deformed mesh vertices and it worked completely fine!).
One issue that was mentioned in https://github.com/godotengine/godot/pull/84384 (btw I don't think that should affect this import one way or the other but not sure) is the unbounded positional error; in effect the simplifier is allowed to produce completely arbitrary result. I do think it would be nice to change this longer term, as it can improve the quality here, but on unskinned meshes I would expect the current approach to be okay if suboptimal - if the deformation is too large, the resulting LOD's error will be large, and it should not be selected for rendering. So I don't think this issue should depend on whether the error is bounded (and I would recommend waiting for bounding the error until Godot fully switches to unmodified meshopt_simplify, so probably 0.21 or thereabouts, as right now just passing a smaller error will not work properly due to differences in metrics that include or exclude attributes in the interface).

the image here is more wrong than anything I've seen with normal skinning

Well, the good news is, I can reproduce this in meshoptimizer's test harness :D so this is not unique to Godot import. And yeah without animation (in the initial position) the simplification looks fine - but that's because the initial pose is a straight cylinder, so the simplifier removes triangles along the cylinder's sides. These collapses are also very low error because, well, the shape doesn't change if you do them.
Of course this all changes once the bones are actually rotated. Something like this is inevitable under bone deformation but this of course is a particularly egregious case.
The only way to solve this particular mesh, other than disabling LOD generation during import :), is to feed some information to the simplifier that would tell it something about the deformation. It could be bone id, some info about transforms, or weights, basically anything that helps differentiate the triangles...
One additional issue though is that because the distance error is very small, and distance error is used for LOD selection, this alone won't help enough - it's also the case that a combined error will need to be stored for LOD selection to properly reject other LODs.
For that to work, import process would need to store the combined error and the default normal weighting will need to be dramatically adjusted - right now the simplification gets weight 2.0 for normals but that dramatically increases the combined position+normal weighting that simplifier uses. I suspect that to make reasonable LOD selection decisions using the combined error, not just position error, on "regular" meshes, the normal weight will need to be in the 0.01..0.1 range, although I haven't really experimented with this.

Quick example for incorporating the dominant bone index as an attribute (sorry for video quality, it took me 3 tries to get a sub-10 MB video that apparently GH attachments are limited to :-/):
https://github.com/godotengine/godot/assets/1106629/eb54e2d0-5c96-4dbe-8b2c-5d1f6cf35634
Per comments above, this alone would not suffice because the resulting attribute error is presently ignored so the mesh will have better quality LODs but will still probably display poorly, but this can likely be a reasonable way to attack this based on this. I would probably experiment with more physically-inspired inputs than the index, for example dominant bone translation or something like this, as this will make more sense when integrated into the overall error, but maybe even ID is workable with carefully tuned weighting.

@zeux I did an attempt to do rig aware lods here and it seems to be ok. https://github.com/V-Sekai/godot/tree/lock-primary-base-influence
Needs optimization


Adding a polygon to a disabled CollisionPolygon2D will allow it to trigger collisions even when disabled

Godot version:  v3.1.1
OS/device including version: WIndows 10
Issue description: Adding a polygon to a disabled CollisionPolygon2D will allow it to trigger collisions even when disabled
Steps to reproduce:
Create Area2D
Create CollisionPolygon2D as child (do not add a polygon in the editor)
Add CollisionPolygon2D.disabled = true in _ready()
After disabling the CollisionPolygon2D, create and populate a PoolVector2Array and assign it to $CollisionPolygon2D.polygon.
This Area2D should now collide with other objects even though disabled = true
Minimal reproduction project:

Can't reproduce in 3.2.2.stable or 3.2.3.beta1, looks to be resolved.
convenience project: 35026.zip

Thanks for testing @tom-jk, closing as fixed in 3.2.


Added Codeforces Dynamic Programming Problems

I have done the modifications that you had asked for. Thanks for reviewing.


Problems with mobile/desktop browsers

Until now, I have used  for my datepicker fields which was working fine in Firefox and Chrome on my desktop browsers. But in Chrome on Android, when I hit the date field I get the Android date rollers come up, I select my date, then when it closes I get the date picker pop up! So I don't need both clearly.
So I changed the field to "text" - again, still working fine on my desktop browsers, but now on my Android it triggers the on screen keyboard to pop up, which means my bootstrap date picker isn't fully in view and when I move the screen it closes it.
ARGHHH!! What's the best integration please for mobile and desktop?

Please try it again with the code in master, as #842 introduced an option for this. Please confirm if you issue is fixed on master

Hi there. Sorry just to be clear - do I download the latest copy of the code or am I just copying and pasting some code into what I already have?
Am I also just setting them to text fields now or keep them as date fields?

You could try to build a case with this base jsfiddle, this example uses the 1.3.1. Or just copy paste the current master code to your case and check if it is still an issue! If the problem still persists, then create a jsfiddle example!


Navigation | Section headings not highlighted when scrolling down

The section headings are not getting highlighted as we scroll down the page. Same for every chapter.

fixed in the latest build


Set a minimum version for pyyaml

[x] Closes #8543
[x] Tests added / passed
[x] Passes pre-commit run --all-files

Can one of the admins verify this patch?

ok to test

@jrbourbeau this PR is ready for review.

The tests pass locally for me (macOS). The windows-tests seem also be failing on main.


tests: add missing WKWebView tests, remove unused SessionChallengeSen…

…der class

Codecov Report

Merging #60 into master will increase coverage by 3.14%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #60      +/-   ##
=========================================
+ Coverage   87.45%   90.6%   +3.14%
=========================================
  Files          26      25       -1
  Lines        1307    1298       -9
=========================================
+ Hits         1143    1176      +33
+ Misses        164     122      -42

Impacted Files
Coverage Δ

...eTracking/Private/_RPTClassManipulator+WKWebView.m
94.15% <0%> (+19.29%)
:arrow_up:

Continue to review full report at Codecov.

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


Add IdenTrust as Certificate Authority

[Can you please add IdenTrust as Cerificate Authority to this page?
We are in the CCADB list as public trusted CA:
https://ccadb-public.secure.force.com/microsoft/IncludedCACertificateReportForMSFT
Feel free to contact me in cae you have questions:
Marco Shambach, Program Manager<EMAIL_ADDRESS>Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e26faf8c-529e-b317-5934-cd41cfe8fa24
Version Independent ID: 3ddd5d0a-6dbb-4940-1814-8346e293701b
Content: Plan Direct Routing - Microsoft Teams
Content Source: Teams/direct-routing-plan.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Please follow up with @FilippSe before making changes to this article.

You may want to take the same approach taken on this webpage:
https://docs.microsoft.com/en-us/microsoftteams/direct-routing-plan#public-trusted-certificate-for-the-sbc
`Direct Routing SIP interface will only trust certificates that are signed by Certificate Authorities (CAs) that are part of the Microsoft Trusted Root Certificate Program. Make sure that your SBC certificate is signed by a CA that is part of the program and that the Extended Key Usage (EKU) extension of your certificate includes Server Authentication. Learn more:
Program Requirements - Microsoft Trusted Root Program Included CA Certificate List https://docs.microsoft.com/en-us/microsoftteams/direct-routing-plan#public-trusted-certificate-for-the-sbc `
where instead of listing explicit CA's, it shows the full list of CA's in the CCADB. This way you no longer need to maintain this page for approved CA's as the ones in CCADB have the full details.
Just an idea 😊
Marco Schambach
Program Manager
IdenTrust
5225 Wiley Post Way,
Salt Lake City, UT 84116-4915
+1-801-384-3553 (Office)
+1-954-608-1344 (Mobile)
@.*** @.***>
http://www.identrust.com/ www.IdenTrust.com
http://www.hidglobal.com/ www.hidglobal.com
From: Manuela Scanu @.>
Sent: Tuesday, December 7, 2021 6:18 PM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness @.>
Cc: Marco Schambach @.>; Author @.>
Subject: [External] Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Add IdenTrust as Certificate Authority (Issue #8235)
Thank you for submitting feedback and contributing to the docs. We are currently investigating this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/8235#issuecomment-988332886 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIQ6DCISY3MKOI24X5KDJZDUP2I3BANCNFSM5JRZRKSQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/AIQ6DCKVLKBLMXBYJAKH6A3UP2I3BA5CNFSM5JRZRKS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHLUMGVQ.gif

Correct, Direct Routing switched to supporting broader general list and not specific CAs. So, we will not be adding any CA individually.

@neymars, you are referring to the same article: Plan Direct Routing.
The following sentence was added by @FilippSe yesterday so I understand that we can close this issue:
"Direct Routing SIP interface will only trust certificates that are signed by Certificate Authorities (CAs) that are part of the Microsoft Trusted Root Certificate Program. Make sure that your SBC certificate is signed by a CA that is part of the program and that the Extended Key Usage (EKU) extension of your certificate includes Server Authentication"
Or maybe I didn't understand, please let me know. Thank you.

Correct. This issue can be closed. Thank you!


Add alias to Intervention Image class in MultipleImage content type

When using the Intervention\Image\Facades\Image facade directly inside the MultipleImage content type, we are getting the following error: Cannot use Intervention\Image\Facades\Image as Image because the name is already in use. This error appears because there is already a class named Image in the same namespace (the content type: Image).
It was reported by @PinkuChanda at #3179 and solved by @emptynick.
I think merging this pull request without squashing will add @emptynick as Co Author in the commit.

Co-Authored-By: Christoph Schweppe
Fixes #3179

Can this fail if you have an Image field + a Multiple Images field in one BREAD?

Yes image and multiple image is working properly at same bread.. i think it is fine now.

No it won't fail with both in the same BREAD.
The issue is a namespace conflict, and each PHP class has it's own locally-scoped namespace.  Adding use statements effectively import the other classes into that local namespace.  The Image import was conflicting because there's a PHP class called Image in the same directory (automatically imported into the same namespace).  This PR should be the end of these issues.


Physicscube appearing at the spawn point of vrm for initial seconds

Describe the bug
While the game is being loaded, the physicscube appears at the position (0,0,0) same as that of spawn-point of vrm.
To Reproduce
Steps to reproduce the behavior:

Load the app
For the initial loading time, till the game is fully loaded, the cube appears at the spawn point.

Expected behavior
It should appear anywhere but the spawnpoint while being loaded. Though this would be solved with a loading screen
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser : Chrome

Additional context
Related: https://github.com/webaverse/app/issues/3413

Fixed here: https://github.com/webaverse/physicscube/pull/2


Fixed incorrectly handled image upload errors

I want to merge this change to provide users with proper error notifications if they try to upload image files in a wrong format.

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://qa.staging.saleor.cloud/graphql/

It’s been fixed now for product and collection pages as well as user avatar update.
Previous changes based on initial scope.


Adding xbot_talker to documentation index for distro

I'd like my_repo xbot_talker to be indexed and documented on ros.org

Also, this now needs to be rebased.

No response for 20 days, so I'm going to close this out for now.  Feel free to fix the problems pointed out in review and reopen.  Thanks for the contribution.


Update azuredeploy.json

Required items, please complete
Change(s):

updated url

Reason for Change(s):

The url on the json file doesn't seem to work and updated to the one which seems to work.

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Updated url, please review


Improved prepared statement related functions.

Added functions to iterate and fold over result rows
Added functions for easier binding of non-64-bit integer values,
named parameters, and multiple values all at once.
Similar for getting integer result column values..
Support blobs as byte sequences as an alternative to strings.

Basically, a bunch of stuff to make it easier to use prepared statements.

@shawnw I'm interested in standardizing the type conversion code.  Can we consolidate this with https://github.com/mmottl/sqlite3-ocaml/pull/39 ?

@tspence What did you have in mind?

@shawnw I've just merged #39 by @tspence.  I haven't made a release yet so you may want to weigh in on the changes and maybe suggest new ones.  If required, could you please update this pull request to the current development branch?

@shawnw Could you please let me know whether you'd be fine with a new major release that does not yet contain your pull request?  Ideally, I'd only want to change APIs once, but if you need more time and don't think your changes would break the API about to be released, I'll make a new release.

I still don't understand what the issue is between this PR and that other one. That one seems to have yet another way to avoid having to explicitly construct variants, but it's also a bit more awkward to use and also doesn't seem to conflict with that part of mine. Would you rather each bit be split up into its own PR or something?

I've significantly rewritten parts of the API, mostly bind and column-related functions.  In particular, they now access the specialized functions already in the SQLite3 C-library.  This should make it easier to understand their behavior.  Please feel free to comment on these changes before I make the next release.


after_cluster_joining_userdata only allows one element

Describe the Bug
The validation of the variable after_cluster_joining_userdata expects the length of the array to be smaller than 2, but the definition says that it can contain commands in plural.
variable "after_cluster_joining_userdata" {
  type        = list(string)
  default     = []
  description = "Additional `bash` commands to execute on each worker node after joining the EKS cluster (after executing the `bootstrap.sh` script). For more info, see https://kubedex.com/90-days-of-aws-eks-in-production"
  validation {
    condition = (
      length(var.after_cluster_joining_userdata) < 2
    )
    error_message = "You may not specify more than one `after_cluster_joining_userdata`."
  }
}

Expected Behavior
The variable to be a string or the validation to allow more than 1 element.

@jchanam Sorry if this is confusing, but it follows the pattern that optional data is provided in a list of zero or one items. The userdata can be a multi-line string:
  after_cluster_joining_userdata  = [<<-EOT
    # After joining the cluster, execute a couple of commands
    echo foo
    echo bar
    EOT
  ]

Closing due to no response after recommending a solution.


Added functionality to close welcome screen when file is selected

Description
Welcome screen now disappears when the 'Open a file or folder' is pressed. This is done by adding a cancel handler parameter to the openDocument function which calls the openWelcome function once cancel is pressed.
Related Issue

#272

Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] I documented my code
[x] Review requested

Screenshots

Many thanks 👍 @lukepistrol

@all-contributors please add @dzign1 for code


Expand jira issue list --status to include querying for issues of different status

Please consider opening a discussion in ideas category if applicable. Your feature will be prioritized on based on the number of votes.

Is your feature request related to a problem? Please describe.
I can only list issues assigned to me with either any status, or a single status type.
Describe the solution you'd like
I'd like to view all issues assigned to me that are Blocked and In Progress, for example, with results in the same table without having to run two separate jira commands.
jira issue list --assignee scriptautomate --status Blocked,'In Progress'

Describe alternatives you've considered
Not sure what other alternative there would be.

@ankitpokhrel, I will handle this feature.

@ScriptAutomate You can use following command as a workaround until this feature gets released.
jira issue list -q "status in (Done, 'In Progress')"


RichTextLabel size not adjusted by (HBox/VBox/Grid)Container

Windows 10 64bit, Godot 2.0.1-stable
RichTextEditor sets size to 0 in HBoxContainer, VBoxContainer and GridContainer. The control size is not properly adjusted to the container.
Steps to reproduce:

Create new scene
Insert a main node (Node2D or Control)
Insert a HBoxContainer, a VBoxContainer, a GridContainer and a RichTextEditor
Insert a RichTextEditor to every container (3, there should be 4 RichTextEditors)
Enable BBCode in each RichTextEditor, enter some content
Optional, add a control to every container as a space filler (buttons, labels, whatever)
Start the scene

Result
RichTextEditors in the containers are not visible (the one outside works just fine). Additionaly, if you try to resize them in the editor:
VBoxContainer RTE snaps the size to 0 if the height exceeds one line,
HBoxContainer RTE can be resized, but the size is ignored,
GridContainer RTE same as VBoxContainer
Updates
Appearantly Containers use the MinSize value as the size (which is 0,0 by default). Setting MinSize makes the element visible, but it's still not adjusted to the text.

Looks like a min-size issue to me...

@bojidar-bg just checked, setting the min size fixes this (values higher than 0)
So I guess the containers just use the MinSize, does RichTextEditor lack the ability to determine it's size?

@WalasPrime ~ Well, Label, Button and all other control classes would set some internal min size, and would then pick the maximum of the two min sizes. Also, do you mean that just having a min size would make it resize to the text's length?

Oh it doesn't fix it in a sense that the control is adjusted, it's just visible now. I'll fix the previous posts.

I think this may be a more general issue, not just affecting RTE (not sure if the title could be changed). I have a similar issue with ConfirmationDialog, which also has a 0 size when inserted into a container. Manually setting the min size was able to get it visible properly.
But indeed, auto-sensing its min size should be needed.

Godot does not provide a width4height implementation to better adjust these kind of controls, so for now the only way is setting a minimum size.

For the reference, the behaviour described in the OP is still the same in the current master branch. Here's the scene resulting from the given steps to reproduce:
RTLinContainers.tscn.zip

Closing in favor of #18260, as it contains more information about the issue at hand.


python310Packages.osqp: 0.6.2.post0 -> 0.6.2.post5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.osqp is: "The Operator Splitting QP Solver"
meta.homepage for python310Packages.osqp is: "https://osqp.org/"
meta.changelog for python310Packages.osqp is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.6.2.post5 with grep in /nix/store/zcb6261j1394qq5kd4x1g0n6vw4ifswi-python3.10-osqp-0.6.2.post5

found 0.6.2.post5 in filename of file in /nix/store/zcb6261j1394qq5kd4x1g0n6vw4ifswi-python3.10-osqp-0.6.2.post5

Rebuild report (if merged into master) (click to expand)

40 total rebuild path(s)

20 package rebuild(s)

20 x86_64-linux rebuild(s)
20 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
python310Packages.cvxpy
python310Packages.osqp
python310Packages.qiskit-aer
python310Packages.qiskit-aqua
python310Packages.qiskit-finance
python310Packages.qiskit-ibmq-provider
python310Packages.qiskit-ignis
python310Packages.qiskit-optimization
python310Packages.scikit-survival
python39Packages.cvxpy
python39Packages.osqp
python39Packages.qiskit
python39Packages.qiskit-aer
python39Packages.qiskit-aqua
python39Packages.qiskit-finance
python39Packages.qiskit-ibmq-provider
python39Packages.qiskit-ignis
python39Packages.qiskit-optimization
python39Packages.scikit-survival

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zcb6261j1394qq5kd4x1g0n6vw4ifswi-python3.10-osqp-0.6.2.post5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.osqp https://github.com/r-ryantm/nixpkgs/archive/5d166c536c5c84cb311dd039d383f242659912a1.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zcb6261j1394qq5kd4x1g0n6vw4ifswi-python3.10-osqp-0.6.2.post5
ls -la /nix/store/zcb6261j1394qq5kd4x1g0n6vw4ifswi-python3.10-osqp-0.6.2.post5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

17 packages failed to build:

python310Packages.cvxpy
python310Packages.qiskit-aer
python310Packages.qiskit-aqua
python310Packages.qiskit-finance
python310Packages.qiskit-ibmq-provider
python310Packages.qiskit-ignis
python310Packages.qiskit-optimization
python310Packages.scikit-survival
python39Packages.cvxpy
python39Packages.qiskit
python39Packages.qiskit-aer
python39Packages.qiskit-aqua
python39Packages.qiskit-finance
python39Packages.qiskit-ibmq-provider
python39Packages.qiskit-ignis
python39Packages.qiskit-optimization
python39Packages.scikit-survival

2 packages built:

python310Packages.osqp
python39Packages.osqp

Maintainer pings
cc @drewrisinger for testing.

Superseded by #154969