Add Dockerfile

Fixes # .
Motivation

Add Dockerfile for Rosetta-cli as an alternative for binary installation
Solution

Open questions

@xiaying-peng Updated readme, in terms of Makefile, I'm thinking we can update it after images publishes to dockerhub, treating image as an arg in makefile looks weird. thoughts?


systemd service permissions

My logs are full of following messages:

Jun 28 14:51:59 servername systemd: Configuration file /etc/systemd/system/splunk.service is marked executable. Please remove executable permission bits. Proceeding anyway.
Jun 28 14:51:59 servername systemd: Configuration file /etc/systemd/system/splunk.service is marked world-inaccessible. This has no effect as configuration data is accessible via APIs without restrictions. Proceeding anyway.

This is due to the file /etc/systemd/system/splunk.service having the execute bit set for the owner and not read to world. Can this be corrected in the following code? Instead of mode 700, mode 644?
 template '/etc/systemd/system/splunk.service' do
    source 'splunk-systemd.erb'
    mode '700'
    variables(
      splunkdir: splunk_dir,
      runasroot: node['splunk']['server']['runasroot']
    )
  end

I’m adding the Type: Jump In GitHub label to this issue. This is a great issue for someone to get their feet wet with and we’d love a PR to resolves the issue.

Fixed and soon to be released


Do you have IMU data in KITTI?

I'm sorry to ask you something that is not related to SFMLearner, but I hope I can ask you about IMU data. Because My attention is the fuse of IMU and visual. But I can't download the IMU data from the KITTI. So here I ask you if you have KITTI ’s IMU data.

I'm sorry to ask you something that is not related to SFMLearner, but I hope I can ask you about IMU data. Because My attention is the fuse of IMU and visual. But I can't download the IMU data from the KITTI. So here I ask you if you have KITTI ’s IMU data.

Hello,did you have downloaded IMU data?


Adds Jenkins instance pool tests to tarmak_test.go

Includes Jenkins instance pool into tarmak_test.go
NONE

My concern is/was we did not call terraform validate on those stacks so the error was not catched.
You can have a look at my code but that will fail as terraform init blocks as it wants some connection to the state bucket
https://github.com/jetstack/tarmak/commit/1f00bd849459a0d910f3e4c63fdc1effc5d98a41

@simonswine Guess we should have some flag for controlling off/on line tests to skip or run this

I guess maybe if we don't configure the remote state (and use local) it should work, I will create an inssue for that
/assign @simonswine

/approve
/lgtm


Needs a license file specified

Hi -  great work but you should probably specify a license. I could not see one :)

Added BSD3 thanks!


Fix onOpen() and onClose() are not called when duration is 0

Fixes michu2k/Accordion#59

Thanks for reporting the bug @xemlock, but this one will not be fixed, as it is not a very significant bug.


[BUG] take defined with matcher will allways take from the buffer

Snippet to reproduce
	const chan = channel()
	yield put(chan, { type: 'OTHER_ACTION' })
	yield fork(function* () {
		while (true) {
			const action = yield take(chan, 'TEST')
			console.info('action', action)
		}
	})

	yield put(chan, { type: 'TEST' })
	yield put(chan, { type: 'JUST_ANOTHER_ACTION' })

If buffer is not-empty during a take effect channel will always take from it, completely ignoring the matcher argument.
Possible solution? implementing matcher arugment in buffer.take?

I'm not sure this should be considered a bug.
the matcher argument is not exposed as a public API. but used internally for the special case of resolving store actions yield take(ACTION). Store actions are handled internally using an eventChannel with no buffering. So when matcher is provided to the channel we'll always have an empty buffer


Can't start oracle

I'm trying to create oracle database in docker but now i'm stuck here. Oracle container isn't started as far as i can see in portainer.
Any ideas please?

root@ser-deb:~# docker container rm oracle
oracle
root@ser-deb:~# docker run --name oracle -p 1521:1521 -p 5500:5500 -v /home/sergentum/oradata:/opt/oracle/oradata oracle/database:<IP_ADDRESS>-ee

LSNRCTL for Linux: Version <IP_ADDRESS>.0 - Production on 11-FEB-2019 03:41:57

Copyright (c) 1991, 2016, Oracle.  All rights reserved.

Starting /opt/oracle/product/<IP_ADDRESS>/dbhome_1/bin/tnslsnr: please wait...

TNSLSNR for Linux: Version <IP_ADDRESS>.0 - Production
System parameter file is /opt/oracle/product/<IP_ADDRESS>/dbhome_1/network/admin/listener.ora
Log messages written to /opt/oracle/diag/tnslsnr/e4a722d1902a/listener/alert/log.xml
Listening on: (DESCRIPTION=(ADDRESS=(PROTOCOL=ipc)(KEY=EXTPROC1)))
Listening on: (DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=<IP_ADDRESS>)(PORT=1521)))

Connecting to (DESCRIPTION=(ADDRESS=(PROTOCOL=IPC)(KEY=EXTPROC1)))
STATUS of the LISTENER
------------------------
Alias                     LISTENER
Version                   TNSLSNR for Linux: Version <IP_ADDRESS>.0 - Production
Start Date                11-FEB-2019 03:41:58
Uptime                    0 days 0 hr. 0 min. 1 sec
Trace Level               off
Security                  ON: Local OS Authentication
SNMP                      OFF
Listener Parameter File   /opt/oracle/product/<IP_ADDRESS>/dbhome_1/network/admin/listener.ora
Listener Log File         /opt/oracle/diag/tnslsnr/e4a722d1902a/listener/alert/log.xml
Listening Endpoints Summary...
  (DESCRIPTION=(ADDRESS=(PROTOCOL=ipc)(KEY=EXTPROC1)))
  (DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=<IP_ADDRESS>)(PORT=1521)))
The listener supports no services
The command completed successfully

SQL*Plus: Release <IP_ADDRESS>.0 Production on Mon Feb 11 03:41:58 2019

Copyright (c) 1982, 2016, Oracle.  All rights reserved.

Connected to an idle instance.

SQL> ORA-01078: failure in processing system parameters
LRM-00109: could not open parameter file '/opt/oracle/product/<IP_ADDRESS>/dbhome_1/dbs/initORCLCDB.ora'
SQL> Disconnected
ORACLE_HOME = [/home/oracle] ? ORACLE_BASE environment variable is not being set since this
information is not available for the current user ID .
You can set ORACLE_BASE manually if it is required.
Resetting ORACLE_BASE to its previous value or ORACLE_HOME
The Oracle base remains unchanged with value /opt/oracle
/opt/oracle/checkDBStatus.sh: line 26: sqlplus: command not found
#####################################
########### E R R O R ###############
DATABASE SETUP WAS NOT SUCCESSFUL!
Please check output for further info!
########### E R R O R ###############
#####################################
The following output is now a tail of the alert.log:
tail: cannot open '/opt/oracle/diag/rdbms/*/*/trace/alert*.log' for reading: No such file or directory
tail: no files remaining

i tried to resolve as it fixed here https://github.com/oracle/docker-images/issues/756 but i got next issue:
LSNRCTL for Linux: Version <IP_ADDRESS>.0 - Production on 11-FEB-2019 05:15:59
Copyright (c) 1991, 2016, Oracle.  All rights reserved.
Starting /opt/oracle/product/<IP_ADDRESS>/dbhome_1/bin/tnslsnr: please wait...
TNSLSNR for Linux: Version <IP_ADDRESS>.0 - Production
System parameter file is /opt/oracle/product/<IP_ADDRESS>/dbhome_1/network/admin/listener.ora
Log messages written to /opt/oracle/diag/tnslsnr/093395ad0729/listener/alert/log.xml
Listening on: (DESCRIPTION=(ADDRESS=(PROTOCOL=ipc)(KEY=EXTPROC1)))
Listening on: (DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=<IP_ADDRESS>)(PORT=1521)))
Connecting to (DESCRIPTION=(ADDRESS=(PROTOCOL=IPC)(KEY=EXTPROC1)))
STATUS of the LISTENER
------------------------
Alias                     LISTENER
Version                   TNSLSNR for Linux: Version <IP_ADDRESS>.0 - Production
Start Date                11-FEB-2019 05:16:00
Uptime                    0 days 0 hr. 0 min. 0 sec
Trace Level               off
Security                  ON: Local OS Authentication
SNMP                      OFF
Listener Parameter File   /opt/oracle/product/<IP_ADDRESS>/dbhome_1/network/admin/listener.ora
Listener Log File         /opt/oracle/diag/tnslsnr/093395ad0729/listener/alert/log.xml
Listening Endpoints Summary...
  (DESCRIPTION=(ADDRESS=(PROTOCOL=ipc)(KEY=EXTPROC1)))
  (DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=<IP_ADDRESS>)(PORT=1521)))
The listener supports no services
The command completed successfully
SQL*Plus: Release <IP_ADDRESS>.0 Production on Mon Feb 11 05:16:00 2019
Copyright (c) 1982, 2016, Oracle.  All rights reserved.
Connected to an idle instance.
SQL> ORA-01078: failure in processing system parameters
LRM-00109: could not open parameter file '/opt/oracle/product/<IP_ADDRESS>/dbhome_1/dbs/initORCLCDB.ora'
SQL> Disconnected
Password: su: Authentication failure
#####################################
########### E R R O R ###############
DATABASE SETUP WAS NOT SUCCESSFUL!
Please check output for further info!
########### E R R O R ###############
#####################################
The following output is now a tail of the alert.log:
tail: cannot open '/opt/oracle/diag/rdbms/*/*/trace/alert*.log' for reading: No such file or directory
tail: no files remaining

That was a year ago and still nobody run into the issue, and nobody fixed it for 12 version, really?

Please remove the contents in /home/sergentum/oradata and try again.
The startup script is able to find a folder with the database name inside /home/sergentum/oradata but cannot open the parameter file:
SQL> ORA-01078: failure in processing system parameters
LRM-00109: could not open parameter file '/opt/oracle/product/<IP_ADDRESS>/dbhome_1/dbs/initORCLCDB.ora'

which suggests that something went wrong during the initial database creation and now you have a unclean state in /home/sergentum/oradata

Closing due to lack of response.


Add functionkey/mediakey ability

It would be cool if one could use Mediakeys or Functionkeys (FN+F7 for example to Skip) with Osu.Music!

I updated the clone and tried to use a hotkey several times but it just crashes the program for some reason. Everything else is running fine
https://user-images.githubusercontent.com/72462771/142771850-d18eda11-407b-48b6-b930-9dbfcc0d7bd3.mp4
!

Hmm... Did you changed any of the hotkeys? If you did, please send me screenshot. Meanwhile I check what could be a potential reason.

