Lazy load grpc in OTEL grpc exporters

Followup on https://github.com/open-telemetry/opentelemetry-js/issues/4145
To avoid issues with init/instrumentation sequence OTLP grpc exporter should load grpc lazy (e.g. on first export).

Self-assigning as I'm working on this as part of a exporter-rewrite.

Hi，I would like to ask when this bug will be fixed？

@ZRainc I'm planning for this to land next month.
Due to various reasons, exporters are currently hard to work with. I'll soon open PRs that will re-structure them in a way that makes it easier to fix bugs and add features. The bugfix will then follow these PRs.

@ZRainc I'm planning for this to land next month.
Due to various reasons, exporters are currently hard to work with. I'll soon open PRs that will re-structure them in a way that makes it easier to fix bugs and add features. The bugfix will then follow these PRs.

Sounds Great, Thanks

Can't wait to have this one solved!
It's a blocking issue when working with Next.js 14.x:
 ○ Compiling /instrumentation ...
 ⨯ ./node_modules/.pnpm/@grpc+grpc-js@1.9.13/node_modules/@grpc/grpc-js/build/src/call.js:21:0
Module not found: Can't resolve 'stream'

https://nextjs.org/docs/messages/module-not-found

Import trace for requested module:
./node_modules/.pnpm/@grpc+grpc-js<EMAIL_ADDRESS>./node_modules/.pnpm/@grpc+grpc-js<EMAIL_ADDRESS>./node_modules/.pnpm/@opentelemetry+exporter-trace-otlp-grpc@0.46.0_@opentelemetry+api<EMAIL_ADDRESS>./node_modules/.pnpm/@opentelemetry+exporter-trace-otlp-grpc@0.46.0_@opentelemetry+api<EMAIL_ADDRESS>./node_modules/.pnpm/@opentelemetry+sdk-node@0.46.0_@opentelemetry+api<EMAIL_ADDRESS>./node_modules/.pnpm/@opentelemetry+sdk-node@0.46.0_@opentelemetry+api<EMAIL_ADDRESS>./node_modules/.pnpm/@opentelemetry+sdk-node@0.46.0_@opentelemetry+api<EMAIL_ADDRESS>./src/instrumentation.ts```


Will translation plug-ins be added?

Checklist

[X] I've checked that there is no other issue about this feature request.
[X] This issue contains only one feature request.
[ ] The title of this issue accurately describes the feature request.

Feature description
Can you upload a translation message plugin?
Automatically translate your language into the target language when sending a message.
Translate the message on the channel from a certain language to a specified language.
Why do you want this feature?
I am a user of other languages, and I can't speak English.
Some people may not know English either.
Additional information
No response

Sorry, this repository is not for plugin requests


Add Loading Spinner to Angular Web App

Description:
As a contributor, I want to see a loading spinner when the web app is loading so that I know the page is still loading and not frozen.
Tasks:

[ ] Create a loading spinner component.
[ ] Use a popular UI library such as Bootstrap or Material UI, or create a custom component.
[ ] Add appropriate animations and styles to make it visually appealing.
[ ] Implement the loading spinner in the app.

My Steps:

[ ] Verify that the loading spinner displays and hides correctly.
[ ] Test the spinner with various network conditions (e.g. slow network speeds).
[ ] Include screenshots or gifs of the spinner in action.
[ ] Seek approval from team members.

Feedback from existing codebase:
Use Material UI of Angular.
However as this library is 5 years old, team can certainly decide to use a new UI library.

Note: To pick up this item later


Bug: Data multiple selection BUG

Chat2DB Version
3.0.14
Describe the bug
Double-click the table name to open the data list. Click the number "1" on the left, then hold down Shift, and click the number "10" on the left, and other rows will be randomly selected.

try new version


Add a flip column order button

Been using a bookmarklet to flip the order of columns in Mapki.
This is very useful for objects with many versions where we need to scroll far to the right of the page.
Specific example here https://osm.mapki.com/history/way/69412370
Would you be able to implement something similar by any chance?

Yeah that would work as well

This is deployed.
Each deep history page now has a "most recent" link, which jumps to the rightmost column:

This is perfect! Cheers for the quick update


Making console container customizable

Hi, again.
I noticed that current default(debian) console doesn't install man pages.
It was installed on the debian console of rancheros.
I found It's because Burmilla's debian container is based on -slim image.
But I want to use BurmillaOS for personal development server.
So I need man page sometimes.
Slim image is for container image, not for console.
So, it's suitable alternative for rancheros's "default" console,
but I need "(plain)debian" if so.
Anyway, how to customize default console ?
We can modify Dockerfile  and build image.
But how to install that container ?
Is it easy at this moment ?
p.s.
I think it's useful if we can override install source of console container.
I prefer not using custom docker registry, but specify source repository of console container image.
For example,
console: build
console-repository: https://github.com/someone/my-custom-console

clones user's repo & build customized console image on installation.
Isn't that cool ?  :-)

One reason why we switched default console to Debian based is that now you can use apt-get to install additional packages.
Alternatively you can build totally customized console like it is described on https://burmillaos.org/docs/installation/custom-builds/custom-console/

I'm sorry for the late response.

One reason why we switched default console to Debian based is that now you can use apt-get to install additional packages.

I'm afraid I think you misunderstand my point.
I'm using the Debian console from the beginning when I started to use rancheros.
I don't want to repeat what I wrote but, Debian's -slim container is not suitable for console container.
Rancher didn't use -slim image.  I believe -slim for the console is not suitable for human use.
*-slim image is built to use inside some app containers.
For example, it didn't install man page even though an additional package has man page inside.
It's defined by *-slim image's dpkg configuration.

Alternatively you can build totally customized console like it is described on
https://burmillaos.org/docs/installation/custom-builds/custom-console/

Did you really read the page above?
It doesn't describe how to build a customized console.
Just to describe how to switch available consoles and those which are not available anymore.
I'm happy with Debian console because I'm a Debian user.
But the people who love Centos or Ubuntu don't want to use Debian console, right?
I prefer we can build & use our own console.
That's what I want to write on this ticket.

OK, I found the documentation about this below.
https://burmillaos.org/docs/system-services/custom-system-services/#creating-your-own-console
Also, I've  read  implementations around these and
perhaps, I understand how to build & use my own image.
I haven't tried yet, but it seems I can even use rancher os's console images like below.
rancher:
  repositories:
    console:
      url: https://raw.githubusercontent.com/rancher/os-services/master

It requires pushing image to docker repository.
I feel it's not handy but implementing custom image build is a bit complicated (i.e. it requires git).
If I'm not satisfied with the current method, I might try to implement this.

Yes it should be possible to use RancherOS consoles on also. And used architecture heavily utilises docker registry so you need to have one.
What comes to Debian vs Debian slim it mostly just define starting point. You should be able to change dpkg config on way that man pages gets included before installing additional packages.

Hi, now I published this.
https://github.com/benok/burmilla-os-console
Could you please take a look?

Looks good method for me. Now you can support whatever special consoles you need and we don't need support them on there.
If you want you can also open pull request which adds link to your project to https://github.com/burmilla/burmilla.github.io/blob/master/content/docs/installation/custom-builds/custom-console.md

Thank you for your comment.

If you want you can also open pull request which adds link to your project to
https://github.com/burmilla/burmilla.github.io/blob/master/content/docs/installation/custom-builds/custom-console.md

OK, I want to do this within this week.

Hi, I'm sorry tooooo be late. :-)
I've recently updated my unofficial custom console images.
I updated Alpine, Ubuntu,  Fedora to latest, and checked these consoles a little bit, and I found no problems.
(But currently, only tested within v2.0.0-beta5, though.)
And also submitted the doc's pull request.


Make a CodeSandbox playground once package is published

https://codesandbox.io/s/cssgui-example-w1c9h5


gist:directs

rdfs:comment has a spelling mistake i.e. Teh set of actualtors that a controller can affect

Good catch, thank you for pointing this out.


[Bug]: Triton shared does not build "out of the box"

The instructions for building Triton shared in the README are wrong --
"This repo doesn't build by itself and must instead by built from within a Triton repo where it is included as a submodule. To add the shared middle-layer in your Triton build do export TRITON_CODEGEN_TRITON_SHARED=1 before invoking your build."
To build Triton shared you need to do two things --

Third party backends are commented out in triton/python/setup.py and will not be built unless lines 312-315
are uncommented.

The environment variable TRITON_CODEGEN_TRITON_SHARED is used by get_codegen_backends() in setup.py to determine the directory name for the plugin. The environment variable uses an underscore, "triton_shared", while the git repo uses a dash, "triton-shared". The directory must be renamed to "triton_shared" or it will not be built.

This issue can probably be merged with #76. Since early December last year, triton refactored 3rd party plugin integrations. The triton commit that will work with what we have is 56c284cf7e39f249cdf1d8d5dba7892deb0286d6. We used to be a submodule of triton and got removed as part of that refactor. With the old build step, you won't have this issue since triton-shared is already named as triton_shared under triton/third_party.
All of this will get resolved once these two PRs are merged:

https://github.com/microsoft/triton-shared/pull/87
https://github.com/openai/triton/pull/3007

* [Update CPU backend to work with latest triton #87](https://github.com/microsoft/triton-shared/pull/87)

* [Support building out of tree plugins openai/triton#3007](https://github.com/openai/triton/pull/3007)

Are we expecting both to be merged soon?

* [Update CPU backend to work with latest triton #87](https://github.com/microsoft/triton-shared/pull/87)

* [Support building out of tree plugins openai/triton#3007](https://github.com/openai/triton/pull/3007)

Are we expecting both to be merged soon?

@zhaoshiz The PR to support building 3rd party plugin hasn't been merged yet. But I have gone ahead and use a temporary branch as the current triton submodule to unblock the project for now. The reference CPU backend has been merged. Everything is back to green now. You can take a look at the new README for instructions on how to build. The steps might change slightly in the future depending on the direction we decide to go in the ongoing PR. Thank you!

Fixed by #87


Any chances of supporting Bluetooth classic/LE on Android/iOS platforms?

This would be the perfect package that can integrate with everything if Bluetooth classic/LE is supported. Is there a plan for that?

You can use this library to get the corresponding byte[] you want to print and then connect yourself via Bluetooth and send the data. Let me show how I did so in one of my Android projects:
private static string bluetoothPrinterAddress = "00:01:02:03:0A:0B"; //replace with your BT Address
private static UUID PRINTER_UUID = UUID.FromString("00001101-0000-1000-8000-00805F9B34FB");

private static BluetoothDevice getDevice(BluetoothAdapter bluetoothAdapter)
{
    foreach (BluetoothDevice device in bluetoothAdapter.BondedDevices)
        if (device.Address == bluetoothPrinterAddress)
            return device;

    return null;
}

private static BluetoothSocket getSocket(BluetoothDevice device)
{
    BluetoothSocket socket = device.CreateRfcommSocketToServiceRecord(PRINTER_UUID);
    socket.Connect();
    return socket;
}

private static void sendData(byte[] bytes, BluetoothSocket socket)
{
    var o = socket.OutputStream;
    o.Write(bytes, 0, bytes.Length);
    o.Close();
}

public static void Print(byte[] byteArray)
{
    BluetoothAdapter btAdapter = getBTAdapter();

    //Bluetooth is turned off
    if (btAdapter is null)
        return;

    BluetoothDevice device = getDevice(btAdapter);

    //Printer not found
    if (device is null)
        return;

    BluetoothSocket socket = getSocket(device);
    sendData(byteArray, socket);
}

@igorocampos  this is awesome, I would love to support this since the library supports basically all other printing types already!  Do you think you could implement this as a BluetoothPrinter type like we have for the other printer types?  How are you using Bluetooth from within the Android namespace, is it built against xamarin or something?  I wonder if we could do this in a cross-platform way?

@lukevp this code example was from a xamarin project alright. I don't own the devices anymore, but they were an Android POS with a BT printer embedded. But once you get the Bluetooth socket from whatever hardware you have, it should be straight forward to send the bytes over.

Yes, this is what I was planning to implement. Just asking if there are plans to support it right in the library. Thanks for the code snippet, will definitely use this library for my purpose. Great library 😁

@igorocampos this is awesome, I would love to support this since the library supports basically all other printing types already! Do you think you could implement this as a BluetoothPrinter type like we have for the other printer types? How are you using Bluetooth from within the Android namespace, is it built against xamarin or something? I wonder if we could do this in a cross-platform way?

Coming back at this one again, @lukevp would you consider haveing 32feetas a dependency for BT comms?
If yes, I think we can definitely do a cross-platform Bluetooth printer, what do you think?


[!] CocoaPods could not find compatible versions for pod "Crisp":

I'm trying to install Crisp on a React Native 0.64.0 project
I have tried the last 3 versions of react-native-crisp-chat-sdk without success
Error when I want pod install :
[!] CocoaPods could not find compatible versions for pod "Crisp":
In Podfile:
react-native-crisp-chat-sdk (from ../node_modules/react-native-crisp-chat-sdk) was resolved to 0.11.0, which depends on
Crisp (= 1.6.1)
Specs satisfying the Crisp (= 1.6.1) dependency were found, but they required a higher minimum deployment target.

Check the line platform :ios, 'XX.X' in your Podfile.
What's it set to?

Check the line platform :ios, 'XX.X' in your Podfile.
What's it set to?

platform :ios, '12.4'

I'm trying to install Crisp on a React Native 0.64.0 project
I have tried the last 3 versions of react-native-crisp-chat-sdk without success
Error when I want pod install :
[!] CocoaPods could not find compatible versions for pod "Crisp": In Podfile: react-native-crisp-chat-sdk (from ../node_modules/react-native-crisp-chat-sdk) was resolved to 0.11.0, which depends on Crisp (= 1.6.1)
Specs satisfying the Crisp (= 1.6.1) dependency were found, but they required a higher minimum deployment target.

Did it solve for you? @nextriot

Did you try setting a higher target?

Did you try setting a higher target?

Worked!


Add button to copy link/password to clipboard

Using clipboard.js
On browsers not supporting evalCommand(), the text will be selected,
and the keyboard shortcut displayed, depending on platform.
(tested on Safari)
Now using Bootstrap readonly text form, instead of a header,
to display the passwords and generated links;
Using fontawesome to get a clipboard icon (like GitHub, GitLab, etc.)
fontawesome and clipboard.js are pulled from a CDN.

Looks like this:

And auto-selection + keyboard shortcut on safari:

Feel free to try it out and suggest improvements.

Given the sensitive nature of passwords in general, I wonder if this would be a case where all JavaScript resources should be best served from the origin instead of using external CDN-hosted sources.

@jparise Good point. I can update that, and I guess pull out jquery as well, which is also currently delivered that way.

Is it possible to do this without the jquery dependency @samueldg ?
I like the feature, but I would rather not add dependencies unless they are absolutely necessary. I wonder how hard it would be to do this without jquery, or with a smaller, more specific lib

@jparise
I think if loading resources via CDN provider, the very least should include SubResource Integrity for each resource being loaded.

SubResource Integrity sounds useful, but browser support isn't quite there yet: http://caniuse.com/#feat=subresource-integrity

@jparise I added the clipboardjs scripts in the static directory, so they are not pulled from a CDN anymore. I also pulled jquery in there, cause I guess the same reasoning applies.
@nichochar Regarding jquery, it's not a dependency I added: it's already there (I guess it's needed for some parts of bootstrap?). The lib I used is indeed smaller and more specific :)
I'm waiting for your feedback!

Looks great. I wasn't aware that jquery was already there Since it is, this is great. Thanks

Thanks for contributing @samueldg !


fix: Wrong repo list on Explore page if user already loggin.

fix #934 and also add unit testing.
User already login.

User not login

Otherwise LGTM

LGTM

let L-G-T-M work


Force scale_run distinction to check which version is running

/run profiling

Confirming that the correct SHA is checked out, but the wrong SHA is being reported in the profiling results. Likely because the SHA passed in the trigger_on_comment workflow is that of main, not the PR branch.


chore: updated downshift to pick the status cleanup feature

Similar to our own cleaning of the add/remove status here https://github.com/stardust-ui/react/pull/1237 we are doing the same thing in Downshift and we want to pick up the change.
Version needed is 3.2.9.

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

:warning:
Package (or peer) dependencies changed. Make sure you have approval before merging!

Changed dependencies in packages/react/package.json

package
before
after

downshift
^3.2.6
^3.2.10

  Generated by :no_entry_sign: dangerJS


Arbitrer-ui using logback

Hello,
i am trying to run the arbiter-ui, while using log4j implementation of slf4j,
and i find that play.api.Logger.shutdown() is using logback, which stops me from using my choice of logger.
i am running the example code in /dl4j-examples/src/main/java/org/deeplearning4j/examples/arbiter/BasicHyperparameterOptimizationExample.java
with my logger of choice set.
i see it has already been discussed in other parts of the code
https://github.com/eclipse/deeplearning4j/issues/6551
https://github.com/eclipse/deeplearning4j/issues/560
and yet i am unclear of what i should do to overcome this hard coded problem.
after excluding logback-classic from arbiter-ui_2.11, i am left with the ui server throwing an exception of java.lang.NoClassDefFoundError: ch/qos/logback/classic/LoggerContext
and the java process fails to exit/finish its run.

after looking at other solutions to the problem i now catch the exception and present it as a log entry, but that does not help with the server hanging forever and the process not exiting the run.

Version Information
<dl4j.version>1.0.0-beta4</dl4j.version>
<slf4j.version>2.0.0-alpha0</slf4j.version>
<log4j-core.version>2.12.0</log4j-core.version>

Deeplearning4j version 1.0.0-beta4
Platform information (OS, etc) win7

Additional Information
Where applicable, please also provide:
Exception in thread "main" java.lang.NoClassDefFoundError:
ch/qos/logback/classic/LoggerContext
	at play.api.Logger$.shutdown(Logger.scala:303)
at play.core.server.Server$class.stop(Server.scala:69)
at play.core.server.NettyServer.stop(NettyServer.scala:175)
at play.server.Server.stop(Server.java:28)
at org.deeplearning4j.ui.play.PlayUIServer.stop(PlayUIServer.java:422)
at app.DigitImageRecognizer.optimizeNetwork(DigitImageRecognizer.java:184)
at app.DigitImageRecognizer.main(DigitImageRecognizer.java:88)
Caused by: java.lang.ClassNotFoundException: ch.qos.logback.classic.LoggerContext
at java.net.URLClassLoader.findClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
at sun.misc.Launcher$AppClassLoader.loadClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
... 7 more

Contributing
i am willing to help, but could use some guidance to what to do.
Thank you for a great framework,
Tal

Related: https://github.com/eclipse/deeplearning4j/issues/3808
This isn't an arbiter restriction, this is a Play framework restriction.
Only option for us here is to upgrade the Play framework version (and just doing it in Maven isn't enough, IIRC there are API changes also). That's not a trivial task, but probably one that we should do anyway for other reasons (Play 2.4 is getting old... the only reason we're sticking with it is the Scala 2.10 support - see the aforementioned issue).

i see,
well right now i am calling System.exit() to end the program, do you think that extending the Play Server and overriding the logging part is a good idea ?
and about the bigger picture, is there a work plan on what needs to be done to upgrade the play/java9/java10, if you make a detailed plan it will be easy to make the change as a community effort.

Closing this as a duplicate of this issue: https://github.com/eclipse/deeplearning4j/issues/3808
A fix for this issue (via Play framework version upgrade that allows configuring logging) will be merged soon: https://github.com/eclipse/deeplearning4j/pull/8098


Shouldn't example use localhost?

Line 22 of the script says
22       url : 'ws://rail-sim2.cs.wpi.edu:9090'
Shouldn't this be using localhost instead, as
22       url : 'ws://localhost:9090'    ?
I am still unable to get the example working:  The code seems to execute but the PR2 does not move.  Any help would be appreciated.

Which branch/version are you using? Looks like the example code is using localhost: https://github.com/WPI-RAIL/keyboardteleopjs/blob/develop/examples/keyboardteleop.html#L25
Are you using this with gazebo? Are the commands coming through?
rostopic echo /base_controller/command

I have been using the version at
http://wiki.ros.org/keyboardteleopjs/Tutorials/CreatingABasicTeleopWidgetWithSpeedControl
Apparently it is downlevel.
I will try again with the example code you indicate on GitHub and "rostopic
echo"
On Tue, Mar 24, 2015 at 12:29 PM, Russell Toris<EMAIL_ADDRESS>wrote:

Which branch/version are you using? Looks like the example code is using
localhost:
https://github.com/WPI-RAIL/keyboardteleopjs/blob/develop/examples/keyboardteleop.html#L25
Are you using this with gazebo? Are the commands coming through?
rostopic echo /base_controller/command
—
Reply to this email directly or view it on GitHub
https://github.com/WPI-RAIL/keyboardteleopjs/issues/15#issuecomment-85659653
.

After cloning the develop branch, I rewrote several script references in
the example as they have a dependency on the folder hierarchy that the
browser sees as the current directory.  (i.e., they contained "../").  I
hope I got that right.  Are these libraries different from the ones on the
CDN that were used by the tutorial I originally used?
Using Firefox, I could not use the debugger; it interfered with the
script's operation.  Finally after closing the debugger I got messages of
the form showing 2 vectors in 3-space each time I pressed the arrow key.
I also got messages like the following, which may have followed the vector
messages.
[WARN] [WallTime:<PHONE_NUMBER>.065994] [63.341000] Could not process inbound
connection: Internal error processing topic
[/base_controller/command]{'message_definition': '# This expresses velocity
in free space broken into its linear and angular parts.\nVector3
linear\nVector3
angular\n\n================================================================================\nMSG:
geometry_msgs/Vector3\n# This represents a vector in free space.
\n\nfloat64 x\nfloat64 y\nfloat64 z\n', 'callerid':
'/rostopic_13423_1427238925281', 'tcp_nodelay': '0', 'md5sum':
'9f195f881246fdfa2798d1d3eebca84a', 'topic': '/base_controller/command',
'type': 'geometry_msgs/Twist'}
The PR2 still has not moved.
On Tue, Mar 24, 2015 at 12:29 PM, Russell Toris<EMAIL_ADDRESS>wrote:

Which branch/version are you using? Looks like the example code is using
localhost:
https://github.com/WPI-RAIL/keyboardteleopjs/blob/develop/examples/keyboardteleop.html#L25
Are you using this with gazebo? Are the commands coming through?
rostopic echo /base_controller/command
—
Reply to this email directly or view it on GitHub
https://github.com/WPI-RAIL/keyboardteleopjs/issues/15#issuecomment-85659653
.

Here are the echoes that I get:
linear:
x: 0.0
y: 0.0
z: -0.87
angular:
x: 0.0
y: 0.0
z: -0.87
linear:
x: 0.0
y: 0.0
z: 0.0
angular:
x: 0.0
y: 0.0
z: 0.0
The PR2 still does not move.
Thanks,
Joe
On Tue, Mar 24, 2015 at 12:29 PM, Russell Toris<EMAIL_ADDRESS>wrote:

Which branch/version are you using? Looks like the example code is using
localhost:
https://github.com/WPI-RAIL/keyboardteleopjs/blob/develop/examples/keyboardteleop.html#L25
Are you using this with gazebo? Are the commands coming through?
rostopic echo /base_controller/command
—
Reply to this email directly or view it on GitHub
https://github.com/WPI-RAIL/keyboardteleopjs/issues/15#issuecomment-85659653
.

Hmmm strange, if the echos are coming through then the messages are indeed making it through rosbridge and into the ROS system itself. Are you able to move the PR2 within ROS itself? I'm not too familiar with the PR2 gazebo environment anymore but you may be able to launch the pr2 simple keyboard teleop node (http://wiki.ros.org/pr2_teleop). I know in some versions of Gazebo too you must first unpause the simulation.
What version of ROS/rosbridge_suite are you using?

I'm using hydro, on Ubuntu 12.04.
At present I am unable to see PR2 moving at all.  At the moment, I am going
to back off and study what I have so far--I need to learn more about ROS,
and Javascript too.  Perhaps I will try a different simulator.
My ultimate objective is to add voice control capability to teleop.
Joe
On Wed, Mar 25, 2015 at 5:19 AM, Russell Toris<EMAIL_ADDRESS>wrote:

Hmmm strange, if the echos are coming through then the messages are indeed
making it through rosbridge and into the ROS system itself. Are you able to
move the PR2 within ROS itself? I'm not too familiar with the PR2 gazebo
environment anymore but you may be able to launch the pr2 simple keyboard
teleop node (http://wiki.ros.org/pr2_teleop). I know in some versions of
Gazebo too you must first unpause the simulation.
What version of ROS/rosbridge_suite are you using?
—
Reply to this email directly or view it on GitHub
https://github.com/WPI-RAIL/keyboardteleopjs/issues/15#issuecomment-86002357
.

Hello Russell,
I'm making progress on my project (teleop with speech) but have run into a
little snag.  Google speech recognition runs awkwardly if the page is not
accessed with https.  This requirement evidently applies to referenced
libraries, too. So I should use https.
My references to

We don't have SSL certs on the actual CDN domain. You can access them securely though directly through their AWS S3 URL, e.g., https://s3.amazonaws.com/cdn.robotwebtools.org/roslibjs/current/roslib.min.js

Thanks, that did the trick.  Of course, it exposed the next problem:  the
websocket connection to the robot.
Mixed Content: The page at 'https://<IP_ADDRESS>/teleop-widget.html' was
loaded over HTTPS, but attempted to connect to the insecure WebSocket
endpoint 'ws://<IP_ADDRESS>:9090/'. This request has been blocked; this
endpoint must be available over WSS.
I tried WSS, but that was refused.
WSS would have to be a feature of rosbridge, right?  Is there a way to do
that?
On Mon, Apr 27, 2015 at 4:30 PM, Russell Toris<EMAIL_ADDRESS>wrote:

We don't have SSL certs on the actual CDN domain. You can access them
securely though directly through their AWS S3 URL, e.g.,
https://s3.amazonaws.com/cdn.robotwebtools.org/roslibjs/current/roslib.min.js
—
Reply to this email directly or view it on GitHub
https://github.com/WPI-RAIL/keyboardteleopjs/issues/15#issuecomment-96850377
.

There is -- the launch file in rosbridge has an SSL flag that defaults to false which will start the server over wss. You can set it to true but you'll need provide an SSL certificate as well (you'll see the parameters in the launch file).

Oops, forgot to link the launch file: https://github.com/RobotWebTools/rosbridge_suite/blob/develop/rosbridge_server/launch/rosbridge_websocket.launch#L4

Thanks.
It's also maybe worth noting that the wss connection to the robot is
rejected if the certificate is self-signed.  The workaround that worked for
me was to first access the robot's address manually (https).  In that case
you get a message and a chance to accept the presumed insecurity.  After
that, the wss connection worked.
On Mon, Apr 27, 2015 at 5:11 PM, Russell Toris<EMAIL_ADDRESS>wrote:

Oops, forgot to link the launch file:
https://github.com/RobotWebTools/rosbridge_suite/blob/develop/rosbridge_server/launch/rosbridge_websocket.launch#L4
—
Reply to this email directly or view it on GitHub
https://github.com/WPI-RAIL/keyboardteleopjs/issues/15#issuecomment-96855186
.


Unusual control flow rarely screws up contify

https://github.com/s-expressionists/Cleavir/blob/38b03b707a2a09c928521daf580a885d813d7cc6/BIR-transformations/interpolate-function.lisp#L239-L260
Here, contify only recognizes two situations. Either there are multiple calls, and the results of the call (if used) are fed into a shared phi; or, there is only one call, and the return point is the successor of the call. If the return point is the beginning of an iblock, it assumes that it is in the former situation. In this situation, in order to communicate the return values of the inlined function, that iblock's phi, assumed to be the shared results, is used.
However, it is possible to contrive things so that the return point is the beginning of an iblock even if there is only one call, and in a way such that the results of the call are actually needed. Here is the smallest example I could come up with, reduced from clasp-developers/clasp#1181:
(lambda (reqs opts)
  (values
   (BLOCK NIL
     (LET* ((LOOP-LIST-HEAD27101 (LIST NIL))
            (LOOP-LIST-TAIL27102 LOOP-LIST-HEAD27101))
       (TAGBODY
        NEXT-LOOP
          (return-from nil (cdr loop-list-head27101))
          (GO NEXT-LOOP))))

   ;; CHECK-TYPE optional parameters
   (TAGBODY
      (GO G27058)
    G27058
      (when opts (GO G27058)))))

For this code, the results of the call to cdr (which is inlined) are used in the final return value. However, the immediate successor of the call, after catch reduction, is a jump to G27058, and the return-point is then the first instruction in that iblock. This "tricks" the contifier into thinking it does not need to create a new iblock/phi. Then when it tries to delete the call, it finds that the call is still used, and the assertion in the delete function ((EVERY CLEAVIR-BIR:UNUSED-P (CLEAVIR-BIR:OUTPUTS CLEAVIR-BIR:INSTRUCTION))) fails.

The above characterization misses the fact that there are also tail recursive calls. A better way to put it would be that this is sort of a consequence of not having strict CPS. We have some calls that are in a similar form - they're immediately followed by a phi using their values, as from (if x (foo y) (foo z)). But we have some calls that are instead in the middle of blocks and have their values used directly. These need different approaches since we have to turn the latter case into the former case, but the way we distinguish the cases is wrong since a call in the second form can still be immediately succeeded by a jump that doesn't use its value.


App crashing when I select three or more photos in offline mode

I've tried to add three photos three separate times and the app crashes each time. I sent crash logs from 2/3.

Doesn't immediately repro for me, would be good to know if there's more context here


Update mypy and typed-ast for M1 support

Closes https://github.com/meltano/meltano/issues/6026

Codecov Report

Merging #6027 (7191408) into main (1f49192) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #6027   +/-   ##
=======================================
  Coverage   90.03%   90.03%
=======================================
  Files         264      264
  Lines       19046    19046
=======================================
  Hits        17148    17148
  Misses       1898     1898

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 1f49192...7191408. Read the comment docs.


Make ES5-compatible

Using haversine-distance as a browserify package and to be loaded in browsers will make old broswers crash. This fixes that.

Thanks @rstacruz :+1:

Published as v1.1.3

@rstacruz at some point this regressed,  but,  shouldn't this be done by tooling nowadays anyway?


lightning-cloud pip check failure - 1.11.3

Observed lightning-cloud: pip check failure on x86.
fastapi 0.109.1 has requirement starlette<0.36.0,>=0.35.0, but you have starlette 0.36.2.'

Have relaxed starlette pin in fastapi and added a patch for it.


improve test output for implementors

It'd be great if the tests for each feature returned a rich response, like a diff of the expected and actual output, instead of just a boolean. You could write the diffs to a separate log file to make it easier to determine the error.

Closed by #25


RHIDP-4329 and RHIDP-4331: Deferencing code blocks

IMPORTANT: Do Not Merge - To be merged by Docs Team Only
Version(s): 1.3
Issue: RHIDP-4329 and RHIDP-4331
Review:

[ ] Docs review: @hmanwani-rh

/cherry-pick release-1.3


Cannot resolve 'file' or 'directory' error when using new react-quill

I am writing a component that uses react-quill (pulling the source from GitHub).  Here's what I'm including in package.json:
"quill": "^1.0.2",
"react": "^15.4.0",
"react-dom": "^15.4.0",
"react-quill": "git+https://github.com/zenoamaro/react-quill.git",
When I build the component with webpack:
./node_modules/.bin/webpack -d --watch --config webpack.example.config.js
I get the following error:
Module not found: Error: Cannot resolve 'file' or 'directory' ./../../css-loader/lib/css-base.js
I don't get the error when I use the released version of react-quill (and the older version quill).
Is there something else that I need to include?

I have same issue with
../node_modules/react-quill/dist/quill.snow.css

I think it's related to https://github.com/zenoamaro/react-quill/issues/74

+1 with
../node_modules/react-quill/dist/quill.snow.css

This should be fixed in master. Can you verify?
Also, CSS files will be bundled properly with the next release.

How long until the next release?

@zackify see #113


https://mistral.ai/ link to LinkedIn

https://mistral.ai/ link to LinkedIn does not direct you to Mistral LinkedIn page.
When clicked it takes you again to https://mistral.ai/ again instead of taking you to https://www.linkedin.com/company/mistralai/

Please note, @Eureka187 is referencing LinkedIn icon on Mistral AI top-level domain:

Fixed thanks!


back button added in profile screen

@plazzy99 please review my pr
i found that we need a back button on the profile screen.


bug: Jellyfin target doesn't work on directories

Description of the bug
After adding a manual trigger for Bazarr, I've noticed that Jellyfin target doesn't seem to be used when using a directory as path
Steps To Reproduce

Set Autopulse log level to debug
Add a manual trigger
curl -G -u 'admin:password' "http://<IP_ADDRESS>:2875/triggers/manual" --data-urlencode "path=/data/media/Movies/Zombieland (2009)/"
curl -G -u 'admin:password' "http://<IP_ADDRESS>:2875/triggers/manual" --data-urlencode "path=/data/media/Movies/Zombieland (2009)/Zombieland.mkv"
Compare logs

Additional Information

path=/data/media/Movies/Zombieland (2009)/

DEBUG autopulse::routes::triggers: added 1 file from my_bazarr trigger
INFO autopulse::service::runner: found 1 new file
DEBUG autopulse::service::targets::command: command output: Output { status: ExitStatus(unix_wait_status(0)), stdout: "", stderr: "" }
INFO autopulse::service::runner: sent 1 file to targets

path=/data/media/Movies/Zombieland (2009)/Zombieland.mkv

DEBUG autopulse::routes::triggers: added 1 file from my_bazarr trigger
INFO autopulse::service::runner: found 1 new file
DEBUG autopulse::service::targets::emby: refreshed item: /data/media/Movies/Zombieland (2009)/zombieland.mkv
DEBUG autopulse::service::targets::command: command output: Output { status: ExitStatus(unix_wait_status(0)), stdout: "", stderr: "" }
INFO autopulse::service::runner: sent 1 file to targets

Note that with a non-movie file (.srt) as path I get the same behaviour as with a directory

Feel free to close this issue if you think that's out of scope (or a Jellyfin's api limitation)
I ended up using {{episode}} in Bazarr command as the path (as it corresponds to the movie/episode file path)
I guess refreshing the video file also refreshes extra files (srt) in Jellyfin

Hey @bonswouar, so what's happening here is that if it doesn't find the file specified in the Jellyfin library, for example, test.mkv, so then it can't send a "Refresh" request (which updates a lot like video encoding format) and instead sends a "Scan" request.
I do notice that I didn't add a debug log for this which I'll add right away, but it is still sending that scan request. If it's not updating the subtitles with a scan request then that would be a Jellyfin limitation (or bug?) and hence why using the episode file as the path worked better.

I'll close this since it's fixed in https://github.com/dan-online/autopulse/commit/20cca4a10aea6ab937e697a50106087ba5de8c38 but let me know if you see any other issues with Bazarr


Make fromArray static again

It has been lost from #2 from dff834c to 17bdcc1 😅
:octocat:

Thanks!


Fix boot error in analytics config-checker code...

Fixing this error:
AssertionError:
CONFIG ISSUE (1) : prometeo_config.json : One or more of the 'gas_limits' configurations [8, 6, 4, 2, 1] is incompatible with the range of the 'nitrogen_dioxide' sensor (min: 0.05, max: 10).
Please note that sensors must have a much larger range than the limits - 2x being a minimum.

Sensors can now report 'range exceeded', so the config-checker code no longer needs to enforce the above as a safety constraint. However they do still recommend it, via this non-fatal warning:
WARNING:GasExposureAnalytics:prometeo_config.json : One or more of the 'gas_limits' configurations [8, 6, 4, 2, 1] is very close to the range of the 'nitrogen_dioxide' sensor (min: 0.05, max: 10).
Sensors shoud have a much larger range than the limits - e.g. 2x at a minimum.

@lidderupk @narengan - here's the fix to that boot-up issue we saw tonight. Would you mind reviewing/merging this fix please? Thanks!

Linking issue #117


Update database-security.md

Edited the Security roles PPAC experience for linked and unlinked environments per recent updates.

Taking to live via my working branch. Needs screenshots and link updates.


Feature/privacy policy

Ensure the doc site has a privacy policy
#369
Overview
Took the gov.uk privacy policy and added it into the docs site in markdown format. The page is called privacy.md
Reason
The site needs a privacy policy so that users know and can accept/reject how we will use information related to a user's browsing of content on the site. We may also need a separate cookie policy as they do on gov.uk sites.
Work carried out

[x] Privacy policy content adjusted to fit standards team needs.
[x] Privacy policy content changed into markdown format.
[ ] Pop-up cookie banner yet to be completed.

Screenshot

This branch is also rather behind develop (that's why the security audit is failing - this vulnerability has already been fixed) so it would be worth rebasing it and then force pushing back up:
git pull origin develop --rebase
git push origin feature/privacy-policy --force

I notice it is showing the privacy policy in the main navigation. To prevent this, we could add some frontmatter to the top of the MDX files we want to exclude from the main navigation:
---
excludeFromNavigation: true
---

# Some MDX content

and then in this file change this line to filter pages based on that frontmatter property: https://github.com/Royal-Navy/standards-toolkit/blob/e4cab1a49b02e378e380ee19136724b0393294a1/packages/docs-site/src/utils/nav.js#L38
You'll also need to update the relevant GraphQL query here:
https://github.com/Royal-Navy/standards-toolkit/blob/e4cab1a49b02e378e380ee19136724b0393294a1/packages/docs-site/src/hooks/useNavigationQuery.js#L12
Feel free to ask us if you have any questions and we can help.


input: Handling multiple surfaces for the text-input-v1 protocol implementation and imporve InputMethodRelay logic

Describe your PR, what does it fix/add?
This PR addresses the issue where the text-input-v1 protocol could not handle cases where a client creates multiple TextInput.
The case one client creates multiple TextInput is common. e.g. chromium-based client applications create a TextInput for each separate surface. They then send activate request with surface param which indirectly specifies the TextInput. Because every TextInput object is bind to specific surface.
Previously, the code iterated through a linked list and returned upon finding a match by comparing the current focus surface's client with TextInput's client. This approach always returned the first matched  TextInput, whose internal surface pointed to a different surface rather than the current focus surface. This was the root cause of issue #2708.
Moreover, the original code heavily relied on iterating through linked list is not so efficient comparing to a hash lookup way. Therefore, in the  InputMethodRelay, a map from surface -> TextInput has been established, centered around the surface. The InputMethodRelay now records the current focus surface, enabling the retrieval of TextInput based on the surface. Consequently, the logic for pendingSurface in the original code has been removed, as the TextInput can now be obtained at any time through the focused surface.
About issue #2708 mentioned

Similar to the behavior of versions below electron21 mentioned at [WIP] text-input-v1 support #1778 .

yeah it's the same problem. It seems that electron below electron 21 will create a transparent surface. So in former implementation, we always caught that transparent surface's TextInput.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Firstly, I rarely write C++, so there might be some naive mistakes.  Please forgive me ~. Though this patch has been running on my main computer for two days without any issues.
Secondly, there's a comment in the InputMethodRelay.cpp code marked "BUG POSSIBLE". This is because the logic for handling  surfaces in text-input-v3 is quite different from v1. It requires the  server to proactively inform the client. Apart from using the client for matching, I can't think of a better method. However, using the client  for matching is the original code's method, so the issue mentioned for text-input-v3 still persists. But as far as the user experience with Kitty is concerned, there doesn't seem to be any major problem.
Is it ready for merging, or does it need work?
yeah
https://github.com/hyprwm/Hyprland/assets/54235339/856e0aad-3299-4c53-8926-ccb83b3fd36a

Sorry, It seems that I've picked up some bad habits from writing too much Go.

generally lgtm, but I'll clang-format it for you cuz it still isn't, lol

good job👍🏻


Image.create sends base64 string with application/octet-stream

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[x] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description

[ ] npm test runs as expected.
[ ] npm run build runs as expected.

!snapshot

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Instarepl incorrectly reports error in macro

With the following macro:
(defmacro condf
[& forms]
`(condp #(%1 %2) ~@forms))
The instarepl says:
java.lang.RuntimeException: Unable to resolve symbol: forms in this context, compiling:(NO_SOURCE_PATH:0:0)

I think this is a general issue with the editor not actually parsing the macro correctly. I'm using tools.reader which currently seems to have issues with the shorthand lambda syntax. As a workaround, I was able to get the following to work, but I'll certainly consider this a bug and see if I can find a fix.
(defmacro condf [& forms]
  `(condp (fn [] (%1 %2)) ~@forms))

This should now be fixed. Please let me know if you still experience it.

Looks good! Thanks!


Fix typos in doc.

Coverage remained the same at 87.566% when pulling 47e61f11b20d7b79c13e249864eed4ba8f94217f on linkmauve:typos into 64af2d1cac7298b511eb671163dcdbb2ad197d17 on Geal:master.

Coverage remained the same at 87.566% when pulling 47e61f11b20d7b79c13e249864eed4ba8f94217f on linkmauve:typos into 64af2d1cac7298b511eb671163dcdbb2ad197d17 on Geal:master.

thank you :)


Inject regex grammar

Would be nice if this grammar would inject the regex grammar in instances like:
function CheckRegexp()
  let l:count = 0
  g/^[^$,\"]/let l:count +=1
endfunction

such that ^[^$,\"] would parse up with the regex grammar.
Currently, it is monolithically a "pattern" node, with no embedded nodes.

Hi, there was discussion of making the parser able to parse vim-specific regexes (as they differ quite drastically from "regular" regexes).
The issue here is that the parsing of the regexes actually depends on multiple factors (such that 'magic').
This this needs a bit of discussion, but I am open to do such things.
In the mean time, injections can be configured in things like nvim-treesiter so you should open an issue there too.

maybe this can help:
https://github.com/statox/VimRegexConverter

Can not find things relevant to "very magic" in nvim-treesiter

I think you don't get what we are doing here: the action of parsing is to turn a flat table of bytes into a tree.
The problem here is that, in order to turn the bytes into a tree, you need first to do a "lexing" pass, grouping the bytes into chunks with meaning.
In the case of tree-sitter, the lexing is said to be "contextual" that is the bytes may be interpreted differently depending on the part we are in the tree.
This is not really important in our case though, but here is the thing: you need to be able to determine what the next chunk of bytes is in order to parse the text correctly.
Now take the following regex written in a file:
\k+

Depending on the value of the 'magic' option, this can mean two things:

Any number of \k
Only one \k then +

Because of this, we cannot parse this correctly without more knowledge about the general state, which is impossible at our step (as we do this "statically").


Add "Formalities" page

This issue is a part of TODO.

This was added by Mouq - I don't know what a Formalities page consists of.

Then we need to cast him here or close the issue.

@Mouq ?

Howdy. This was a TODO from Smls' original site redesign. A better name might be "About" or similar.

I think, we don't need to create another page, since we already have about.html. We'd better add necessary things to About page and this issue can be closed.

We don't have a list of what those things are; closing this ticket as too vague. Mouq, feel free to open a new one with specifics.


[PLAT-5731] Improve Header docs readability

Goal
The Header docs for bugsnag-cocoa have readability issues when it comes to Objective-C properties.
The tool being used to generate them no longer appears to be maintained so it is necessary to find a better alternative.
Design
Jazzy is a documentation generation tool created and actively maintained by Realm. It is used by several popular open-source libraries, including for documenting Realm's own library.
The documentation it generates can shows Objective-C and Swift versions of each symbol, similar to Apple's documentation.
Other tools considered

appledoc is no longer actively maintained
Doxygen is popular but is not focused on Objective-C / Swift and would not be able to show Swift generated interfaces. Its default theme is also not as smart as Jazzy's.

Changeset
Some of the header files were updated to add missing documentation for classes, and fix the formatting of some existing ones to be parsed more correctly by Jazzy.
Note
The "Header docs" link on https://docs.bugsnag.com/platforms/ios/ will need to be updated after deploying new documentation, since Jazzy uses a different file naming convention and Bugsnag_h/index.html will no longer exist.
Testing
Tested by running make docs locally and opening docs/index.html.
Note that a bundle install is required before running make docs.
Generated output attached below for convenience.
Demo

docs.zip

Update
The docs update CI job has been moved to GitHub Actions, because our BuildKite agent does not have write access to the repos.

Update
The docs update CI job has been moved to GitHub Actions, because our BuildKite agent does not have write access to the repos.

Rebased to fix merge conflicts

Rebased to fix merge conflicts


[deliver] App store connect -> App -> App Information - Content Rights doesn't set

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Deliver command run with submission_information but nothing happened to content rights.

Command executed
    deliver(
        submit_for_review: false,
    	skip_screenshots: true,
    	skip_metadata: true,
	skip_binary_upload: true,
	skip_app_version_update: true,
	automatic_release: false,
        force: true,
        submission_information: {
            add_id_info_uses_idfa: false,
            content_rights_contains_third_party_content: false,
            export_compliance_platform: 'ios',
            export_compliance_compliance_required: false,
            export_compliance_encryption_updated: false,
            export_compliance_uses_encryption: false,
            content_rights_has_rights: false
        }
	)

Complete output when running fastlane, including the stack trace and command used

[✔] 🚀
[12:29:34]: fastlane detected a Gemfile in the current directory
[12:29:34]: However, it seems like you didn't use `bundle exec`
[12:29:34]: To launch fastlane faster, please use
[12:29:34]:
[12:29:34]: $ bundle exec fastlane deliver
[12:29:34]:
[12:29:34]: Get started using a Gemfile for fastlane https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
objc[21489]: Class AppleTypeCRetimerRestoreInfoHelper is implemented in both /usr/lib/libauthinstall.dylib (0x1df503458) and /Library/Apple/System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x1065f84f8). One of the two will be used. Which one is undefined.
objc[21489]: Class AppleTypeCRetimerFirmwareAggregateRequestCreator is implemented in both /usr/lib/libauthinstall.dylib (0x1df5034a8) and /Library/Apple/System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x1065f8548). One of the two will be used. Which one is undefined.
objc[21489]: Class AppleTypeCRetimerFirmwareRequestCreator is implemented in both /usr/lib/libauthinstall.dylib (0x1df5034f8) and /Library/Apple/System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x1065f8598). One of the two will be used. Which one is undefined.
objc[21489]: Class ATCRTRestoreInfoFTABFile is implemented in both /usr/lib/libauthinstall.dylib (0x1df503548) and /Library/Apple/System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x1065f85e8). One of the two will be used. Which one is undefined.
objc[21489]: Class AppleTypeCRetimerFirmwareCopier is implemented in both /usr/lib/libauthinstall.dylib (0x1df503598) and /Library/Apple/System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x1065f8638). One of the two will be used. Which one is undefined.
objc[21489]: Class ATCRTRestoreInfoFTABSubfile is implemented in both /usr/lib/libauthinstall.dylib (0x1df5035e8) and /Library/Apple/System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x1065f8688). One of the two will be used. Which one is undefined.
[12:29:35]: ------------------------------
[12:29:35]: --- Step: default_platform ---
[12:29:35]: ------------------------------
[12:29:35]: Driving the lane 'ios deliver' 🚀
[12:29:35]: ---------------------
[12:29:35]: --- Step: deliver ---
[12:29:35]: ---------------------
[12:29:35]: Successfully loaded '../FastlaneFiles/fastlane/Deliverfile' 📄
[12:29:35]: No values defined in './fastlane/Deliverfile'
[12:29:35]: Login to App Store Connect<EMAIL_ADDRESS>[12:29:39]: Login successful

+--------------------------------------+--------------------------------------+
|                           deliver 2.210.1 Summary                           |
+--------------------------------------+--------------------------------------+
| submit_for_review                    | false                                |
| skip_screenshots                     | true                                 |
| skip_metadata                        | true                                 |
| skip_binary_upload                   | true                                 |
| skip_app_version_update              | true                                 |
| automatic_release                    | false                                |
| force                                | true                                 |
| submission_information.add_id_info_  | false                                |
| uses_idfa                            |                                      |
| submission_information.content_righ  | false                                |
| ts_contains_third_party_content      |                                      |
| submission_information.export_compl  | ios                                  |
| iance_platform                       |                                      |
| submission_information.export_compl  | false                                |
| iance_compliance_required            |                                      |
| submission_information.export_compl  | false                                |
| iance_encryption_updated             |                                      |
| submission_information.export_compl  | false                                |
| iance_uses_encryption                |                                      |
| submission_information.content_righ  | false                                |
| ts_has_rights                        |                                      |
| screenshots_path                     | ./fastlane/screenshots               |
| metadata_path                        | ./fastlane/metadata                  |
| username                             |<EMAIL_ADDRESS>      |
| app_identifier                       | com.Company.testAppforContentRigh  |
|                                      | ts                                   |
| platform                             | ios                                  |
| edit_live                            | false                                |
| use_live_version                     | false                                |
| overwrite_screenshots                | false                                |
| sync_screenshots                     | false                                |
| verify_only                          | false                                |
| reject_if_possible                   | false                                |
| phased_release                       | false                                |
| reset_ratings                        | false                                |
| team_id                              | 122375404                            |
| dev_portal_team_id                   | CJ9DT5854U                           |
| run_precheck_before_submit           | true                                 |
| precheck_default_rule_level          | warn                                 |
| ignore_language_directory_validatio  | false                                |
| n                                    |                                      |
| precheck_include_in_app_purchases    | true                                 |
+--------------------------------------+--------------------------------------+

[12:29:40]: Running precheck before submitting to review, if you'd like to disable this check you can set run_precheck_before_submit to false
[12:29:40]: Running precheck 👮♀ 👮

+--------------------------+--------------------------------------+
|                  Summary for precheck 2.210.1                   |
+--------------------------+--------------------------------------+
| default_rule_level       | warn                                 |
| include_in_app_purchases | true                                 |
| app_identifier           | com.Company.testAppforContentRights  |
| username                 |<EMAIL_ADDRESS>              |
| platform                 | ios                                  |
| team_id                  | 122375404                            |
| use_live                 | false                                |
+--------------------------+--------------------------------------+

[12:29:40]: Checking app for precheck rule violations
[12:29:47]: Reading in-app purchases. If you have a lot, this might take a while
[12:29:47]: You can disable IAP checking by setting the `include_in_app_purchases` flag to `false`
[12:29:48]: Done reading in-app purchases
[12:29:48]: ✅  Passed: No negative  sentiment
[12:29:48]: ✅  Passed: No placeholder text
[12:29:48]: ✅  Passed: No mentioning  competitors
[12:29:48]: ✅  Passed: No future functionality promises
[12:29:48]: ✅  Passed: No words indicating test content
[12:29:48]: ✅  Passed: No curse words
[12:29:48]: ✅  Passed: No words indicating your IAP is free
[12:29:48]: 😵  Failed: Incorrect, or missing copyright date-> using a copyright date that is any different from this current year, or missing a date
[12:29:49]: ✅  Passed: No broken urls
+-----------+----------------+
|     Potential problems     |
+-----------+----------------+
| Field     | Failure reason |
+-----------+----------------+
| copyright | missing: 2022  |
+-----------+----------------+
[12:29:49]: precheck 👮♀ 👮  found one or more potential metadata problems, but this won't prevent fastlane from completing 👍

+------+------------------+-------------+
|           fastlane summary            |
+------+------------------+-------------+
| Step | Action           | Time (in s) |
+------+------------------+-------------+
| 1    | default_platform | 0           |
| 2    | deliver          | 13          |
+------+------------------+-------------+

[12:29:49]: fastlane.tools finished successfully 🎉

Environment
🚫 fastlane environment 🚫
Stack

Key
Value

OS
12.4

Ruby
3.1.2

Bundler?
false

Git
git version 2.34.1

Installation Source
/opt/homebrew/Cellar/fastlane/2.210.1/libexec/bin/fastlane

Host
macOS 12.4 (21F79)

Ruby Lib Dir
/opt/homebrew/Cellar/ruby/3.1.2_1/lib

OpenSSL Version
OpenSSL 1.1.1q  5 Jul 2022

Is contained
false

Is homebrew
true

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
13.3

Swift Version
5.6

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./fastlane/Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
#update_fastlane

default_platform(:ios)

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :gybe_gym do
	api_key = app_store_connect_api_key(key_id: "", issuer_id: ", key_filepath: "", duration: 120, in_house: false)
    cert( api_key: api_key)
    sigh( force: false, api_key: api_key)
    build_app(
        scheme: "Unity-iPhone",
		workspace: "Unity-iPhone.xcworkspace",
		build_path: "../archives/project",
        skip_package_ipa: true
    )
  end
end

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :gybe_archive do
    build_app(
        skip_build_archive: true,
        archive_path: "path_to_xarchive"
    )
  end
end

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :gybe_upload do
	api_key = app_store_connect_api_key(key_id: "", issuer_id: "", key_filepath: "", duration: 120, in_house: false)
	ENV['ITMSTRANSPORTER_FORCE_ITMS_PACKAGE_UPLOAD'] = 'true'
	upload_to_testflight(
        wait_processing_timeout_duration: 300,
        api_key: api_key,
        ipa: ""
    )
  end
end

platform :ios do
  desc "Push a new beta build to TestFlight for Rollic"
  lane :rollic_gym do
    build_app(
		workspace: "Unity-iPhone.xcworkspace",
        scheme: "Unity-iPhone",
        export_options: {
            provisioningProfiles: {
"com.Company.Project" => ""
            }
        },
build_path: "../archives/project-ios",
        skip_package_ipa: true
    )
  end
end

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :rollic_archive do
    build_app(
        skip_build_archive: true,
        archive_path: "path_to_xarchive"
    )
  end
end

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :rollic_upload do
  ENV['ITMSTRANSPORTER_FORCE_ITMS_PACKAGE_UPLOAD'] = 'true'
    upload_to_testflight(
        wait_processing_timeout_duration: 300,
        ipa: ""
    )
  end
end

platform :android do
    desc "Push an aab to google play store"
    lane :gybe_android do
        supply(
            aab: "",
            track: "internal"
        )
    end
end

platform :android do
    desc "Push an aab, upload descriptions and screen shots to google play store"
    lane :gybe_android_draft do
        supply(
            track: "internal",
            release_status: "draft"
        )
    end
end

platform :ios do
  desc "Create new ios game"
  lane :gybe_new_game do
    produce(
username<EMAIL_ADDRESS>app_identifier: "com.Company.Project",
app_name: "Project",
        language: "English",
team_id: "",
itc_team_id: ""
    )
  end
end

platform :ios do
  desc "Upload ios game ss and descriptions"
  lane :upload_ios_ss_and_desc do
  ENV['ITMSTRANSPORTER_FORCE_ITMS_PACKAGE_UPLOAD'] = 'true'
  deliver(
        force: true
    )
  end
end

platform :ios do
  desc "upload_app_privacy"
  lane :upload_app_privacy do
    upload_app_privacy_details_to_app_store(
  	    json_path: "fastlane/app_privacy_details.json"
	)
  end
end

platform :ios do
  desc "upload_age_rating"
  lane :upload_age_rating do
    deliver(
        submit_for_review: false,
    	skip_screenshots: true,
	    skip_binary_upload: true,
	    skip_app_version_update: true,
	    automatic_release: false,
	    price_tier: 0,
	    app_rating_config_path: "./fastlane/metadata/app_store_rating_config.json",
	    primary_category: 'GAMES',
        force: true
	)
  end
end

`./fastlane/Appfile`
app_identifier("com.Company.Project") # The bundle identifier of your app
<EMAIL_ADDRESS># Your Apple email address
itc_team_id("") # App Store Connect Team ID
team_id("") # Developer Portal Team ID
package_name("com.Company.Project")
json_key_file("/path")# static constant path for every project

fastlane gems

Gem
Version
Update-Status

fastlane
2.210.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

error_highlight
0.3.0

did_you_mean
1.6.1

atomos
0.1.3

rexml
3.2.5

CFPropertyList
3.0.5

claide
1.1.0

colored2
3.1.2

nanaimo
0.3.0

xcodeproj
1.22.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.8.0

terminal-table
1.8.0

plist
3.6.0

public_suffix
5.0.0

addressable
2.8.1

multipart-post
2.0.0

word_wrap
1.0.0

optparse
0.1.1

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

artifactory
3.0.15

babosa
1.0.4

colored
1.2

highline
2.0.3

commander
4.6.0

excon
0.92.4

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-httpclient
1.0.1

faraday-multipart
1.0.4

faraday-net_http
1.0.1

faraday-net_http_persistent
1.2.0

faraday-patron
1.0.0

faraday-rack
1.0.0

faraday-retry
1.0.3

ruby2_keywords
0.0.5

faraday
1.10.2

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.5

faraday-cookie_jar
0.0.7

faraday_middleware
1.2.0

fastimage
2.2.6

gh_inspector
1.1.3

json
2.6.1

mini_magick
4.11.0

naturally
2.2.1

rubyzip
2.3.2

security
0.1.3

xcpretty-travis-formatter
1.0.1

dotenv
2.8.1

bundler
2.3.11

simctl
1.6.8

jwt
2.5.0

uber
0.1.0

declarative
0.0.20

trailblazer-option
0.1.2

representable
3.2.0

retriable
3.1.2

mini_mime
1.1.2

memoist
0.16.2

multi_json
1.15.0

os
1.1.4

signet
0.17.0

googleauth
1.2.0

httpclient
2.8.3

webrick
1.7.0

google-apis-core
0.9.0

google-apis-playcustomapp_v1
0.10.0

google-apis-androidpublisher_v3
0.27.0

google-cloud-env
1.6.0

google-cloud-errors
1.3.0

google-cloud-core
1.6.0

google-apis-iamcredentials_v1
0.14.0

google-apis-storage_v1
0.17.0

rake
13.0.6

digest-crc
0.6.4

google-cloud-storage
1.42.0

emoji_regex
3.2.3

jmespath
1.6.1

aws-partitions
1.633.0

aws-eventstream
1.2.0

aws-sigv4
1.5.1

aws-sdk-core
3.151.0

aws-sdk-kms
1.58.0

aws-sdk-s3
1.114.0

set
1.0.2

forwardable
1.3.2

logger
1.5.0

pathname
0.2.0

shellwords
0.1.0

cgi
0.3.1

date
3.2.2

timeout
0.2.0

stringio
3.0.1

securerandom
0.1.1

uri
0.11.0

openssl
3.0.0

digest
3.1.0

io-nonblock
0.1.0

ipaddr
1.2.4

io-wait
0.2.1

zlib
2.1.1

resolv
0.2.1

time
0.2.0

open-uri
0.2.0

mutex_m
0.1.1

net-protocol
0.1.2

net-http
0.2.0

ostruct
0.5.2

english
0.7.1

erb
2.2.3

strscan
3.0.1

abbrev
0.1.0

io-console
0.5.11

tempfile
0.1.2

delegate
0.2.0

fileutils
1.6.0

tmpdir
0.1.2

base64
0.1.1

singleton
0.1.1

open3
0.1.1

nkf
0.1.1

prettyprint
0.1.1

pp
0.3.0

find
0.1.1

yaml
0.2.0

psych
4.0.3

generated on: 2022-11-07

I have the same reoccurring issue

I'm having exactly the same issue, were you able to find a workaround (that's not manually going to the store?)

I notice your log shows skip_metadata: true and skip_binary: true. Do we know for sure that content rights information is sent up without those flags.
For what it's worth, mine still doesn't work when passing skip_metadata: false.

I actually noticed in the source code that two criteria are required for this information to be uploaded:

pass submit_for_review: true
pre-check passes successfully

Looks like in your original call, you were failing both criteria.


Avoid needing the Android Context in Sentry.init(..) common code

Problem
Sentry Android init needs the Android context: SentryAndroid.init(context) { } which is not easily replicated in the common code.
Our current solution involves a hacky workaround that declares an expect class in commonCode which typealiases to Android context on Android but does nothing on iOS.
common: expect abstract class Context
android: actual typealias Context
iOS: actual abstract class Context
Avoiding context would also mean the user doesn't need to specifically create another init function just for the context which removes some of the burden on them.
Solution
A user pointed to an article/blog with a possible solution for this in discussions: #112
The article: https://proandroiddev.com/how-to-avoid-asking-for-android-context-in-kotlin-multiplatform-libraries-api-d280a4adebd2
Essentially what we would be going to use is the androidx.startup:startup-runtime package which handles the initialization before the library code - repo
Update:
This can be implemented using a simple internal ContentProvider on KMP Android. The incorporation of the startup library is still something that is being discussed.

Merged in #117


remove delegator reward of referee bonus part

remove delegator reward of referee bonus part  for using new campagn's referee

Codecov Report

Merging #300 (ec6213d) into master (1ed9df8) will decrease coverage by 0.16%.
The diff coverage is 95.00%.

@@            Coverage Diff             @@
##           master     #300      +/-   ##
==========================================
- Coverage   67.30%   67.14%   -0.17%
==========================================
  Files          56       56
  Lines       15841    15738     -103
==========================================
- Hits        10662    10567      -95
+ Misses       5179     5171       -8

Impacted Files
Coverage Δ

primitives/src/credit.rs
47.15% <0.00%> (+1.12%)
:arrow_up:

pallets/staking/src/lib.rs
67.03% <83.33%> (-0.02%)
:arrow_down:

pallets/credit/src/lib.rs
65.09% <100.00%> (-0.82%)
:arrow_down:

pallets/credit/src/tests.rs
100.00% <100.00%> (ø)

pallets/staking/src/mock.rs
94.94% <100.00%> (ø)

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 1ed9df8...ec6213d. Read the comment docs.


Added animated loading bar and quote to turn transition screen

This works correctly but imo the quotes system could use a little refactoring so that the transition screen UI class is not closely coupled to the implementation of how you manage quotes.
Could you move the quote declarations into a separate file, e.g. src/text/quotes.ts or something like that, which exposes only a single getRandomQuote() method that returns a Quote object containing the text and attribution as fields?
In general I think it's preferable to use objects rather than [string, string]tuples because in the tuple the type signature can't tell you which string represents what, and you have to look into the implementation of whoever provides that tuple to find out that quote[1] is the attribution, whereas with an object there's no ambiguity about what quote.attribution refers to.

Sure, can do that

A couple of things:

Please make a comment explaining what that regex in the constructor does and why it's there
I know using private variables and getters is standard practice for Java, but it's kinda verbose for this case, and in Typescript you can just have give the variables the visibility modifier readonly instead of private
It would be cleaner to use Random.fromArray(this.Quotes) instead of  this.QUOTES[Math.floor(Math.random() * this.QUOTES.length)]. You can import it from src/util/Random.ts.


Adsk Contrib - Add native bit depth support to the 1D LUT Op

From the issue #609:

OCIO currently only accepts floating-point image buffers. However, in many scenarios one is starting with an integer buffer. Furthermore, the first operation often performed is a 1d-LUT. So rather than accepting an integer buffer and doing a fast look-up, OCIO currently converts to float and does interpolation rather than a simple look-up.

The complete implementation of the feature requires several steps. But the pull request #699 added the structural changes:

The pull request has all the changes to the OCIO v2 public API to natively support the pixel formats while maintaining full backward compatibility with version 1.  As the final implementation would have created a huge pull request, only minimal set of changes (with lot of TODOs and comments) are in to limit the size and ease the review.

The current pull request adds native bit depth support to the 1D LUT Op to benefit from the look-up table for integer bit depths i.e. one major performance optimization for the CPU processing. Several unit tests now validate the new capability.
Note:
One or two pull requests are still expected to complete the feature.

Following the two-weeks rule, here is a friendly reminder that the branch could be merged next week unless there are some objections.


Tests base env

changes:

removed a hack from base_env
added tests to the abstraction in base_env
added tests to the coloring aspect of base_env

Pull Request Test Coverage Report for Build 1223

57 of 61 (93.44%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 76.954%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tests/fast_tests/test_environment_base_class.py
57
61
93.44%

Totals

Change from base Build 1213:
0.4%

Covered Lines:
5139

Relevant Lines:
6678

💛  - Coveralls


Refactor getLinode to be implemented in mapStateToProps

See comments on #1107
We have multiple places that import a function across layout modules that could be better placed in a mapStateToProps at the top level linode layout. (named select in most places)

Implemented in https://github.com/linode/manager/pull/1601


Look at alternative CI platforms

GitHub Actions was great for getting started but doesnt seem to support ARM64 Apple platforms (among others). We might want to move to something more general (CDash? We're already using CTest so it would be easy integration)

It is very much possible to add other platform support by providing your own runners and docker containers to GitHub Actions https://docs.github.com/en/actions/hosting-your-own-runners/about-self-hosted-runners
There is a lot of useful integrations that you might lose or which may become time-intensive to manage if you go for another solution. You would also most likely still end up setting up your own runners and containers on whatever other platform you choose.

According to the docs, self hosted runners are not supported for any platform but Linux on the ARM64 ISA.
It's not a question of self hosting vs third party, it's a question of whether CI for our Mac port is even possible on GitHub Actions or not.

Just so it's 'on paper'. It is perfectly possible to set up a mac m1 runner within the GitHub CI system, but the Windows end of things has proven far more frustrating as errors end up in the Windows equivalent of /dev/null. This may be due to msys2/mingw, or due to some other reason why errors end up in e.g. a messagebox instead of console. Simultaneous GL initialization? Something else entirely? I was not able to figure it out in the time I allotted to the problem.

Closing as I think we have a good path forward on this now.


ci-operator: document build cache

Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>Docs DPTP-2051
/assign @petr-muller @droslean

Fixed the typo, retagging.


Re-organize docs

We need to organize docs by workflow.

README.md - base readme for the project. Links to docs for individual workflows

builders/go/README.md - Doc for Go builder.
builders/slsa/README.md - Doc for provenance-only workflow
etc.

We also need to document the release process in a RELEASE.md. See https://github.com/slsa-framework/slsa-github-generator/issues/74


Add support for specifying a logo in the AuthMethodPickerActivity

This is related to issue https://github.com/firebase/FirebaseUI-Android/issues/142

LGTM, merging.


How to set setOnSeekBarChangeListener() in RecyclerView adapter

Hi CarGuo
I want to know how to set Seekbar in recyclerview adapter so that at specific time or percentage of video play interval it will open some dialogbox/custom message in video playback.

Hi
I also want same thing like this.. Can  u tell how to get progress of video ,means how much percentage video has watched

finally i found the solution and it's working fine to display custom dialog box by calling
holder.gsyVideoPlayer.mProgressBar.setOnSeekBarChangeListener()


Support for packed storage edition

Statically-sized variables (everything except mapping and dynamically-sized array types) are laid out contiguously in storage starting from position 0. Multiple items that need less than 32 bytes are packed into a single storage slot if possible

Add support for packed storage in lopt, both for variables and structs.
Tests representing this functionality can be found skipped at editable-storage-logic.spec.ts

Closed by #23


fixThrift metastore URI reverse resolution could fail in some environments

Proposed changes
When custom URI resolver is not specified, the default thrift metastore URI goes through DNS reverse resolution (getCanonicalHostname) which is unlikely to resolve correctly when the HMS is sitting behind LBs and proxies. This is a change in behaviour from hive 2.x branch which isn't required. If reverse resolution is required, custom URI resolver can be implemented.
apache hive have repaired this bug: https://github.com/apache/hive/pull/2933
Further comments

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.82 seconds
stream load tsv:          530 seconds loaded<PHONE_NUMBER>9 Bytes, about 134 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>4 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.5 seconds
stream load tsv:          545 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size:<PHONE_NUMBER>1 Bytes

Hello, this PR has been marked as Reviewed and approved, when will it be merged into the main branch? 😂 @dutyu @dataroaring  Can you help find someone to merge it into the main branch? 😅

run buildall

run buildall


Fix  from SVG replacing page

Oops, actually didn't mean to open this PR here as I'm not sure if this is project is actively maintained — feel free to close if so.
If a PJAX response contains an embedded <svg> with <title> tag inside it, this prevents the SVG's title from overwriting the page's title.

You should have opened it in pjax lib itself. This project doesn't maintain this script


Not an issue, but feature request.

The latest update looks great.    Would it be possible to add a third option to the light / dark modes?   Specifically, the [i]previous[/i] dark theme - as in attached picture.
Great utility.

Any particular reason why you would want the third option?

It is easier on my eyes; (side_by_side.png)
Also I think it goes better with some of the other utilities I normally have open at any given time, while flying around the Elite Dangerous galaxy.  (old_theme.png)   (new_dark.png)
Just a feature request, I understand this project is done on your time.

Hmm, I get you. Although there is a little problem of how do we control switching between 3 themes. Right now, its a 2 way toggle.
Right now, the theme selection is saved in your browser so the theme will be light when you access through a new browser. One way this can be handled is by using the system theme as default and only allowing logged in users to set a custom theme. This way the theme selection is stored in your user account. This is how its handled in GitHub for example.
Give me some time to ponder on this and maybe ask for feedback in my Discord server.

As I said this feature will require some core changes in how themes are handled and the above PR is a start of that.


Add GroupNorm option to Conformer block

Codecov Report

Merging #195 (66a059b) into master (2d49f98) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #195      +/-   ##
==========================================
+ Coverage   37.50%   37.55%   +0.05%
==========================================
  Files          87       87
  Lines       10069    10072       +3
  Branches     2119     2120       +1
==========================================
+ Hits         3776     3783       +7
+ Misses       6098     6096       -2
+ Partials      195      193       -2

Impacted Files
Coverage Δ

neural_sp/models/modules/conformer_convolution.py
100.00% <100.00%> (ø)

...eural_sp/models/seq2seq/decoders/rnn_transducer.py
76.20% <0.00%> (+0.64%)
:arrow_up:

neural_sp/models/modules/cif.py
91.13% <0.00%> (+2.53%)
:arrow_up:

Continue to review full report at Codecov.

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


Running the .jar got me A Java exception has occured error

I am very interested in using your LastFM-Potplayer Scrobbler. Unfortunately, I am unable to run the .jar file as instructed from your Readme.md. I have installed the latest jre-8u421-windows-x64 and run the .jar but was introduced to this error:

Can you advise how do I run this so that I can have PotPlayer scrobble what videos I am watching. I am using the latest PotPlayer. Does your app require that I log into my LastFM account btw? Do I need to install pre-requisite apps or other tools in order to run your program?

Please use a newer Java version. It is working for me on Java 21.0.4 on Windows x64.

Is this the latest one? So JDK is required? https://www.oracle.com/java/technologies/javase/21-0-4-relnotes.html

I personally use binaries from https://adoptium.net
Here is the JRE 21 for Windows x64 .msi from there: https://github.com/adoptium/temurin21-binaries/releases/download/jdk-21.0.4%2B7/OpenJDK21U-jre_x64_windows_hotspot_21.0.4_7.msi

Is there a way to save the settings so that I don't have to login everytime I quit Windows 11?

I believe you can pass the setting as a command line argument, but I do not remember for sure. You can thus make a shortcut or batch script to call java -jar (...).jar (setting)


Key Words Auto Cart Error

Honestly, I commend you for fixing my issue so quickly yesterday, but now I am at a new standstill where the keywords actually find the product now, but they don't get added to cart only goes to the items page. Hope this helps.

Okay please try to clear all your supreme's cookies.
If it doesn't work anymore, let me know I'll check.
This works nice for me because I'm on UK Web Store, US is probably different...

I just checked and indeed there is an error, I fixed it.


Integrate with JMRI launcher

The JMRI launcher for Linux needs to accept /dev/tty.* as a valid device node.

In r29688 in JMRI SVN.


Replace DateTime usage with DateTimeOffset

Really pleased to see OpenTracing start up for .NET! Based on what they're used for, I wanted to suggest that the various interfaces be updated to use DateTimeOffset for logging etc. This would be really helpful for implementers (for example the company I work for is implementing a tracer for use internally).
This would match MSDN docs on the subject. There's also a great StackOverflow topic providing an analogy for why DateTimeOffset should be used for this sort of tracing.
Top of the list is that DateTimeOffset serialises unambiguously which I believe will be a big help in implementing propagators correctly.
Example of Serilog logging framework using this for timestamps: https://github.com/serilog/serilog/blob/dev/src/Serilog/Events/LogEvent.cs#L52

See https://github.com/opentracing/opentracing-csharp/issues/4 for the reasons why we chose DateTime.
Please post your scenario if you disagree.

The intent of the interface ISpan Log(DateTime timestamp, string eventName); appears to be to log an occurrence at a specific point in time. Providing a DateTimeOffset is always correct & unambiguous, and in codebases that are following Microsoft guidance, all of the existing code would already be using DateTimeOffset for tracking timestamps (as I indicated, major .NET logging frameworks already do).
The choice of DateTime in the interface means we must convert from DateTimeOffset everywhere to DateTime which returns, as per docs, The value of the DateTime.Kind property of the returned DateTime object is DateTimeKind.Unspecified. which gives exactly the problem you outlined in #4 - every time we convert to OpenTracing format, we have to be aware of all of the pitfalls that DateTimeOffset is designed to avoid. This makes the library harder to work with, not easier.
I can provide more arguments, but I'd stand by the original linked article - Microsoft have made it very clear that modern .NET code should prefer DateTimeOffset wherever possible.

DateTimeOffset is only useful when you want to store the offset (e.g. for local user dates). This (IMO) does not make much sense for server scenarios where you always need UTC.
Also, all other opentracing languages do not support offsets either. This probably also means that most tracers do not support time zones.
DateTimeOffset also has some quite critical drawbacks e.g. it assumes DateTimeKind.Local if a DateTime with DateTimeKind.Unspecified is passed)
The MSDN guidelines say that DateTime should be used if only UTC is needed.
Could you please give me an actual use case for passing a non-UTC timestamp and also how this should be handled by a tracing system?
Sorry for arguing, I just want to make sure, we do the right thing here.
PS: Microsoft recently added an Opentracing-like component (Activity in System.Diagnostics.DiagnosticSource) where they also use DateTime.

No problem with debate! I too want to be sure things are right :) Interfaces are important.
I am in no way proposing that tracers pass down offsets; apologies if I conveyed that. Far from it; I would expect the bulk of tracers to always convert to UTC and/or even epoch time. The most important criteria from my perspective is that what I pass in as a client is 100% correctly reflected as a time entry by the tracer. If that is corruptable in any way, that weakens faith in the entire tracing stack.
I believe you're focusing on the other end of the problem (and nothing wrong with that!). My concern is callers doing the right thing and not accidentally using the interface incorrectly. The current interface requires the caller to have all the knowledge that we are discussing, and that does not appear to be as generally available as one might like.
I would argue DateTimeOffset is one of few correct representations for a timestamp because if the timestamp happens to contain an offset, code still has all available knowledge in fetching the UTC representation of that timestamp. DateTimeOffset.Now still gives you a correct timestamp that is the same moment in time as DateTimeOffset.UtcNow; just it so happens it formats to a different string if you ask for e.g. ISO8601. DateTime equivalents do not do this.
It is correct that if only UTC is used, DateTime is acceptable, but you cannot enforce that contract in an .NET interface. One cannot use a DateTime as a timestamp except as explicitly UTC (because all the other Kind values are meaningless for timestamping - you can never be 100% certain that Local is actually referring to the current machine's local time zone), and that is a more fragile contract for storage & handling. If I pass in a DateTime of kind Unspecified, the tracer has absolutely no idea what the correct representation is, nor how to correctly convert to a UTC propagation format.
You are in essence requiring that every implementation throw an exception if the DateTime provided is not Kind.Utc (or else you are into undefined behaviour or the tracer having to guess, both of which are undesirable). Thus my argument is based in the type being too generic/fuzzy and it's feasible to pass in well-intended but incorrect data. It's harder to 'fall into the pit of success'.
I'm arguing for an interface where the users are least likely to shoot themselves in the foot. I am less invested in the specific of DateTimeOffset except that it happens to be a convenient .NET representation of absolute moment in time that is not arguable. I would be just as comfortable with a long describing epoch microseconds in UTC, because that too is inarguably an absolute moment in time. It's easy to write a wrapper/extension method that converts DateTimeOffset to such a long. It's impossible to write one generically & fully correctly for all kinds of DateTime.
I recognise the concerns around taking a DateTime of varying Kind, wanting to pass it into the span, and having the DateTimeOffset conversion do something just as irritating. I would still believe that either a long that is well defined or DateTimeOffset guides the caller more towards the right thing. Someone debugging their span calls would at least be able to see that the DateTime->DateTimeOffset conversion did something they were not intending, correct it in their calling code, and tracing then works correctly.
A compromise would be that the interface itself uses long, and either the library itself or contributors can add extension methods that do the conversion from as many different types as one cares to add, as long as it's always clear how to get UTC.
Hope that explains more thoroughly, with apologies for the lengthy reply :)

I fully agree that DateTimeOffset is a "better/more complete" representation of a point in time (internally, it's a DateTime and an int16-offset) and that there are many good use cases for it. However, I'm still missing the actual scenario for a logging/tracing tool.
The logging/tracing tool has to show all events from all involved systems in a ordered way. This can only be achieved if all timestamps use the same time zone. This means that either none (=UTC) or all timestamps have to include time zone info. As other languages don't support time zone info, I doubt that tracers can/want to deal with it.
In other words, there's not really a functional advantage of having the separate offset that using DateTimeOffset provides.
So the only reason to use DateTimeOffset would be that its usage is "easier/more obvious" for developers.
However, I don't think that this is the case. No matter which type is used, developers always have to know their behavior:

With DateTime they would have to know that they need to specify Kind.Utc
With DateTimeOffset they have to know that they may not pass a DateTime with Kind.Unspecified either to the DateTimeOffset-constructor or to the methods directly - as that will implicitly assume Kind.Local and therefore "change" the date (unless people really expected this behavior which I doubt).
Also, if we would use ticks with a long, people would still have to know how to get this value.

So I don't think there's a perfect solution for this and the main question is, what do most people want/expect and is moving to DateTimeOffset worth the (certainly small) performance impact.
CoreFx's new Activity type requires UTC by giving the parameters an "Utc" suffix and by raising an error if an invalid kind is passed.
Unfortunately, we can't enforce the Kind by throwing an exception in an interface. We could however add the "Utc" suffix to the parameters - however this would just be a conventional hint.

Why would you pass a DateTime to a DateTimeOffset? Why wouldn't you start with DateTimeOffset and just have correct time throughout your codebase to start with? That's the opposing viewpoint I suppose I'm coming from.
As I said previously, I am not stating that internal timestamps of OpenTracing need to support offsets. I have not stated this. The internals of how a tracer handles a timestamp is completely irrelevant to the conversation. What matters is a useful usable interface that allows specification of the timestamp in a correctly handled fashion. If you offer ISpan Log(DateTime timestamp, string eventName) you are expecting a caller to provide a time.
The concrete example is that I can pass the result of DateTimeOffset.Now to a modern logging system like Serilog and the timestamp will be 100% correct, and whatever conversions it goes through internally gets me a UTC timestamp in whatever serialisation is desired that reflects that. I don't need to care as a caller; it does the right thing; it has enough information to do the right thing. That is the significant functional advantage over a logging system where we have to tell our developers 'do not use this directly, use this wrapper that actually works correctly'.
For me, sticking Utc on the end and then requiring all implementers to remember to implement a check is setting up for failure at some point. A different matter if you were implementing a concrete type, of course, but this code is not. Why use conventional hints when you have types that do it for you? Why not provide extension methods that do the right thing?
Ultimately you can organise the library as you wish, and if necessary we can fork or wrap it, and indeed other implementations can surface more effective methods. It's fairly easy for us internally to just state 'don't use OpenTracing interfaces directly' and add extensions that perform correctly. The intent of this issue was to reduce the pain & confusion of downstream implementors since we are hitting exactly that.

My mind isn't made up. :smile: It's a very important interface so this should just be well-discussed.

If you offer ISpan Log(DateTime timestamp, string eventName) you are expecting a caller to provide a time.

There's overloads without the timestamp parameter for each method and people are expected to call those most of the time. The overloads with a timestamp mostly exist to support scenarios where a span is created from historical data (e.g. from a log file). So there's not really a reason to call one of these overloads with DateTime.UtcNow, DateTimeOffset.Now etc.

A different matter if you were implementing a concrete type, of course, but this code is not. Why use conventional hints when you have types that do it for you? Why not provide extension methods that do the right thing?

I agree to this argument!
I also think that tracers could still work with DateTime internally to get the performance benefit.
Given this and the fact that DateTimeOffset has more advantages than disadvantages, I'm ok with changing this!
Does anyone else have an opinion about this? (e.g. @dawallin )

fixed with #43

I got moved on projects and managed to completely forget about this issue in a haze of urgent internal issues, so I just wanted to pop back and say thank you for this! Really appreciate the dialogue and the willingness to listen.


Multi-day events in Calendar view

What would you like to be added?
Problem:
I have some plans that begin on 20/04/2023 and end on 29/06/2023. I want a Calendar view that can look like Notion, like this:

But I can not do that with Project plugin, it just show either start_date or end_date:

It doesn't show project duration.
Why is this needed?
I and many other people will be able to manage projects better if I look at the overview in Calendar view because it can help me visualize what projects will start from, and end on. what day, and how long is the execution time visually?

just to leave a note of interest, this would be much appreciated

I made a crude multi-day support for the calendar. It's pretty basic, but it does the job.

https://github.com/marcusolsson/obsidian-projects/pull/764

I made a crude multi-day support for the calendar. It's pretty basic, but it does the job.
#764

Wow, it looks good. I hope the next release will include it, lol 😂


[Breaking Change] (Cross-Version) PR check report does not identify specific change

The Breaking Change (Cross-Version) PR check in the azure-rest-api-specs repo does not point to the specific element with a breaking change for some cases. An example is the first "TypeFormatChanged" issue reported in PR 21807:

Both the old and new links go to the "parameters" of an operation that has 5 parameters.  Which one is the message about?

Related investigation:

#6036


Wrap error messages with links/pointers to help

Once Go SDK is merged I will send out a PR that wraps all errors returned by it with a link pointing somewhere for help.
Original idea was to just wrap engine.Start errors with this, but we are planning on refactoring that to not be a callback soon so not worth it.

@shykes @samalba are we ok with pointing the users to our discord channel and telling them to use the help channel for support?

Sent out a PR: https://github.com/dagger/dagger/pull/3342
Just need to decide what message to include there

@shykes @samalba are we ok with pointing the users to our discord channel and telling them to use the help channel for support?

Given that the binary will be harder to change than a web page, I would make the link short and easy to remember (eg. https://dagger.io/help with a short message). On this page I would simply give both options (github issues and discord), and we can change it later without having to re-release.

@shykes @samalba are we ok with pointing the users to our discord channel and telling them to use the help channel for support?

Given that the binary will be harder to change than a web page, I would make the link short and easy to remember (eg. https://dagger.io/help with a short message). On this page I would simply give both options (github issues and discord), and we can change it later without having to re-release.

I agree. The URL should also include an element that is specific to the cloak Go SDK, as opposed to eg. Dagger Europa, NodeJS SDK etc.
For example https://dagger.io/help/go . That's probably specific enough?

For example https://dagger.io/help/go . That's probably specific enough?

Semantic bike shedding if I may, I'd use an anchor instead (/help#go) - because the content is likely to be the same for all SDKs initially and vary over time.

works for me 👍

Noted - created an issue to create URL #3350

@aluzzardi can this move to you instead since Erik is OOO? He has a PR open, but it needs some updates.

Done here: https://github.com/dagger/dagger/pull/3342