We found the fix! We had to delete the AppData/Romaing/osu.Music folder in order to see the Hotkey-settings.

Oh, thanks for the info. I've opened a separate issue #13 for this bug.


Skip select_next for single item

It would be nice to be able to have a completion key which, when there is more than one match, scrolls downward through the list (like select_next) but when there is only one match, accepts it (like select_and_accept). I've tried chaining these like this:
        ["<Tab>"] = { "select_next", "select_and_accept", "fallback" },

... but that didn't work how I expected, it just acts like select_next even when there is only one match available. I am using to completion being very simple and it's tricky to get used to have to use different keys for these.
This is similar to (but not identical to) the menu option for NeoVim's built-in completeopt.

I think the latter. It's possible right now with #cmp.windows.autocomplete.items but that's an internal API which won't be exposed long term. Ideally the list isnt coupled with the autocomplete window, and instead, moved into a separate module, which also handles selection.


Explicitly add references to package.json

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Expected or Desired Behavior
I am very happy that in 1.8.2, "for the react scaffolded projects, we now explicitly add office-ui-fabric-react version 6 to package.json".
Shouldn't you include in the scaffolding the other two common shadow references as well?

tslint
typescript

As you said, shadow dependencies are a bad thing and including them in the default scaffolding will help to eliminate these shadow dependencies and make easier the use of pnpm and rush.
Observed Behavior
tslint and typescript are common shadow references as they are not in the scaffolded package.json.
Steps to Reproduce
Run yo @microsoft/sharepoint and see the package.json

Good question @Rafaelki .  In this case, however, the answer is no, and we should probably add some additional documentation around it.  The rush-stack-compiler-x.y package brings in the appropriate typescript and tslist packages, as well as other build tools (webpack, etc.).  The goal here is that for tooling, you include the one package, and we make sure that everything matches correctly and is tested to work together.

I see! So basically tslint and typescript are no longer shadow references since rush-stack-compiler was added (1.8.0), right?
Thank very much @patmill!


Add NewReno congestion control.

Summary
NewReno congestion control algorithm is added in order to solve the problem of network congestion breakdown. NewReno congestion control includes slow start, collision avoidance, fast retransmission, and fast recovery. The implementation refers to RFC6582 and RFC5681.
In addition, we optimize the congestion algorithm. In the conflict avoidance stage, the maximum congestion window max_cwnd is used to limit the excessive growth of cwnd and prevent network jitter caused by congestion. Maximum congestion window max_cwnd is updated with the current congestion window cwnd and the update weight is 0.875 when an RTO timeout occurs.
Impact

Adds the configuration macro NET_TCP_CC_NEWRENO for NewReno, which relies on the fast retransmission macro NET_TCP_FAST_RETRANSMIT.
Add duplicate detection (dupacks) and use it to replace duplicate detection (wb_nack) in the  Fast Retransmission function.

Testing
We test the NewReno using sim in ubuntu 22.04 environment. Simulate a congestion environment by limiting the total uplink bandwidth and running tcp streams over the two SIMS.

Compile: Open NewReno macro NET_TCP_CC_NEWRENO and iperf3 before compiling.
Environment configuration: Open sim network, start two simulators and set them with different ip addresses; Use the tc tool to limit the inbound bandwidth of nuttx0.
Stream Testing: Open two iperf3 servers on ubuntu, using different ports; In both sim, start the iperf3 client respectively.
Comparison Testing: Compares the test results of enabling and disabling NewReno.

The test results showed that the total network throughput was significantly increased when NewReno congestion control was enabled, which was close to the actual total network bandwidth, and the rates of both sim devices were stable.

Hi @liqinhuixm please include the summary information in the log message. Please consider transforming this Testing selection in a small user guide at Documentation/ to explain people how to test this NewReno congestion control on NuttX.

Hi @liqinhuixm please include the summary information in the log message. Please consider transforming this Testing selection in a small user guide at Documentation/ to explain people how to test this NewReno congestion control on NuttX.

The newreno.rst file is added in the directory (Documentation/reference/os) to explain this function and how to test it.


Set the size of the preallocated output buffer

Output buffer by default preallocates 64KB of memory. When serializing
very small objects like Person in this test this is unnecessary and
costly (.NET will zero out allocated memory). Specifying the output size
to be more appropriate for the size of the data shows the actual cost of
serialization rather than redundant memset. In our tests Bond is faster
then protobuf.net for both serialization and deserialization.

Full disclosure: I'm the author of Bond at Microsoft.

I've added more information to the performance section of the C# docs: https://github.com/Microsoft/bond/commit/dcfe9fba28b60f9fa356064a45d6d42d71ddf7e7

@sapek thank a lot for the explanation.


check memory

check available memory and the running processes
testing only, won't get merged.

Coverage increased (+0.003%) to 34.14% when pulling e542d8852034a2a4ba3448c543e5a858011d7001 on alanren/check-memory into acefe7b081e101be911c776df06c0d36dd7ec13e on master.


NullReferenceException in LogManager.Shutdown()

Type: Bug
NLog Version: 4.3.3 (NuGet)
Platform: .Net 4.0
NLog config: (not applicable)
I have a console application which crashes on exit with a NullReferenceException in LogManager.Shutdown(). To reproduce:

Create a new console application project
Add the "NLog" NuGet package
Insert the following program:

class Program
{
    static void Main(string[] args)
    {
        try
        {
            if (true)   // here are some conditions which check the requirements to load the application
            {
                Console.WriteLine("Press any key to exit!");
                Console.ReadLine();
                return;
            }
        } finally
        {
            LogManager.Shutdown();
        }
    }
}

Note that there is a NLog config file, but is has a custom name. The program is exited before
LogManager.Configuration = new XmlLoggingConfiguration("...");
was executed.

What is the current result?
NullReferenceException in LogManager.Shutdown()

What is the expected result?
Shutdown() should be able to handle the absence of a default config file and should not throw an exception.

Did you checked the Internal log?
No, because there is no config file to turn it on.

Please post full exception details (message, stacktrace, inner exceptions)
No inner exception, the stack trace only contains LogManager.Shutdown() after the last frame of my own code.

Are there any work arrounds?

Is there a version in which it did worked?
Problem occurs with v2.x, v3.x and v4.x

Thanks! This is fixed in #1453


can't use cohere

i've set cohere as api provider, but when query codegpt, it seems use openapi for provider.
ps: both user and workspace are set to cohere

Exactly the same in my case.

Every time you make a change in settings, use the command Developer: Reload Window
Also, you must change the model to:
command-xlarge-nightly or command-medium-nightly

change model solved. thx~


Searching a JSON with special character in key

I am using alasql to search through a JSON object. But when I try to return values of a specific key (eg; "@type") it doesn't return any data.
Is it because the key name has special character? If so, how would I be able to process it.
Thanks!

Can you please try to use `@type` or [@type] and see if this helps?

@mathiasrw Yeah, [@type] works. Thanks!


function instead of function declaration in members section is not detected

I inadvertently used incorrect syntax on a member function declaration. In the base case, the compiler does not detect anything wrong. In an altered case, it generates an unrelated and unexpected error.
qx.Class.define("testapp.Application",
{
  extend : qx.application.Basic,

  members :
  {
    main : function()
    {
      var             map;

      this.base(arguments);

      map =
        {
          timestamp : null
        };

      this.doStuff(map);
    },

    doStuff(config)
    {
      let             arr = [];
      let             timer;

      config.timestamp = new Date().getTime();

      if (false)
      {
        arr.push(
          {
            timestamp : config.timestamp,
            location  : "someplace"
        });
      }
    }
  }
});

Note that doStuff(config) was supposed to be doStuff : function(config). As written, the compiler generates no error on this code. I believe it should. Furthermore, if the line that says if (false) is changed to if (true), then the following error is generated by the compiler:
[32,24] Unresolved use of symbol config.timestamp

It complains about the line inside of the map: timestamp : config.timestamp, but it has no complaint about the line above which assigns a value to config.timestamp.

I think that in the first instance, that's a feature of babel compiling es6 - it's a valid syntax and would be compiled to doStuff: function(config) {...} on the fly.  Ironically, I keep meaning to try the doStuff() {...} syntax and not got round to it!  Although it's alien to ES5, IMHO we should keep it because it is valid ES6.
It would be useful to be able to flag a file (or wildcard) as needing to be ES5 compliant, or as a warning however...although I'd have to find out how to detect that babel had enacted any ES6 transforms on a file.
With the second warning and not-warning, that's a bug for sure.

ES6 shorthand method definition! Cool! No, definitely don't flag it as an error. I love it, now that I know it exists. We just need to fix the warning/not-warning bug.


chore: cache Travis CI docker builds

Let's see if we can reduce the time it gets to build and test Etcher on
GNU/Linux.
See: https://giorgos.sealabs.net/docker-cache-on-travis-and-docker-112.html
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

I'm merging so we can see if the usbboot branch finally builds correctly.


feat(Jenkinsfile): override 'sh' to add colorized output

Can't wait for fancy terminal colors :rainbow:

Not a whole lotta color now, but there is color!  (and this will support fanciness down the line..) https://ci.deis.io/job/Deis/job/workflow-cli/job/PR-152/1/console


Bump seccomp version to be the same as one in runc repo

Looks like the runc folks bumped almost a year ago

https://github.com/search?q=repo%3Aopencontainers%2Frunc 2.5.4&type=code
https://github.com/opencontainers/runc/pull/3481/files

Context: query on slack - https://cloud-native.slack.com/archives/CGEQHPYF4/p1684243817550399

Should this be cherry-picked to 1.6.x and 1.7.x?

Since we dynamically-link libseccomp, do we know if distros are carrying libseccomp >= 2.5.4 yet?  Which distros do we lose support for by upgrading?


refactor: rewrite stories toggle tooltipicon unitinput

Summary
Type

Refactor

Summarise concisely:
What is expected?
Migrate stories Toggle TooltipIcon and UnitInput

The tooltip is misaligned on the toggle tooltip story:


Idea: Toggle Message Box Right Elements

Make the Stickers / Gift / Gif button toggleable. Perhaps do the same with the right click menu on messages?

You can now disable / enable all buttons by using the new module in #25.


Release date of v3.3.3

Hi,
Do you have any idea when v3.3.3 will get released? (Waiting for the maven proxy fix).
Thanks

I'm hoping to release 3.3.3 tomorrow (Sunday Oct 21st).

@jeremylong  Great! Thanks a lot

It is taking me a little more time due to an issue with the gradle plugin.  3.3.3. should be published this week.


Color assignments changed after Python 3.7 or ImageMagick update

Colors are no longer in the order they were before one of these two updates, and I find it quite jarring since they don't look near as good as they used to. Can we get them reordered for the new version of imagemagick?

This is fixed in the latest master. I’ll make a release when I’ve got some more time.


[Feature] crio config handling

What are you seeing
I saw this module supports crio as a container engine by default. I checked the code and did not find a piece of code handling the crio related configs /etc/crio/crio.conf and /etc/containers/storage.conf. I just wanted to ask, Is there a specific reason not to manage these files?
Maybe @ananace knows
What behaviour did you expect instead
These files are managed

It's partly a legacy thing, that particular design was grandfathered in from an earlier hyperkube module written before CRI-O switched to using built-in defaults and user-specified (/etc/crio/crio.conf.d/) overrides. So managing the main config in the module would've been a bit too fragile and version/setup -dependent, as it would've either needed to have all settings specified at the correct values for all possible versions and OSes, or only managing the existence of the file - which would've broken some earlier upgrades where creating an empty settings file would've meant that CRI-O couldn't launch to finish migrating its config.
Managing /etc/containers/storage.conf also showed similar issues between the systems I was developing against at the time, so that ended up being solved with file_line resources in profile classes instead, acting off of whatever default configuration the distribution packages deployed.

@ananace Thanks for explaining.
My suggestion would be to add $manage_crio_config and $manage_storage_config variables and manage the configs by default. Would that be a good solution? It would also be possible to give the user an option to decide between /etc/crio/crio.conf and /etc/crio/crio.conf.d/. Storage has no override directory AFAIK, but can take an alternative config location.

Managing the CRI-O configuration nowadays is probably reasonably easy; just ensure that the main config file and override directory exist along with a friendly comment about being Puppet-managed, might be interesting to include a purge option to also remove all unmanaged override files - probably along with a defined type (and hiera-friendly parameter) to add configuration data.
Managing the storage config might be a little more complex though, since it seems that you need to have a few options set in it - some of which also seem to differ between platforms and versions. (A quick check - with comments removed - across my three clusters shows at least four different versions of that file, though I've not done a deeper dive into the exact differences in the values)

With the latest changes merged, the crio configuration management should now be entirely done conf.d-style, but I've not had a chance to dig all that much deeper into storage.conf yet.


Experiment with incorporating owner balance into project balance meter

When a project's funding cycle changes and its balance has been fully withdrawn, its balance meter will go to to 0. This can be misleading. How to incorporate the amount that's been paid so far into the balance meter?

use the owner's wallet balance
show how much has been withdrawn in previous funding cycles.

A small dropdown like Zapper does it might be useful for projects where duration = 0.
Show owner balance ? on/off

implemented a basic fix for SharkDAO, but leaving this issue open to track conversation while it gets built out further


Zoom Gesture

Hello mate!
Can you please tell me how to implement zoom gesture in video player like brightness and volume control?

@taimoorMalik
see exoplayerview/src/main/java/com/jarvanmo/exoplayerview/gesture/VideoGesture.java.
you can star or fork this project for more information

Hello thanks for your reply i have star your project i also explore video gesture class but cant figure out how to enable zoom gesture, noob here :)

There's a lot of math here。I can't figure out how it works here.The gesture class works with a view and you should override its touch event if you want to implement your own zoom gesture.

I have follow this post for zoom gesture it work alone but when i try to implement it in this project it wont detect any zoom gesture https://github.com/Manuiq/ZoomableTextureView. Can you please look into it as its already done all the math required for zoom gesture.

do you disable the playback controller?The gesture in this project works with the playback controller.The controller will interrupt touch event.Maybe you should override the playback controller

Well i need all this project  gesture + zoom gesture from that project and i don't know how to do it can you please help me to do this?

I will have a try.

I override the touch event of  ZoomableTextureView as above.
Remove useless code in VideoGesture and OnVideoGestureChangeListener.
Override the touch event of  ZoomableTextureVie in the following states:MotionEvent.ACTION_DOWN,MotionEvent.ACTION_UP,MotionEvent.ACTION_MOVE.
VideoGesture only works when  ZoomableTextureView can'be zoomed.

Hello, "VideoGesture only works when ZoomableTextureView can't be zoomed" it means i can't uses gesture of brightness seek and volume when zoomed? right?

Can we handle it with a button in that case will it work?

Maybe

Hmm thank you very much for your help much appreciated your effort.

I just tried the above code and its not working still only volume etc gesture working!


Remove incomplete client, remove submit your own button

@ProbablePrime, thanks for your PR! By analyzing the blame information on this pull request, we identified @connor4312 and @SimonSchick to be potential reviewers

looks good.


Go to definition on callable property returns two definitions

Bug Report

🔎 Search Terms

go to definition
callable

🕗 Version & Regression Information
4.3.0-dev.20210316
⏯ Playground Link

Playground link with relevant code
💻 Code
interface ICallable {
    (): any
}
class K {
    callable: ICallable;
}

new K().callable()

Run go to definition on callable
🙁 Actual behavior
Two definitions are returned, for for K.callable and one for (): any in ICallable
🙂 Expected behavior
Just the definition for K.callable is returned

This seems like the right behavior for a method invocation? We don't know if you mean "show me the property" or "show me the overload that was selected"

When I think about this more I can sort ofunderstand the current behavior but it still catches me off guard.
The main place I see this in the VS Code code base is  for our Event type. For example, when I run go to definition on the event onDidChangeActiveColorTheme in this file:
https://github.com/microsoft/vscode/blob/a6d7d86e85799e6d243aed4430456627c76f7782/src/vs/workbench/api/common/extHost.api.impl.ts#L672
I want to navigate to where the  extHostTheming.onDidChangeActiveColorTheme property is set instead to the call signature of Event.
Perhaps this example is too specific and there are other cases where you really do want to go to the call signature? Would it be possible to have go to definition go to the property definition and some other operation (such as go to declaration) go to the original signature?

We can tweak it however we like; I'm not dead set on any particular behavior. Personally I don't find the "definition" vs "declaration" distinction to be particularly meaningful, but I can see the value in having one go one place and the other go elsewhere.
I think if you had overloads scattered in a few different declarations, which is somewhat common, it's useful to go to the right one. We didn't do this for a while and people found it very annoying. There should be some way to get to the selected overload; maybe this is just putting the caret between the method name and the parentheses.

@RyanCavanaugh I am really hit badly by this, see this flow for example:

I am not ending up on the type I want anymore...

@bpasero that looks like a totally different bug? OP is about returning two definitions when one is expected

@RyanCavanaugh I don't know if that is different, I filed https://github.com/microsoft/vscode/issues/119304 and it was closed as duplicate of this one. There are 2 definitions, but I have a setting enabled in VSCode to automatically go to the first definition. I would expect it should go to the most relevant.

It seems like that's a pretty bad setting to turn on.

😕

Can we start to treat this as a regression please. This was clearly working for years and has degraded significantly - any Event member in VS Code cannot be navigated to anymore.

okay, so something in VS Code has changed with https://github.com/microsoft/vscode/pull/117424. There is some sorting happening too early and then we don't know anymore what TS returned as the first element. The first is the most relevant for us and the one we would goto by default. So, assuming TS has the order well defined, e.g the type definition last, this might be all good


Val loss is Nan，but the train loss is normal？

I meet the same problem。How did you solve this problem，thanks a lot


Issue with vcgencmd python module, measure_clock.

when run from a script, measure_clock("arm") the output is the maximum clock speed of the CPU instead of the actual clock speed.
Example:
import vcgencmd
Hz = vcgencmd.measure_clock("arm")
print(Hz)
running using python3
pi@behemoth:~ $ python3 vcg.py
1000000000
Though when running in an interactive python3 shell:
pi@behemoth:~ $ python3
Python 3.7.3 (default, Jul 25 2020, 13:03:44)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

import vcgencmd
Hz = vcgencmd.measure_clock("arm")
print(Hz)
700000000

the odd issue with this the same bug is present in another Vcgencmd python module found at:
https://github.com/sushantnadkar/vcgencmd

How do you know that the CPU frequency isn't changing? Is it reproducible? Have you tried both uses an idle CPU and a loaded CPU? I'd be very surprised if the interactive vs non-interactive have different results.

I also checked when running the system command at the same time shows a different clock speed.
--
Listen to the silence let it ring on
On 6 Sep 2020, at 04:02, Nic McDonald<EMAIL_ADDRESS>wrote:

How do you know that the CPU frequency isn't changing? Is it reproducible? Have you tried both uses an idle CPU and a loaded CPU? I'd be very surprised if the interactive vs non-interactive have different results.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/nicmcd/vcgencmd/issues/3#issuecomment-687644638, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEESK7MO5PTXPTAREOJBCDSEJ6MTANCNFSM4QPQ4A5A.

@ivyvisors did you get this figured out?


Solved issue with Cordova 7.0.*

Hi!
I discovered an issue with the uglify process on Cordova Android 7.0.*, because the root folder to assets now is in platforms/android/src/main/assets/www
We must to change the line 46 for this:
wwwPath = path.join(platformPath, platform, 'app', 'src', 'main', 'assets', 'www');
Greetings ;)

Hey thank you for the comment.  I believe this is already fixed as of https://github.com/rossmartin/cordova-uglify/commit/eca730222c0bc9816c3b8f096c985fb999c13135 and is in 0.3.3


Rename uv-traits and split into separate modules

This is driving me a little crazy and is becoming a larger problem in #2596 where I need to move more types (like Upgrade and Reinstall) into this crate. Anything that's shared across our core resolver, install, and build crates needs to be defined in this crate to avoid cyclic dependencies. We've outgrown it being a single file with some shared traits.
There are no behavioral changes here.

Sorry about the diff 🤷♀


add emily freeman

I'm official! 🎉

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/emily-freeman.yml
:white_check_mark:Succeeded
View

advocates/media/profiles/emily-freeman.png
:white_check_mark:Succeeded
View

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

@emilyfreeman ,
looks good :) you will be covered soon most likely this week. Talking to other CDAs profile fixes.

 All bits are live now!
Enjoy the weekend :)


packing algorithm class is included in scheduler files

https://github.com/twitter/heron/blob/master/tools/rules/heron_client.bzl#L30 which is a duplicate of https://github.com/twitter/heron/blob/master/tools/rules/heron_client.bzl#L43. That said, lib/scheduler will have two jar about packing algos. Any idea why? @billonahill

why include heron-core (heron-dist) inside of client? internal build rule does not include it in client package https://github.com/twitter/heron/blob/master/scripts/packages/BUILD#L254

I don't know why the packing files are duplicated in heron_client.bzl but this seems incorrect to me.

Heron client includes core for running local cluster.

On Feb 21, 2017, at 10:17 PM, Bill Graham<EMAIL_ADDRESS>wrote:
I don't know why the packing files are duplicated in heron_client.bzl but this seems incorrect to me.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@kramasamy Good to know. Thanks!


Problems with ";"

When I export CSV Report and in the text have a ";" symbol, the report use other cell.
I mean the report take this symbol like a new line.

TableTools is now retired software and won't be seeing any updated other than security patches if they are required.
Buttons is its replacement. Could you try creating the issue with Buttons and if this is still a problem, please post a link to a test case in a new thread in the DataTables forums.
Thanks,
Allan

Thank you!
Awesome job!
Regards,
Antony Granados.
El mié., abr. 27, 2016 3:18 PM, Allan Jardine<EMAIL_ADDRESS>escribió:

TableTools is now retired software and won't be seeing any updated other
than security patches if they are required.
Buttons <//datatables.net/extensions/buttons> is its replacement. Could
you try creating the issue with Buttons and if this is still a problem,
please post a link to a test case in a new thread in the DataTables forums
<//datatables.net/forums>.
Thanks,
Allan
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/DataTables/TableTools/issues/108#issuecomment-215232332


Remove the elyra pipeline plugin with additional resource

Remove the elyra pipeline plugin with additional resource
Description
Team has decided to with hold elyra pipeline plugin inclusion, till the reference of Data Science Pipeline is not fixed.
Related-to: #90
Reverting(partially):
https://github.com/opendatahub-io/notebooks/commit/e95157becb5a5a0fb09215f54dcfc5b29b8de8b6
https://github.com/opendatahub-io/notebooks/commit/5f3701f1ccf6a07fa459d3f828a101e1f5c9caad
How Has This Been Tested?

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/test runtime-pytorch-ubi9-python-3-9-pr-image-mirror

/test notebook-jupyter-pytorch-ubi9-python-3-9-pr-image-mirror

/test notebook-cuda-jupyter-minimal-ubi8-python-3-8-pr-image-mirror

/approved

/approve


XML File Size Supported?

My TestResult.xml is 25MB.  Is that supported?  reportunit appears to hang.  I see "78823 tests processed..." as the last thing listed in the console.  The HTML files never get created.

After it processes tests, it will take a few more minutes to actually write everything to the file and save. But it will work.  There are some fixes planned for performance issues, shouldn't take so long in the upcoming versions.

I've waited > 3 hours.

I can probably send you my testresults.xml if you want to give it a go.

Yes that would help.  Its a big file not sure you will be able to email, can you upload it and send me the link.  You can email me at<EMAIL_ADDRESS>

I sent you an email with a link to the testResult file.  The HTML pages did eventually work (kind of).  It took maybe 10 hours.  Also chrome crashed on attempt to browser the tests from  TestResult_BankClient.html.

Did you get the 25MB test file (testresults.xml) okay?

Yes, thank you for that.  I haven't had a chance to test yet though.  I will do it tonight or tomorrow morning.  Will update you soon.

Thanks, man

I tested the file last night and have made changes to the parser.  This process now takes a second as compared to several minutes.  Still working on the markup generation, its still very slow.
Also, you must note that the result HTML file will be approx 25-30MB in size and will be quite slow and laggy to view because the js will have to run for thousands of elements.

got it.  I appreciate your attention.

The parser will be completely revamped.  I will launch the new version (in beta) in a few weeks.  There will be a lot of changes so would take some time..

I was able to create most of the NUnit parser which is completely Linq and it took 18 mins to process. The resulting file was 50MB - I can share it with you.


SQL compiler now accepts FunctionDecl which represent user defined functions

Simplified and generalized compiler to handle all function invocations in on case and support functions of arbitrary arity
Introduced Symbol to avoid having to remember to call toLowerCase in many places in compiler
Compiler will now return WrongArgumentCount instead of FunctionNotFound if user attempts to call coalesce with more or less than 2 arguments

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ce9932a). Click here to learn what that means.
The diff coverage is 91.5%.

@@            Coverage Diff            @@
##             master    #1917   +/-   ##
=========================================
  Coverage          ?   65.57%
=========================================
  Files             ?      376
  Lines             ?    18596
  Branches          ?      227
=========================================
  Hits              ?    12195
  Misses            ?     6401
  Partials          ?        0

Impacted Files
Coverage Δ

...ontend/src/main/scala/quasar/sql/SqlRelation.scala
85.29% <ø> (ø)

frontend/src/main/scala/quasar/sql/package.scala
96.33% <100%> (ø)

...main/scala/quasar/api/services/query/compile.scala
79.16% <100%> (ø)

foundation/src/main/scala/quasar/Predef.scala
86.84% <100%> (ø)

...main/scala/quasar/api/services/query/execute.scala
100% <100%> (ø)

core/src/main/scala/quasar/fs/mount/view.scala
97.52% <100%> (ø)

frontend/src/main/scala/quasar/functions.scala
87.5% <100%> (ø)

...ala/quasar/frontend/logicalplan/LogicalPlanR.scala
81.08% <100%> (ø)

frontend/src/main/scala/quasar/sql/parser.scala
96.96% <100%> (ø)

...in/scala/quasar/fs/mount/MountRequestHandler.scala
100% <100%> (ø)

... and 5 more

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 ce9932a...8017703. Read the comment docs.

@sellout I think I have addressed all your concerns. Let me know if there is anything else


Tags are not respected?

We tag our resources that we want to keep from nuking with key do-not-nuke and value true, then we add this to filter section - and for majority of resources it works. But for some resources it's ignored. For example we tagged a Sagemaker domain:

And using filter similar to this one:
accounts:
  '123456789':
    filters:
      SageMakerDomain:
      - property: "tag:do-not-nuke"
        value: "true"

Similar setup works for majority of other resources, yet here we're getting output
us-west-2 - SageMakerDomain - d-lkdfjldfadsf - [DomainID: "d-lkdfjldfadsf"] - would remove

is it a bug, or some omission on our side? How do we make it respect the tags?

@YuriGal it looks like SageMakerDomain doesn't have tag support, should be trivial for me to add, I'll take a look tonight.

That would be awesome! Would it be possible to add this to other similar resources? I know some resources can't be tagged at all in AWS, but if all of those that can could be filtered by tags it would be great.

Yes, as long as the resource supports tags, the resource in this project just needs to be identified as needing it's tags exposed and then filtering will work.

Not sure if I am doing something wrong, but with v3.0.0-beta.32 SageMakerDomain is always excluded, no matter tagged or not.

What happens exactly? Does it show up as "filtered by config"? Can you please provide the output and the config?

@YuriGal let me know when you get a chance. I'll see what I can do to replicate or test.

Ok, so to simplify testing I am using a basic config like this
regions:
  - global
  - us-west-2
  - us-east-1
  - eu-central-1
  - af-south-1
account-blocklist:
  - "123456789"
accounts:
  '987654321':
    filters:
      SageMakerDomain:
      - property: "tag:role:do-not-nuke"
        value: "true"
resource-types:
  targets:
    - SageMakerDomain

There is a sagemaker domain in us-west-2, but no matter whether it's tagged or not I am getting
Scan complete: 0 total, 0 nukeable, 0 filtered.

@YuriGal I broke it when I added tests. Fix inbound.

Thanks! And thank you for such a fast response time.

:tada: This issue has been resolved in version 3.0.0-beta.33 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

Looks like it's works properly for SageMakerDomain, thanks!
Would it be possible to do the same for other resources that support tags, but tags aren't exposed?

Ideally just need an issue with a list of Resources that are missing tags that support tags and I can look into it. It depends on the resource and the API as to how much effort it takes.

I think I can provide the list, just have to collect it.
Meanwhile I am afraid I found another bug, will open a separate issue.

Ok, so teams provided me with following list of resources that were tagged, but got nuked anyway
SageMakerUserProfiles
ECSCluster
ServiceDiscoveryNamespace
SQSQueue
CognitoUserPoolDomain
CognitoUserPoolClient
CognitoUserPool
CognitoIdentityProvider
ElasticacheSubnetGroup
ElasticacheCacheCluster

I know for sure that this is true for ECSCluster and SQSQueue, but didn't get a chance to verify other ones.

btw, (unrelated) question: does this nuke supports --max-wait-retries option?

It's suppose to, but looking at the options, it appears to be missing. The goal is to be 99% backwards compatible with the original, minus the nuke subcommand.
I'll fix this oversight, mind opening an issue? You'll get notified that way when it get's fixed.

:tada: This issue has been resolved in version 3.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[#3503] Add support for MSI to DotNet's samples and generators - bot essentials (part 2/2)

Addresses #3503

Note: The pipeline Samples-CI-PR-Pipelines-Runner is failing due to not finding the 4.15.0-daily.20211006.271232.c20cc39 version in NuGet, whereas this is only available in Artifacts or MyGet.

Note: Currently, MSI and SingleTenant support needs the version 4.15.x that's under development, therefore these PR changes will use the 4.15.0-daily.20211006.271232.c20cc39 version instead. Once the 4.15.x version is released, this will be updated.

Description
This PR updated the bot essentials samples adding support for MSI and SingleTenant App types.
Detailed Changes

Updated AdapterWithErrorHandler class to receive the BotFrameworkAuthentication in the constructor.
Updated the Microsoft.Bot.Builder.Integration.AspNet.Core dependency to the latest preview version.
Updated Startup class to create the BotFrameworkAuthentication.
Added the new settings MicrosoftAppType and MicrosoftAppTenantId to the appsettings.json file.

This PR updates the following samples:

08.suggested-actions
13.core-bot
14.nlp-with-orchestrator
15.handling-attachments
43.complex-dialog
45.state-management

Testing
The following images show the nlp-with-orchestrator bot working with the three app types: MultiTenant, SingleTenant, and MSI.

/azp run Samples-CI-PR-Pipelines-Runner


Automate prepared statements

This PR aims at reducing the boilerplate associated with preparing statements in a Dialect.

Depends on canonical/go-dqlite#308 .

Marking as ready since dqlite released the fix.

On my machine this change makes the benchmarks unstable because of canonical/dqlite#675 .


Update memory config for CockroachDB configuration

With the previous setting the cockroach process is killed by the OOM-killer.

thanks!


GestureDetector docs: video probably incorrect

The video on this page probably is incorrect.
https://github.com/flutter/flutter/blob/d79295af24c3ed621c33713ecda14ad196fd9c31/packages/flutter/lib/src/widgets/gesture_detector.dart#L130
Video URL: https://flutter.github.io/assets-for-api-docs/assets/widgets/gesture_detector.mp4
According to the code example, the setState defines lights as on. The video wouldn't have the proposal to illustrate this?

Confirmed.
When this video was first submitted, the light bulb will be turned on. https://github.com/flutter/assets-for-api-docs/blob/431c1b893e9155d12be207aa79e1f23cb0319399/assets/widgets/gesture_detector.mp4
Now this video is basically a still image.
@gspencergoog I traced the culprit to be https://github.com/flutter/assets-for-api-docs/pull/124.
Also I wonder if we still need this video at all. It seems to be best changed to dartpad.

In my opinion, a dartpad as an example is better.

@gspencergoog Have you figured out why the generation no longer works as intended though? Does it affect other videos?

No, I was going to take a look at that separately, but I forgot that I set the PR to close the bug. Reopening.

I guess this issue should be closed

@gspencergoog is there anything remaining about this issue?

Nope.


fix: RTE + Form + JSONForm reskinning

Fixes UI inconsistencies in RTE, Form and JSON Form reskinning. Also fixes one small color bug for checkbox in disabled state.
Fixes #15171
Fixes #10977
Fixes #10986
Fixes #16879
Type of change

Bug fix

How Has This Been Tested?

Manually tested

Checklist:

[ ] My code follows the style guidelines of this project
[ ] 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
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Tested this for:

UI inconsistencies in RTE, Form and JSON Form reskinning
Fixes one small color bug for checkbox in disabled state

/ok-to-test sha=a0abade

/ok-to-test sha=72156eb

/ok-to-test sha=d02b5ea

/ok-to-test sha=ab3bd31

/ok-to-test sha=937431f


Use CSS + semantic dom to show/hide the elements.

Eliminates dependency on data.json XHR.
To productionize, there should be an index.php that takes the
data_fragment.html generated by the cloud function into the
location-list.

https://findthemasks-semantic-d-sdi1zq.herokuapp.com/index-test.html

This will also help with #208  because the donation list will show up immediately since it's already in the DOM.

This is a good idea, but my main concern here is maintainability. Consolidating index.html, give.html, and index.php+data_fragment.html, and on the script side: locations-list-map.js and locations-static.js. This should be done first before adding a new variant that will feature drift as independent devs continue to work on issues.

Very good observation. If we were to merge this, my hope would be to delete index.html and locations-list-map.js once we've verified this all runs on the main site.
The question then just becomes merging give.html in.

In plain terms, what is the primary goal (or set of goals) for this PR? I. Guessing speed up page load and centralize the two current pages, index + give, correct? Just confirming since it looks like the ticket starts out with implementation details (probably missing something).

(1) load faster, esp on mobile.
(2) if we push a js bug (due to cache skew, etc) data still available.

K, awesome, that’s a great idea. If I’m not mistaken, it may also really reduce the overhead of generating the HTML client side (with all the template interpolations and concatenation into the main list). Oh, plus you can take advantage of server-side gzip compression of HTML, so it still downloads quickly.
Not at a computer to verify but: If not already, ensure you’ve got a bullet point to verify that the server’s configured (or CloudFlare) to gzip that content so it gets dumped down the pipe more quickly.

cloudflare is already doing gzips and reencodes though I disabled the minify cause being able to read the code live was nice.

Just choosing to log some of the results from my quick analysis here for posterity. Maybe not the best PR/issue to do this in, but I think this one has a lot of potential.
My conclusion is that out of the ~3.5s page render time (which includes the ~100ms-200ms network latency for data.json)*, about ~2.5s is spent inside createContent looping over 2k separate addMarkerToMap JS calls which involves lots of HTML parsing, then finally DOM render + CSS reflow.
So, I think the ideal solution appears to be a combination of multiple approaches for best performance:

Pre-render the HTML list here which will save lots of extra time currently delegated to the client right now in createContent (not including addMarkerToMap), see PR #223 (this ticket)
Delegate addMarkerToMap to end of createContent and try to add later all at once or ideally in chunks, see PR #274 for an approach to this

Also: Looking at addMarkerToMap there are some optimizations that could be performed there that may help with optimizing load speed, including:

Calling $.i18n just once for each label ahead of time and caching labels for current language instead of calling it ~2,000 x 4  = 8,000 times
Pre-linkify HTML in the data.json file (the linkifyHtml() func called over and over for the marker pop-up)
Create a single click listener function, named function in a higher scope, and then attach that to each marker (so less memory is used and potentially less CPU overhead when attaching listeners)

* Variables: I'm on a high speed cable connection (320mbit/s) with 4.3Ghz boost and it's still fairly CPU bound.

With the infinite scroll / partial loading, this is now obsolete.

Oh my #396... that is a beautiful thing. It's so fast now! 🚀


LINK-1932 | have only one field to define vat percentage

Description
Have only one field in registration/event form to define vat percentage for the price groups.
Closes
LINK-1932
Screenshot

LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr359-ui.dev.hel.ninja :rocket::rocket::rocket:

e2e tests result is failed for https://linkedcomponents-pr359-ui.dev.hel.ninja :crying_cat_face::anger::boom::boom:

e2e tests result is success for https://linkedcomponents-pr359-ui.dev.hel.ninja :laughing::tada::tada::tada:

Noticed a possible bug while testing: if I create or edit a registration and try to create registration price groups for it without having first created them for the event, I get the error "None is not a valid choice" when I try to save the changes.

Very good catch. Vat percentage was missing from the payload. Added the vat percentage and issue should be fixed now.

LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr359-ui.dev.hel.ninja :rocket::rocket::rocket:

e2e tests result is success for https://linkedcomponents-pr359-ui.dev.hel.ninja :laughing::tada::tada::tada:


added Added and Dropped fields into balancer update done info

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (6373d76) 66.81% compared to head (9f6f344) 39.12%.

:exclamation: Current head 9f6f344 differs from pull request most recent head e3e3bd5. Consider uploading reports for the commit e3e3bd5 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #892       +/-   ##
===========================================
- Coverage   66.81%   39.12%   -27.70%
===========================================
  Files         258      207       -51
  Lines       24319    21160     -3159
===========================================
- Hits        16249     8278     -7971
- Misses       7233    12535     +5302
+ Partials      837      347      -490

Flag
Coverage Δ

?

go-1.17.x
?

go-1.20.x
39.05% <80.43%> (-27.41%)
:arrow_down:

go-1.21.x
39.04% <80.43%> (-27.40%)
:arrow_down:

integration
?

macOS
39.01% <80.43%> (+0.12%)
:arrow_up:

ubuntu
39.00% <80.43%> (+0.06%)
:arrow_up:

unit
39.12% <80.43%> (+0.09%)
:arrow_up:

windows
39.05% <80.43%> (+0.06%)
:arrow_up:

ydb-22.5
?

ydb-23.1
?

ydb-23.2
?

ydb-23.3
?

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

Files
Coverage Δ

log/driver.go
0.00% <0.00%> (-65.71%)
:arrow_down:

internal/balancer/balancer.go
32.34% <84.09%> (-50.36%)
:arrow_down:

... and 191 files with indirect coverage changes

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


docs: fix FluentBadge custom css

🙋 Documentation Request
https://www.fluentui-blazor.net/Badge
the scoped css sample does not work
::deep > fluent-badge {
    --badge-fill-highlight: #ffd800;
    --badge-fill-lowlight: #c8c6c4;
    --badge-color-highlight: #000;
    --badge-color-lowlight: #fff;
}

in the browser tools I see the warning that the css variable is not found
💁 Possible Solution
Remove the scoped version and use the global version in the docs?
fluent-badge {
    --badge-fill-highlight: #ffd800;
    --badge-fill-lowlight: #333;
    --badge-color-highlight: #000;
    --badge-color-lowlight: #fff;
}

I am new to blazor and fast so I am unsure if the scoped css variables should work at all

Hi,
And welcome to Blazor, Fluent UI and Fast. Sometimes the browser tools gets it wrong. It shows a var as grayed out but it actually is found and works. Think that is the case here as well. This is what I see in Edge Canary:

The yellow marked id is what Blazor uses to scope the variables.  You can see the ::deep has been replaced with that id in the right side of the image.

It doesn't work in edge canary for me either

Ok, let's take a step back. What are you trying to achieve? And does setting BackgroundColor and Color not work (see https://www.fluentui-blazor.net/Badge#appearances)?

I'll create a sample repro and get back to you

hi @vnbaaij
I found the issue.
I had to change
::deep > fluent-badge {
    --badge-fill-highlight: #ffd800;
    --badge-fill-lowlight: #c8c6c4;
    --badge-color-highlight: #000;
    --badge-color-lowlight: #fff;
}

to
::deep fluent-badge {
    --badge-fill-highlight: #ffd800;
    --badge-fill-lowlight: #c8c6c4;
    --badge-color-highlight: #000;
    --badge-color-lowlight: #fff;
}

Thanks


Is it possible to selectively cancel tasks in an actionChannel? Ie cancel the 3rd task out of 5 running ones.

Description
Is it possible to cancel the ith running async API task with redux saga? For instance, my task is postApiData(data) and a user calls that a few times, just with different data posted.
I was using React-Thunk to call APIs async, but now I need to call APIs AND cancel any queued API requests/dispatches if the user cancels the queued API task.
The current cancel example, and most of what I've found online focus on a single primary/main task to cancel.
Hypothetically, it'd be something like below. I run all items in all api calls in a queued manner, A -> B -> C. But, I can trigger a cancel for any item, ie Cancel B. End result = A -> C.

function* watchRequests() {
  while (true) {
    const {payload} = yield take('REQUEST')
    const task = yield fork(handleRequest, payload)

    // Where {ID} is the unique passed ID to track the event, and thus trigger a frontend action when the user click's that item.
    yield take('CANCEL_TASK_{ID}')
    yield cancel(task)
  }
}

function* handleRequest(payload) { ... }

After thinking about it, I don't think it's really possible for not only redux saga, but most of the recommended side-effect managers. Something about how it's not really the common workflow.

Greetings!  The example you provided looked pretty good to me, what about it doesn't work for your needs?

Hello from a fellow developer.
When I have to cancel a specific task, I usually use a Map Object to keep record of the tasks.
Then if needed I make sure to find that individual task and cancel.
You can find a naive example but similar to your case here: https://codesandbox.io/s/redux-saga-cancel-task-76ufws?file=/src/App.tsx
I hope it helps.
Regards,
Vlad
De la: Codex101 @.>
Trimis: joi, 30 martie 2023 08:46
Către: redux-saga/redux-saga @.>
Cc: Subscribed @.***>
Subiect: [redux-saga/redux-saga] Is it possible to selectively cancel tasks in an actionChannel? Ie cancel the 3rd task out of 5 running ones. (Issue #2374)
Description
Is it possible to cancel the ith running async API task with redux saga? For instance, my task is postApiData(data) and a user calls that a few times, just with different data posted.
I was using React-Thunk to call APIs async, but now I need to call APIs AND cancel any running API call if the user cancels the queued API task.
The current cancel example https://redux-saga.js.org/docs/advanced/TaskCancellation , and most of what I've found online focus on a single primary/main task to cancel.
Hypothetically, it'd be something like
function* watchRequests() {
while (true) {
const {payload} = yield take('REQUEST')
const task = yield fork(handleRequest, payload)
// Where {ID} is the unique passed ID to track the event, and thus trigger a frontend action when the user click's that item.
yield take('CANCEL_TASK_{ID}')
yield cancel(task)

}
}
function* handleRequest(payload) { ... }
—
Reply to this email directly, view it on GitHub https://github.com/redux-saga/redux-saga/issues/2374 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ANFOKQ4X5PKHZ2J6IY36J2TW6UMZJANCNFSM6AAAAAAWMZGNQU .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/ANFOKQ2OLW6ATGQU3MOFGB3W6UMZJA5CNFSM6AAAAAAWMZGNQWWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHGEKUM3E.gif Message ID: @.*** @.***> >

Greetings! The example you provided looked pretty good to me, what about it doesn't work for your needs?

Don't think you can dynamically assign the cancel string right?
    yield take('CANCEL_TASK_{ID}')
So if I called the below, it wouldn't auto map the request to the cancelled task.
    dispatch({type: 'REQUEST_123'})
    dispatch({type: 'CANCEL_TASK_123'})

Hello from a fellow developer. When I have to cancel a specific task, I usually use a Map Object to keep record of the tasks. Then if needed I make sure to find that individual task and cancel. You can find a naive example but similar to your case here: https://codesandbox.io/s/redux-saga-cancel-task-76ufws?file=/src/App.tsx I hope it helps. Regards, Vlad De la: Codex101 @.> Trimis: joi, 30 martie 2023 08:46 Către: redux-saga/redux-saga @.> Cc: Subscribed @.> Subiect: [redux-saga/redux-saga] Is it possible to selectively cancel tasks in an actionChannel? Ie cancel the 3rd task out of 5 running ones. (Issue #2374) Description Is it possible to cancel the ith running async API task with redux saga? For instance, my task is postApiData(data) and a user calls that a few times, just with different data posted. I was using React-Thunk to call APIs async, but now I need to call APIs AND cancel any running API call if the user cancels the queued API task. The current cancel example https://redux-saga.js.org/docs/advanced/TaskCancellation , and most of what I've found online focus on a single primary/main task to cancel. Hypothetically, it'd be something like function watchRequests() { while (true) { const {payload} = yield take('REQUEST') const task = yield fork(handleRequest, payload) // Where {ID} is the unique passed ID to track the event, and thus trigger a frontend action when the user click's that item. yield take('CANCEL_TASK_{ID}') yield cancel(task) } } function handleRequest(payload) { ... } — Reply to this email directly, view it on GitHub <#2374> , or unsubscribe https://github.com/notifications/unsubscribe-auth/ANFOKQ4X5PKHZ2J6IY36J2TW6UMZJANCNFSM6AAAAAAWMZGNQU . You are receiving this because you are subscribed to this thread. https://github.com/notifications/beacon/ANFOKQ2OLW6ATGQU3MOFGB3W6UMZJA5CNFSM6AAAAAAWMZGNQWWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHGEKUM3E.gif Message ID: @.** @.***> >

Was hoping could avoid the manual queue system implemention, but, it doesn't seem possible. Thanks for the solution :)


Highlighted uncommented function call python file program

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

VS Code Version:
OS Version:

Steps to Reproduce:

Python file basic program
run program using vs code code runner extension
comment out a function call like get_opponent_comparison(gs_url, mil_url)
uncomment and run NameError: name 'get_opponent_comparison' is not defined
only works if you unhighlight and re run the file
if remains highlighted it wont work

/causedByExtension

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


interp: extend dot debugging

As dotty does not seem to be readily available on MacOS, this PR extends
the ability to debug with dot, through the new YAEGI_DOT_CMD env var,
which allows one to specify the command to process the generated dot
graphs.
For example, to print the AST to svg:
YAEGI_AST_DOT=1 YAEGI_DOT_CMD='dot -Tsvg -ofoo.svg' yaegi ./foo.go
If any of YAEGI_AST_DOT or YAEGI_CFG_DOT is set, but YAEGI_DOT_CMD is
not defined, the default is to write to a .dot file next to the Go
source file.

ah I've just realized if both YAEGI_AST_DOT and YAEGI_CFG_DOT they would overwrite each other as they would write to the same default file name. I need to add a differentiator in the filename.


Dashboard metrics view crashes when viewing metrics of a Node.js app running with a Dapr sidecar

Full repro case in this repo: https://github.com/asimmon/aspire-node-metrics-bug

An error banner with "An unhandled error has occurred" appears at the bottom of the metrics view if I try to display the metrics of an instrumented Node.js express app running with a Dapr sidecar:
builder.AddNpmApp("nodeapp", Path.Combine("..", "nodeapp"), "watch")
    .WithDaprSidecar()
    .WithHttpEndpoint(port: 3000, env: "PORT")
    .WithEnvironment("NODE_TLS_REJECT_UNAUTHORIZED", builder.Environment.IsDevelopment() ? "0" : "1");

Here's the error:

The error does NOT appear if:

I remove the Dapr sidecar, OR
I pass new DaprSidecarOptions options with an AppId property set to something else than the Node.js app resource name, like so:

builder.AddNpmApp("nodeapp", Path.Combine("..", "nodeapp"), "watch")
    .WithDaprSidecar(new DaprSidecarOptions
    {
        AppId = "nodeapp-renamed" // <-- FIX
    })
    .WithHttpEndpoint(port: 3000, env: "PORT")
    .WithEnvironment("NODE_TLS_REJECT_UNAUTHORIZED", builder.Environment.IsDevelopment() ? "0" : "1");

With one of these two changes, the metrics are shown:

@JamesNK

Fix https://github.com/dotnet/aspire/pull/3861


Fix logout bug

Fixes #148

Same issue occurs on my end, where the "Uh no" pages still appears. Maybe we can explicitly setIsLoading(true) at the start of the logout function then setIsLoading(false) at the end of the logout function?

Don't think it'll work because the state will only be set at the end of the function or something. Anyways, I think adding setIsLoading(true) to the logout function as well as removing auth from the landing page seems to fix it.


Allow db.js to work in Service Worker context

Web Workers don't have a Window

I'm having trouble getting a Service Worker demo up and running (I haven't done anything with service workers).
My concern is around the introduction of the global global variable.

I'm closing this as it'll be replaced by #161


MIDI_JINGLE packet does not work on the JavaScript client.

Works fine on the Java client.
Only quest reward screen is using this atm.

I'm going to rename the midi_jingle variables for clarity, my bad. delay isn't the time to start the jingle, it's the time to continue the music it replaced. If you send 0 it won't continue music tracks, and the webclient is set to loop midis so that's why it sounds strange.
midi_jingle("quest complete <tostring($random)>", 0);
$delay (time to end/time to next track) is in ms (unique per jingle), but it isn't like you open VLC and get the length of the track since there can be empty notes or properties their synthesizer doesn't understand. quest complete 2 should be like 7000. The Java client will play empty noise, the JavaScript client may restart the track instead of empty noise.
So I'll look into the looping difference, but you should send a time here. There also may be a bug with restarting the music twice after resuming, checking that out too


Support coloring the current line's gutter

Requested in a theme extension https://github.com/Tyriar/vscode-theme-sapphire/issues/12#issue-243072245

Ignore this, you can do it via the editor.renderLineHighlight setting 😄


Add BuildBuddy to MNF

@bootstraponline suggested that we join - excited to get involved!

Happy to have you!


Switched order of AzureAsyncOpenAI to AsyncAzureOpenAI

Switched order of AzureAsyncOpenAI to AsyncAzureOpenAI
to match name in repo
https://github.com/openai/openai-python/blob/main/src/openai/lib/azure.py

Good catch, thank you for contributing!


Report proper Jenkins status into PR

Type of change

Bugfix

Description

Report correct PR message in case job failed or was aborted
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests profile=smoke

:x: Test Summary :x:
TEST_PROFILE: smoke
TEST_CASE:
TOTAL: 18
PASS: 18
FAIL: 0
SKIP: 0
BUILD_NUMBER: 789
BUILD_ENV: oc cluster up


Update readme for create and deploy roles

Update readme for create and deploy roles to resolve #135

Looks good now.

Thanks @marehler!


What is "PreheatControllerDelegate"?

I saw in your example, there is a PreheatControllerDelegate.
After I installed Preheat, cleaned and build, the compiler shows PreheatControllerDelegate is undeclared. (I have import Preheat)
And I can't find the documentation about it, what is it and how to fix this?

Fixed it, it no longer exists in v2


 This was not occurring on the original catcher. I have created a demo project to demonstrate the issue.
To Reproduce
Steps to reproduce the behavior:

Clone https://github.com/HugoCSWork/catcher_2_integration_test_bug
run the integration test flutter test integration_test/main_test.dart
Test will fail as expected but will cause the integration tests to get stuck

Expected behavior
Test will complete in terminal showing one integration test failure
Device

OS: MacOS and Linux
catcher_2 version: 1.0.0

Checklist

[x] I have read and followed the entire README and it has not provided the solution I need.
[x] I have provided all the information I can (incl. auth URL etc.)

Thanks for the MRE! I will take a look at it in a few hours when I am home.
I suspsect that this might be very related to #10 and I fear that some internal changes inside Flutter's runApp function are the real culprit in both cases.
In the mean time, can you observe the same behaviour in Flutter 3.13.7 or even older (like 3.13.0 and 3.10.7 or something?)

Thanks for the quick response! I've done some quick testing with FVM using version 3.13.7 and 3.10.7 but am still experiencing the same issue.
When using the previous catcher it was on flutter version 3.7.8 but that is incompatible with catcher 2.

Alright, thanks for the testing and reporting back! I will see what I can do then. Catcher 2 has only very few internal changes to Catcher, so it will most likely be some change between Flutter 3.7.x and Flutter 3.10.x. Maybe, I will find something suspicious.

I was not able to reproduce this exact issue, because I had another one.
Please see whether the test fails properly on your machine in https://github.com/HugoCSWork/catcher_2_integration_test_bug/pull/1

The fix should now be available as part of version 1.0.1


Introduce WebDepsInstall

Using a mvnpm.json file to manage web dependencies
Can install in a different node_modules directory
Re-install only what's needed (and delete what's unused).
Resilient if the json file is broken
Allow to have typed (mvnpm/webjars) dependencies in the same bundle
Support for grouped mvnpm deps
Tests
Remove need for apache common-text dependency

See WebDepsInstallerTest to see how it behave

I am testing it in some projects before we merge..


Group capstone: set up your Kanban board

Group capstone: set up your Kanban board
We are two people in our team.
Here is the link.

Hi @Team,

Great job so far! 👏
You've done well setting up the Kanban board, however, there are some issues that you still need to work on to go to the next project, but you are almost there!
To highlight 🎯

No linter errors.✔
Nice Project setup. ✔
Descriptive PR. ✔

Required Changes ♻
Check the comments below.

:point_up_2:  Hey, the above tasks are group tasks but one team member is assigned to this task. please, kindly update this.

[ ] Please, kindly tag the names of each team member e.g. Team member A @example.

Optional suggestions

N/A

Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.

Thank you for your feedback @zieeco, we fixed the changes required.

Hi @alvaroiporre and @romans-adi ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉 🎉 🎉
Highlights
✔ Kanban board setup correctly with all cards assigned to team members
✔ Good issue Description with correct info and link
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Unable to install python bindings in Docker container

I tried my best to install the python bindings in docker container but for some reason it always fails.
Here is the log:
-- Looking for lzma_lzma_preset in /usr/lib/x86_64-linux-gnu/liblzma.so - found
-- Found LibLZMA: /usr/lib/x86_64-linux-gnu/liblzma.so (found version "5.2.4")
-- Found BZip2: /usr/lib/x86_64-linux-gnu/libbz2.so (found version "1.0.8")
-- Looking for BZ2_bzCompressInit
-- Looking for BZ2_bzCompressInit - found
-- Found ZLIB: /usr/lib/x86_64-linux-gnu/libz.so (found version "1.2.11")
-- kenlm library not found; if you already have kenlm installed, please set CMAKE_LIBRARY_PATH, KENLM_LIB or KENLM_ROOT environment variable
-- kenlm utils library not found; if you already have kenlm installed, please set CMAKE_LIBRARY_PATH, KENLM_UTIL_LIB or KENLM_ROOT environment variable
-- kenlm model.hh found in /opt/kenlm/include/kenlm/lm
-- Found kenlm: /opt/kenlm/include
-- Found kenlm (include: /opt/kenlm/include, library: KENLM_LIB-NOTFOUND;KENLM_UTIL_LIB-NOTFOUND;/usr/lib/x86_64-linux-gnu/liblzma.so;/usr/lib/x86_64-linux-gnu/libbz2.so;/usr/lib/x86_64-linux-gnu/libz.so)
-- Found PythonInterp: /usr/bin/python3 (found version "3.8.5")
-- Found PythonLibs: /usr/lib/x86_64-linux-gnu/libpython3.8.so
-- Performing Test HAS_FLTO
-- Performing Test HAS_FLTO - Success
-- LTO enabled
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
KENLM_LIB
    linked by target "fl-libraries" in directory /flashlight
KENLM_UTIL_LIB
    linked by target "fl-libraries" in directory /flashlight

-- Configuring incomplete, errors occurred!
See also "/flashlight/bindings/python/build/temp.linux-x86_64-3.8/CMakeFiles/CMakeOutput.log".
See also "/flashlight/bindings/python/build/temp.linux-x86_64-3.8/CMakeFiles/CMakeError.log".
Traceback (most recent call last):
  File "setup.py", line 111, in <module>
    setup(
  File "/usr/lib/python3/dist-packages/setuptools/__init__.py", line 144, in setup
    return distutils.core.setup(**attrs)
  File "/usr/lib/python3.8/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/usr/lib/python3.8/distutils/dist.py", line 966, in run_commands
    self.run_command(cmd)
  File "/usr/lib/python3.8/distutils/dist.py", line 985, in run_command
    cmd_obj.run()
  File "/usr/lib/python3.8/distutils/command/build.py", line 135, in run
    self.run_command(cmd_name)
  File "/usr/lib/python3.8/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/lib/python3.8/distutils/dist.py", line 985, in run_command
    cmd_obj.run()
  File "setup.py", line 56, in run
    self.build_extensions(ext)
  File "setup.py", line 105, in build_extensions
    subprocess.check_call(
  File "/usr/lib/python3.8/subprocess.py", line 364, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['cmake', '/flashlight', '-DCMAKE_LIBRARY_OUTPUT_DIRECTORY=/flashlight/bindings/python/build/lib.linux-x86_64-3.8', '-DPYTHON_EXECUTABLE=/usr/bin/python3', '-DFL_BUILD_STANDALONE=OFF', '-DBUILD_SHARED_LIBS=ON', '-DFL_BUILD_LIBRARIES=ON', '-DFL_BUILD_CORE=OFF', '-DFL_BUILD_TESTS=OFF', '-DFL_BUILD_EXAMPLES=OFF', '-DFL_BACKEND=CUDA', '-DFL_LIBRARIES_BUILD_FOR_PYTHON=ON', '-DFL_LIBRARIES_USE_CUDA=ON', '-DFL_LIBRARIES_USE_KENLM=ON', '-DFL_LIBRARIES_USE_MKL=ON', '-DCMAKE_BUILD_TYPE=Release']' returned non-zero exit status 1.

RUN pip3 install -r requirements.txt

RUN wget https://arrayfire.s3.amazonaws.com/3.8.0/ArrayFire-v3.8.0_Linux_x86_64.sh
RUN sh ArrayFire-v3.8.0_Linux_x86_64.sh --include-subdir --prefix=/opt

# install kenlm
RUN git clone https://github.com/kpu/kenlm.git && \
    cd kenlm && git checkout 0c4dd4e && \
    mkdir build && cd build && \
    cmake ..  -DCMAKE_BUILD_TYPE=Release \
             -DCMAKE_INSTALL_PREFIX=/opt/flashlight \
             -DFL_BACKEND=CUDA \
             -DArrayFire_DIR=/opt/arrayfire \
             -DKENLM_ROOT=/opt/kenlm \
             -DFL_LIBRARIES_BUILD_FOR_PYTHON="ON" \
             -DPYBIND11_PYTHON_VERSION="3.8" \
             -DCMAKE_POSITION_INDEPENDENT_CODE="ON" && \
    make install -j12
RUN cd flashlight/bindings/python && pip3 install .

# install torch for cuda 11
RUN pip3 install --pre torch torchaudio -f https://download.pytorch.org/whl/nightly/cu111/torch_nightly.html -U

# install fairseq
RUN git clone https://github.com/pytorch/fairseq.git && \
    cd fairseq && \
    git checkout v0.10.2 && pip3 install .

Any help is appreciated. :)

Please use KENLM_ROOT instead of KENLM_ROOT_DIR.

Realized it sometime after posting this issue and thus closed it :) Thank you for the help :)

So what's the solution to this question? I've tried to add export KENLM_ROOT=/path/to/kenlm/include to my ~/.bashrc  file(I'm not a root user) but still get the same problem.

It should be export KENLM_ROOT=/path/to/kenlm/ also make sure that you didd source on bashrc before building fl


Add cgroupsV1 flag to nodepools

This allows falling back to cgroups v1 for clusters > 1.18.
The node pool now has a cgroupsV1 boolean flag to bring up nodes with cgroups v1 enabled
Fallingback to cgroupv1 is possible without a reboot since flatcar 3033.2.4: https://www.flatcar.org/docs/latest/container-runtimes/switching-to-unified-cgroups/

Apparently this is not needed anymore as we don't have pressing bug reports about cgroupsv2 with recent kubernetes versions


RS-321: Bump collector version post-jsoncpp move

For the Collector team
Please feel free to take over and edit this PR and change COLLECTOR_VERSION to anything you find reasonable. The only thing I seem to need is that it points to commits not earlier than 21eae4f5c78015a6ffb9c77ebacb709ffedd47c0.
Description
See https://srox.slack.com/archives/CFMQ5C2TT/p1664365943486639 for context.
This is supporting change to unblock https://gitlab.cee.redhat.com/stackrox/rhacs-midstream/-/merge_requests/234
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added - no, relying on existing ones.

No need in the following because the collector version will be bumped again at the release time.

[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

Testing Performed

Relying on CI.

/test all

/test all


feat: upgrade clickhouse to v24.4

Description
This PR upgrades the clickhouse version to v24.4. This version includes support for Recursive CTEs, the QUALIFY clause, and join performance improvements. See more information.

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

I don't think we want to move to a non-LTS version. 24.3 is the latest LTS

@Ian2012 Even though your pull request wasn’t merged, please take a moment  to answer a two question survey so we can improve your experience in the future.


Stop Ollama in ubuntu linux

hello guys , I am trying to stop ollama service followihg the guide here https://github.com/jmorganca/ollama/issues/690
but  as follow seems not working :
pgrep ollama
>123
sudo kill 123

I tried finding the service in the ps list. As you can se is not working

As follow my machine

It registers itself as a service on my machine, try systemctl stop ollama.service and to start it again you can systemctl start ollama.service.

thanks

It looks like your question was answered so I will go ahead and close it now. If you think there is anything we left out, reopen and we can address. Thanks for being part of this great community.

I know this is not the proper solution to kill the service, but killing the service as suggested above didn't work for me.
I ended up using sudo killall -s 9 ollama - only thing that worked for me.

Use Ctrl + d or /bye to exit.


Wrong Test dataset on VoxCeleb1 (?)

Just to confirm:
https://github.com/speechbrain/speechbrain/blob/44ce741de00d56082749600b287df9378d75e27c/recipes/VoxCeleb/SpeakerRec/speaker_verification_plda.py#L175-L178
on L175, test_data is loaded from the corresponding CSV, but on L178, it is overwritten with the filtered enrol_data.
Should it not be: test_data = test_data .filtered_sorted(sort_key="duration") ??
The EER does not change, but I am wondering about the notation.

Right, this had no impact of the performance but it is something definitely to fix. I actually already fixed in the development branch. Thanks noticing it!


Rebrand Office 365 to Microsoft 365: User Story 1737442, Part 10

Fixes #1737442
Addresses rebranding "Office 365" to "Microsoft 365".
I'll add a comment when the PR is ready for review. I'll leave the PR open for 48 hours and address any feedback. After that, I'll sign off for merge.
Thanks -

This PR is ready for review.
You're welcome, but not required, to review these proposed rebranding changes. I'll leave the PR open for 48 hours and address any feedback. After that, I'll sign off for merge.

@Rick-Anderson ... FYI ... In the repo, I see some links that have "office365" segments.
I see one more change in this vein for the naming of a AAD security role (Office apps administrator). I'll verify that they changed it in the Azure portal and then change the doc. I'll ping u on that @damabe or report back here either way. I'll look at that now.

No 🎲🎲 @damabe ... They didn't change the role name. I've opened a new issue to address it when they do.
https://github.com/dotnet/AspNetCore.Docs/issues/19694

@guardrex Please do report that to them. I don't have any real interaction with the team.

@Rick-Anderson ... I have an AAD engineering contact. Should I reach out to him, or would you take it from here?

Try your contact and let me know if you need help.

I sent him a message offline.
btw - I also inquired in the message on the AAD built-in Administrator Roles table of Object Ids. I plan to open an issue next week on the Azure docs repo to see if we can get this table ...
https://docs.microsoft.com/en-us/aspnet/core/blazor/security/webassembly/azure-active-directory-groups-and-roles?view=aspnetcore-3.1#aad-adminstrative-role-group-ids
... moved over there. Then, I can link to it. We really shouldn't have to maintain that ... it's not an ASP.NET Core concept.


Adds StructSlice features

StringSlice will generate slice of string based on struct field name
StringSliceUnique will generate a unique slice of string based on struct field name
NoAllocStringMap will generate map[string]struct{} of string based on struct field name

Codecov Report

Merging #9 (add46e7) into main (544098d) will decrease coverage by 2.38%.
The diff coverage is 96.61%.

@@             Coverage Diff             @@
##              main       #9      +/-   ##
===========================================
- Coverage   100.00%   97.61%   -2.39%
===========================================
  Files            2        2
  Lines           25       84      +59
===========================================
+ Hits            25       82      +57
- Misses           0        1       +1
- Partials         0        1       +1

Impacted Files
Coverage Δ

sutil.go
97.59% <96.61%> (-2.41%)
:arrow_down:

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 544098d...add46e7. Read the comment docs.


[Enhancement]: Ensure Leader Node can Scale Vertically, no downtime

Version
cndi v1.13.4
kubeseal v0.21.0
terraform v1.5.5
Please provide a summary of the enhancement you are proposing:
Prevent the loss of uptime or data when a leader node is scaled vertically.
Please provide the motivation or use case for this enhancement:
Scaling a leader node is likely a common thing, and downtime shouldn't be the result
How can we best workaround this issue so far?
No response
How would you approach solving this problem within CNDI?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

If this solution involves setting a leader node using array index 0 in infrastructure.cndi.nodes for the initial deployment, and not setting a node as role: leader that is totally fine.
This may be required because if a leader is upgraded in place, it may not be the leader when it comes back online.


[Pass Infra] Tuning Pass API

Overview
This PR lands the tuning pass support proposed in our discussion https://github.com/tlc-pack/relax/issues/71#issuecomment-1104563398.
Due to its density, I will split its delivery into two or three chunks.
This PR mainly contains the fundamental functionalities, and later PRs will focus on the further integration with MetaSchedule and improving the code examples to show its usage.
What's New

class for Choice/Knob/Trace, default functions: By using this API, users can write a tuning pass in both Python/C++ side.

Each tuning pass will has its own generation function, evaluation function, and select function. Default functions are provided in the API.
Mock code snippet to show the basic usage - the sequence of code may not make sense.

def transform1(mod):
    # do transformation
    return new_mod
def transform2(mod):
   # do another transformation
  return new_mod
choices = {"c1": Choice(transformation1), "c2": Choice(transformation2)}
knob = Knob("TestKnob", choices)
# create a empty trace
trace = Trace(mod)
# add decision to the trace
trace.add(knob, "c1")
# when printing the trace, it will show the sequence of decisions applied.
print(trace)

# we can generate the candidate traces. In this case, it generates two candidates.
candidates = default_generate_candidate(knob, trace)

# we can also provide eval_passes for joint-optimization.
# this will return two candidates that are chosen in a tournament fashion with eval passes.
candidates = default_generate_candidate(knob, trace, eval_passes)

# evaluate with MetaSchedule builder/runner
default_evaluate(candidates, "llvm", ctx)

# pick the best one
best_trace = select_best_candidate(candidate)

See the tests/python/relax/test_tuning_pass.py for the better examples.
Each pass performs without the need to know what other passes are doing - it only uses its incoming trace/IRModule and Choices to generate candidates, evaluate them, and select the best one. This will help alleviating the complexity of joint-optimization significantly given that consideration of interaction between optimizations has known to be extremely difficult.

Each pass is traceable if it is written with tuning pass API.
trace mode: By providing an initial trace, a pass infra pipeline can start the trace mode.

# Start the trace mode
with PassContext(trace=Trace(mod)):
    optimized_mod = seq(mod)

# Trace mod will implicitly make non-traceable pass traceable by applying a default wrapper knob with a single choice
# However, we can explicitly provide a set of passes that we want to make traceable - other passes won't be wrapped.
with PassContext(trace=Trace(mod), make_traceable=["FoldConstant"]):
    optimized_mod = seq(mod)

trace stack: each pass pipeline manages a trace stack to deal with the different granularities of candidates (e.g., IRModule, Function, operators, ...). Whenever the granularity changes, we can push the current trace and start new trace. Once the new trace is done, we can pop the stack to continue the previous trace.
This API also works with Sequential, ModulePass, FunctionPass, DataflowBlockPass.
Pipeline customization

# Say search space of TuningPass1/2 is s1/s2 respectively.

# 1. Two sequential tuning passes: search space grows additively
Seqential([TuningPass1(), TuningPass2()])
# -> Search space: s1+s2

# 2. Joint optimization: search space grows combinatorially
Seqential([TuningPass1(eval_pass=TuningPass2())])
# -> Search space: s1*s2

# Since joint-optimization expands search space rapidly,
# it is highly recommended to apply two tuning passes in sequential
# if they are orthogonal to each other

# You can also provide heuristic passes anywhere.

What's Next

Further integration with MetaSchedule

MetaSchedule also has its Instruction (similar to knob) and trace
Since either one is not strictly superset for another, it would require some thoughts.
e.g., Instruction in MetaSchedule is specifically designed for kernel tuning and expresses a probabilistic language.

Database
Better code example

This would require infrastructural improvement. e.g., modular compilation, pattern match

Remark
Since this is quite experimental design, I believe there can be many things to improve. Please do not hesitate to ask any question or concerns if you have any - I would love to think about and reflect them.
Thanks!
cc. folks who had discussions online/offline:
@YuchenJin @junrushao1994 @psrivas2 @yongwww @tqchen @ZihengJiang @comaniac @hypercubestart @slyubomirsky

Thank you for the feedback at the relax dev meeting.
@ZihengJiang I think your suggestion for candidate generation function seems more generic so I changed it to handle multiple knobs.
@YuchenJin For the erroneous measurements (e.g., build, runtime errors), we should return the large enough number to get ignored at candidate selection phase as we discussed yesterday. I realize previous commit was using assertion. This is fixed now.

Merged, thanks @sunggg for the great effort!!


Failed setup error

Can no longer start integration due to failed to setup error.

Please search before posting.


docs: json_path_exists function

What's Changed in this PR
As title. Waiting for https://github.com/GreptimeTeam/greptimedb/pull/4881 to update the case when json is null.
Checklist

[x] Please confirm that all corresponding versions of the documents have been revised.
[x] Please ensure that the content in sidebars.ts matches the current document structure when you changed the document structure.
[ ] This change requires follow-up update in localized docs.

What scenarios does the json_path_exists solve?

For me it can test the structure of a json value without caring about its value. . It's a T-SQL function actually.


Remove Python 2 Unicode conventions

This PR cleans-up and removes various Python 2 Unicode conventions.

Coverage remained the same at 87.725% when pulling 39d1a5b0603173dde86c9c6700474b7626320738 on mwtoews:un-unicode into a89c61157097cc1d561e589c1091e750fecaf9e9 on Toblerity:master.

Coverage remained the same at 87.725% when pulling 39d1a5b0603173dde86c9c6700474b7626320738 on mwtoews:un-unicode into a89c61157097cc1d561e589c1091e750fecaf9e9 on Toblerity:master.


Switch to a federated serverlite extension

Continue the switch to a federated serverlite extension:

https://github.com/jupyterlite/jupyterlite/pull/104
https://github.com/jupyterlite/jupyterlite/pull/352

TODO

[x] Switch back to 0.1.0-alpha.x versioning
[x] Add liteExtension key to the package.json to indicate this is not a regular lab extension
[ ] Add docs to the readme
[ ] Fix CI
[ ] Fix releaser config
[ ] Book name on PyPI

Just thinking out loud: the happy path might be:

labextension build

puts stuff in p5_kernel/labextension

setup.py sdist bdist_wheel

puts stuff in $CRAZYTOWN/share/jupyter/labextensions

pip install jupyterlite-p5-kernel

puts stuff in $PREFIX/share/jupyter/labextensions

jupyter lite build

finds it

add to federated_extensions
looks in package.json

find it's a serverlite_extension

add to lite_extensions

at run-time

hand to the JupyterBackEndApplication
don't hand to the JupyterFrontentEndApplication

easy, right!?

import { JavaScriptKernel } from '@jupyterlite/javascript-kernel'

Should we also sharedPackage for @jupyterlite/javascript-kernel, at least for the time being?

Should we also sharedPackage for @jupyterlite/javascript-kernel, at least for the time being?

Yes sounds good :+1: At least if we continue shipping it by default. Otherwise it would be using its own copy of @jupyterlite/javascript-kernel which should also be fine for now.

Just thinking out loud: the happy path might be:

Yes, which should almost correspond to the current state of https://github.com/jupyterlite/jupyterlite/pull/352.
I'll finish this PR and cut a first release of p5-kernel-lite to PyPI so we can test on https://github.com/jupyterlite/jupyterlite/pull/352
For the name of the federated kernel, we could go for:

jupyterlite-p5-kernel
p5-kernel-lite

The second sounds good, but would be fine with the first one too. Later we can imagine having xeus-lua-lite which might look better than jupyterlite-xeus-lua.

The second sounds good, but would be fine with the first one too

jupyterlite-p5-kernel is actually similar to what many lab extensions have been using (jupyterlab-myextension). So going with that name would also be fine.

Should be good to go for a first iteration. We can iterate in follow-up PRs.


added intermediate resources

I added resources that will help beginners transition into intermediate flutter devs

This is great! Thanks for adding intermediate resources. Could you add advanced resources as well?

Sure. I'll add some more.