Failing Test: default-mongodb-org-repo-ubuntu-1004

% kitchen test default-mongodb-org-repo-ubuntu-1004                                                                                20 ↵
-----> Starting Kitchen (v1.4.0)
-----> Cleaning up any prior instances of <default-mongodb-org-repo-ubuntu-1004>
-----> Destroying <default-mongodb-org-repo-ubuntu-1004>...
       Finished destroying <default-mongodb-org-repo-ubuntu-1004> (0m0.00s).
-----> Testing <default-mongodb-org-repo-ubuntu-1004>
-----> Creating <default-mongodb-org-repo-ubuntu-1004>...
       Bringing machine 'default' up with 'virtualbox' provider...
       ==> default: Importing base box 'opscode-ubuntu-10.04'...
==> default: Matching MAC address for NAT networking...
       ==> default: Setting the name of the VM: kitchen-mongodb-cookbook-default-mongodb-org-repo-ubuntu-1004_default_1436913996945_4160
       ==> default: Fixed port collision for 22 => 2222. Now on port 2203.
       ==> default: Clearing any previously set network interfaces...
       ==> default: Preparing network interfaces based on configuration...
           default: Adapter 1: nat
       ==> default: Forwarding ports...
           default: 22 => 2203 (adapter 1)
       ==> default: Running 'pre-boot' VM customizations...
       ==> default: Booting VM...
       ==> default: Waiting for machine to boot. This may take a few minutes...
           default: SSH address: <IP_ADDRESS>:2203
           default: SSH username: vagrant    default: SSH auth method: private key
           default: Warning: Connection timeout. Retrying...
           default:
           default: Vagrant insecure key detected. Vagrant will automatically replace
           default: this with a newly generated keypair for better security.
           default:
           default: Inserting generated public key within guest...
           default: Removing insecure key from the guest if it's present...
           default: Key inserted! Disconnecting and reconnecting using new SSH key...
       ==> default: Machine booted and ready!
       ==> default: Checking for guest additions in VM...
       ==> default: Setting hostname...
       The following SSH command responded with a non-zero exit status.
       Vagrant assumes that this means the command failed!

       hostnamectl set-hostname 'default-mongodb-org-repo-ubuntu-1004'

       Stdout from the command:

       Stderr from the command:

       stdin: is not a tty
       bash: line 2: hostnamectl: command not found
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: Failed to complete #create action: [Expected process to exit with [0], but received '1'
---- Begin output of vagrant up --no-provision --provider virtualbox ----
STDOUT: Bringing machine 'default' up with 'virtualbox' provider...
==> default: Importing base box 'opscode-ubuntu-10.04'...
==> default: Matching MAC address for NAT networking...
==> default: Setting the name of the VM: kitchen-mongodb-cookbook-default-mongodb-org-repo-ubuntu-1004_default_1436913996945_4160
==> default: Fixed port collision for 22 => 2222. Now on port 2203.
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
    default: Adapter 1: nat
==> default: Forwarding ports...
    default: 22 => 2203 (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
    default: SSH address: <IP_ADDRESS>:2203
    default: SSH username: vagrant
    default: SSH auth method: private key
    default: Warning: Connection timeout. Retrying...
    default:
    default: Vagrant insecure key detected. Vagrant will automatically replace
    default: this with a newly generated keypair for better security.
    default:
    default: Inserting generated public key within guest...
    default: Removing insecure key from the guest if it's present...
    default: Key inserted! Disconnecting and reconnecting using new SSH key...
==> default: Machine booted and ready!
==> default: Checking for guest additions in VM...
==> default: Setting hostname...
STDERR: The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

hostnamectl set-hostname 'default-mongodb-org-repo-ubuntu-1004'

Stdout from the command:

Stderr from the command:

stdin: is not a tty
bash: line 2: hostnamectl: command not found
---- End output of vagrant up --no-provision --provider virtualbox ----
Ran vagrant up --no-provision --provider virtualbox returned 1]
>>>>>> ----------------------
>>>>>> Please see .kitchen/logs/kitchen.log for more details
>>>>>> Also try running `kitchen diagnose --all` for configuration

Passing when upgraded to vagrant 1.7.4


OP-1903: Dockerfile addition

Created Dockerfile with multi-stage build for casper-node-launcher
Added ci scripts for building and pushing to dockerhub and steps to .drone.yml.
Expecting user to map in volumes for /etc/casper and /var/lib/casper.

bors merge


Closes #2487: Enable brief API output utilizing nested serializers

Fixes: #2487
Minimize API output when ?brief is passed as a query parameter.

While not required by the RFC, I have never been a fan of valueless query params as they can sometimes be annoying to dealwith on the client side. That said, this use case should be fine. Otherwise this looks good to me!

I might sugest a test for this though.

@lampwins Fair point. Should we look for params['brief'] == True? And yeah, I'll add a test if you're good with the overall approach.

@jeremystretch Yeah, I think if request.query_params.get('brief', False) should do the trick. I can't think of any other better way to get to the Nested serializer, especially since we already have get_serializer_for_model(), so this looks good.


Using MDC for labels

Hey there,
i want to migrate from ELK Stack to Loki.
Is it possible to send the MDC Parameters (set dynamically in ThreadContext.put(key, value)) as Labels to Loki? And how would i achieve this with log4j2?
Thanks in advance

Hello,
yes, you can do that. In log4j you have variable ctx that can be used in Label definition, you can use mdc or X:
  <appenders>
        <Loki name="Loki">
...
           <Label name="tid1" pattern="%X{tid}"/>
            <Label name="tid2" value="${ctx:tid}"/>
            <Label name="tid3" pattern="%mdc{tid}"/>

            <Label name="tenant" value="prefix_${tenant}"/>
            <Label name="tid" value="${tid}"/>
            <Label name="default_value_test" value="${ctx:missing:-use_this_if_missing}"/>
        </Loki>
    </appenders>

The ${ctx} syntax also allows for specifying default values if context is empty:
            <Label name="default_value_test" value="${ctx:missing:-use_this_if_missing}"/>

You might find out this documentation helpful:

https://logging.apache.org/log4j/2.x/manual/layouts.html
https://github.com/tkowalcz/tjahzi/blob/9e07ef34ca5bb6cbb1a13f69151db35aade22bf3/log4j2-appender/src/test/resources/log4j-patterns-in-labels-configuration.xml

@stefnats Please let me know or reopen the issue if you need more info.

Is there any way to do the same but without knowing previously the name of the MDC vars? I have dynamic context variables depending on the code and i don't want to re-write my xml every time i add a new one.
By instance, i have 2 methods with 3 different value, can i do something at XML to show 3 without telling the name up front?
// Method 1..
ThreadContext.put("foo1", "value1");
ThreadContext.put("foo2", "value2");
// Method 2
ThreadContext.put("foo1", "value1");
ThreadContext.put("foo3", "OtherValue3");

Is there any way to do the same but without knowing previously the name of the MDC vars? I have dynamic context variables depending on the code and i don't want to re-write my xml every time i add a new one.
By instance, i have 2 methods with 3 different value, can i do something at XML to show 3 without telling the name up front?
// Method 1.. ThreadContext.put("foo1", "value1"); ThreadContext.put("foo2", "value2");
// Method 2 ThreadContext.put("foo1", "value1"); ThreadContext.put("foo3", "OtherValue3");

@lucianocallero I don't think it is possible. Is it possible in other appenders?
I guess we could add a switch that will convert the whole thread context into loki labels.

Thanks. That would be great. In me case more than one appender does not fit because labels are dynamic depending on the context of the appliances.


Add animation showing steps taken when feature is submitted

Current behavior

users confirms they want to submit feature
AssembleApp running on jupyter server goes through steps to submit feature: (1) extract code from payload, (2) fork upstream repo if applicable (3) clone to tmp (4) render feature module (5) copy code to feature module (6) commit files (7) push to origin (8) pull request to upstream
after failure or success, a new modal shows result to user

Desired behavior

user confirms they want to submit feature
UI immediately shows some sort of "processing" animation, ideally showing the actual steps the server is taking
after failure or success, the processing animation is replaced by the result modal

I would like to approach implementing this behavior in two separate stages:

show a simple spinner while the feature is being submitted, just to communicate to the user that something is happening

this can be achieved my showing a modal immediately after the user confirms the feature in the dialog, instead of waiting for the api call

show detailed steps instead of a spinner:

when a feature is submitted to the extension backend, the state of the feature submission is kept track of in a global variable (or more specifically in a dictionary, to keep track of multiple submissions concurrently)
the feature submission endpoint returns a random uuid that can be used by the frontend to make repeated (~500ms) calls to the backend in order to fetch the state of the submission as it updates and update the animation accordingly.


Replace filtered/query by bool/must

The filtered query was removed in Elasticsearch 5.0 so the filter method in Document was raising an exception when used with the ids parameter: elasticsearch.exceptions.RequestError: TransportError(400, 'parsing_exception', 'no [query] registered for [filtered]').
This PR also replaces a few other references to filtered and adds filter as a valid keyword for bool in Query.

Here's a simple example which triggers this issue:
import toute
from elasticsearch import Elasticsearch

class Person(toute.Document):
    _doctype = "person"
    _index = "testing"
    _es = Elasticsearch(host='localhost', port=9200)
    name = toute.KeywordField()

Person.init()

person = Person(id=1, name="John")
person.save()
person = Person(id=2, name="Mary")
person.save()
person = Person(id=3, name="Smith")
person.save()

for person in Person.filter(ids=[1,2]):
    print(person.name)

Closing to recreate from another branch.


KAFKA-7483: Allow streams to pass headers through Serializer.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@vvcephei @mjsax
Could you please review this PR ?

Call for second review (anyof) @guozhangwang @bbejeck @vvcephei
Retest this please

LGTM, thanks @kamalcph ! I will merge it once jenkins passed.

Cherry-picked to 2.1


FEATURE: Replace interactions with quick replies

https://developers.facebook.com/docs/messenger-platform/send-messages/quick-replies
This will be a much more pleasant UI for the SEND and RECENT actions, as well as any mode toggling like that described in #21

Resolved as of #30


Error during upgrade: cannot find top supervisor for application ?

Hope this is the right place to pose a question :)
I successfully managed to create a release and deployed it. Now I changed some little things and I would like to do an upgrade. I didn't create an appup file.
In the application I have exometer_core as a dependency which in turn has setup as a dependency and setup is also listed as an application in exometer_core.
Now when I run the upgrade command line I get the following error:
./bin/server upgrade "0.2.1"
using /opt/server/releases/0.2.0/server.conf to populate "/opt/server/running-config".
Release 0.2.1 is already unpacked, now installing.
escript: exception error: no case clause matching
                 {error,{'EXIT',{timeout,{sys,get_status,[<7052.2983.0>]}}}}

In console.log I see the following:
2015-08-04 23:52:28.214 [error] <0.1502.0> release_handler: cannot find top supervisor for application setup

Do you know what I can do about that without forking either exometer_core or setup? Should I add a appup file which explicitely mentions exometer_core and tell it should just halt & restart it (or something similar)?
Thanks in advance :) Great job you do here!

@mweibel An appup should've been created for your upgrade release automatically if you didn't generate one yourself, could you look in rel/<app>/lib/<app>-<version>/ to see if there is an appup file there and gist it if it exists? Could you also verify that in rel/<app>/releases/<version> that a relup file exists?
I'm not sure what would produce that error, so I think we'll be doing some digging on this one ;)

obviously now I tried to do various stuff to make it work.. and the first release upgraded now successfully but I can't reproduce the case anymore.. will reopen in case I can :)
thanks anyway ;)

@mweibel Damn, always hate those kind of bugs. Well if you do stumble on it again, try and open up this issue again if you can find it, that way we can keep related info in one place :)

Could reproduce it now, after fiddling a bit with versions... not sure if related ;)
Anyway, had to sanitize the files a bit, hope it's still readable..
https://gist.github.com/mweibel/06681cc38f737394ed73
server is the application I work on. Should I post any dependency appup or so?

Added a 3rd case clause to install_upgrade.escript in the call to rpc:call(TargetNode, release_handler, install_release, [Vsn], ?TIMEOUT) and it logged this output:
ERROR: release_handler:install_release failed: {'EXIT',
                                                {timeout,
                                                 {sys,get_status,
                                                  [<7053.1172.0>]}}}
[{erl_eval,match1,4,[{file,"erl_eval.erl"},{line,1087}]},
 {erl_eval,match_tuple,5,[{file,"erl_eval.erl"},{line,1180}]},
 {erl_eval,match_tuple,5,[{file,"erl_eval.erl"},{line,1180}]},
 {erl_eval,match,4,[{file,"erl_eval.erl"},{line,1073}]},
 {erl_eval,match_list,4,[{file,"erl_eval.erl"},{line,1206}]},
 {erl_eval,match_clause,5,[{file,"erl_eval.erl"},{line,982}]},
 {erl_eval,case_clauses,6,[{file,"erl_eval.erl"},{line,941}]},
 {erl_eval,local_func,5,[{file,"erl_eval.erl"},{line,556}]}]

well.. not much of a help though ;)

FYI, this time it happened for another app/lib: https://hex.pm/packages/facebook - my own hex package... Which has indeed no supervisor.. wondering why I'd need one though

@mweibel It seems the question is not related to exrm, but to Erlang's SASL release handler functionality (or OTP Design principles for applications, I'm not exactly sure :))
I've recently had the same issue and figured out that the app needs to have supervisor process Erlang to handle upgrade correctly.
Here is my case. I had:
defmodule MyApp do
  use Application

  def start(_, _) do
    GenServer.start_link(MyApp.Worker, [])
  end

end

defmodule MyApp.Worker do

  def init(state) do
    # some code
  end

end

This worked fine but not for upgrading. What/How I've modified:
defmodule MyApp do
  use Application

  def start(_, _) do
    MyApp.Supervisor.start_link([])
  end

end

defmodule MyApp.Supervisor do

  use Supervisor

  def start_link(state) do
    Supervisor.start_link(__MODULE__, state, name: __MODULE__)
  end

  def init(state) do
    children = [
      worker(MyApp.Worker, [state], restart: :permanent)
    ]

    supervise(children, strategy: :one_for_one)
  end

end

defmodule MyApp.Worker do

  def start_link(state) do
    GenServer.start_link(__MODULE__, state, [name: MyApp.Worker])
  end

  def init(state) do
    # some code
    {:ok, state}
  end

end

So that's it. Hope this will help :)

@alPacino will try this thanks! However, if the dependency (or the dependency of the dependency) is not in my control, it might be not that easy :/

As @alPacino found, you do indeed want to have your application start a root supervisor (otherwise you won't have the ability to handle failures properly). I'm still unsure why this occurs, but I'll try and do some reading to find out.

@mweibel agree, but sometimes we should contribute into such dependencies (many of them are hosted on GitHub) like I'm doing right now :). My app depends from lib which doesn't work with exrm properly (including the problem described above). So I'm solving own problem and help others (I hope :)). Sorry for off-topic

@alPacino sure! I do that often actually :) problem is that it's not always treated as necessary from the developer or it's, for the specific case, not even necessary at all. Will try though .. ;)

closing again ;) hopefully this fixed it, but can't yet test really because of some hex issues

Hi everyone! I'm sorry for refreshing the old issue, but it happens I've got the same problem and I don't know how to find out which of the application does not have supervisor. Do I need to check them one by one? crash.log file says only:
2016-01-20 15:34:58.714 [error] <0.1524.0> release_handler: cannot find top supervisor for application setup

Oh my, now I've read it again and I understood that setup is the name of the application! I thought that it's just application setup phrase

Hi everyone (sorry for the previous comments, I removed them to just leave the necessary information)
I'm having the same issue now, when I try to perform upgrade I get the message about the setup application. @mweibel do you remember how you solved it? I tried to use the last release of setup but it still causes the error. Is there some for that has it fixed? Thanks in advance!

@arnvald need to check again, didn't work on this (side-)project in a while.. but I think this did the trick:
https://github.com/mweibel/facebook.ex/commit/fd2aab72b50654c8f41cc13dfe58cf0b606e732b
which is a dependency of the project I'm working on.

@mweibel thanks for response! So in the end you didn't have to change anything in the setup application?

@arnvald It looks to me like the setup application does not start a root supervisor, rather it directly spawns a process via proc_lib. The release handler expects there to be a supervisor tree under an application (by which I mean, something which implements the application behaviour), so that appears to be why it fails with setup, and likely any other application with a similar setup (like @mweibel's facebook application).
The quick solution would be to have the application behaviour start a root supervisor which then does whatever you were starting straight from the application start callback previously. If you look at @mweibel's approach in the link he provided, you'll see what that looks like (in Elixir anyway).
Is setup a direct dependency of yours or a transitive one? From what I gather, it's somewhat at odds with using exrm, or another release handling tool, but that's just from my brief look. Do you have a specific need for it?

@bitwalker unfortunately it's dependency of exometer library which I'm using. That's why I was wondering how @mweibel solved the problem, since he mentioned in the beginning that he was using exometer as well.
I opened issue in setup repository. For now it seems the only way to make it work is to fork setup and change it to start supervisor, and probably I'd have to fork exometer as well to use my fork instead of the original one.

@arnvald No need to fork exometer, just use the override: true option for your fork of the setup dependency in your mix.exs and it will use yours instead of the one specified in the exometer mix.exs.

@bitwalker thanks for the hint! With supervisor added to setup, and using override in my mix.exs, upgrades started working.


Update README.md

typo in README

Wooo nice catch! Thanks! Since our manuscript is current under review, we can't modify the master branch. Can you please make a new pull requests to https://github.com/poloclub/cnn-explainer/tree/dev ?

Oh never mind, I can change it on my side. I have merged it. Thanks!


Icon generation not working

The following command is not working since the v2 version
pnpm  theme:generate-icons

It seems to be an issue with prettier formatting, it seem to use "flow" as the parser (which I suppose do not handle correctly this kind of imports)
I'll check to see how to override this behavior (I think we can specify a custom parser, maybe we can do it for generated icons files)

I think it is linked to this issue: https://github.com/gregberge/svgr/issues/893
I will do a PR to fix it, I will disable prettier formatting from svgr since it will still be done with lint-staged


Screen crash on view onStop

When minimize the app on broadcast view and back , the screen crashes, a loop starts on Android Monitor and screen blackout, the log is:
 io.antmedia.android.livevideopublisher I/CameraSurfaceRenderer: Drawing before incoming texture size set; skipping
I'll try fix by myself if someone know some way to fix this, please, answer this issue.
Thank you.

Is this the only log that you can get from logcat? If not, is it possible to send more log about this issue.
Regards,

Hello mekya,
Thanks for answer, here is more logs as you requested:
java.lang.IllegalStateException: setRenderer has already been called for this instance.
    at android.opengl.GLSurfaceView.checkRenderThreadState(GLSurfaceView.java:1821)
    at android.opengl.GLSurfaceView.setRenderer(GLSurfaceView.java:348)
    at io.antmedia.android.broadcaster.LiveVideoBroadcaster.init(LiveVideoBroadcaster.java:173)
    at com.eventials.eventialsbroadcaster.LiveVideoBroadcasterActivity$1.onServiceConnected(LiveVideoBroadcasterActivity.java:98)
    at android.app.LoadedApk$ServiceDispatcher.doConnected(LoadedApk.java:1223)
    at android.app.LoadedApk$ServiceDispatcher$RunConnection.run(LoadedApk.java:1240)
    at android.os.Handler.handleCallback(Handler.java:746)
    at android.os.Handler.dispatchMessage(Handler.java:95)
    at android.os.Looper.loop(Looper.java:148)
    at android.app.ActivityThread.main(ActivityThread.java:5459)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:728)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:618)
I/System.out: --- releaseCamera call in doInBackground ---
I/System.out: --- camera opened ---

Regards.

I see, I guess the problem.
Is it possible to let me know the exact scenario?
You start broadcasting.
While you broadcasting you press the home or back button?
When you return back the app, it throws this exception right?
Regards,

When I am at broadcasting view, not broadcasting (but when broadcasting happen the same thing)
), just the view open, I press the home button or the button to switch to other app, I back to the app and it throws this exception.
Regards.

Hi @clamorious
Please apply the below patch and let me know if it works or any other troubles.
diff --git a/app/src/main/java/io/antmedia/android/liveVideoBroadcaster/LiveVideoBroadcasterActivity.java b/app/src/main/java/io/antmedia/android/liveVideoBroadcaster/LiveVideoBroadcasterActivity.java
index 076bfde..a0b231c 100644
--- a/app/src/main/java/io/antmedia/android/liveVideoBroadcaster/LiveVideoBroadcasterActivity.java
+++ b/app/src/main/java/io/antmedia/android/liveVideoBroadcaster/LiveVideoBroadcasterActivity.java
@@ -65,9 +65,11 @@ public class LiveVideoBroadcasterActivity extends AppCompatActivity {
                                        IBinder service) {
             // We've bound to LocalService, cast the IBinder and get LocalService instance
             LiveVideoBroadcaster.LocalBinder binder = (LiveVideoBroadcaster.LocalBinder) service;
-            mLiveVideoBroadcaster = binder.getService();
-            mLiveVideoBroadcaster.init(LiveVideoBroadcasterActivity.this, mGLView);
-            mLiveVideoBroadcaster.setAdaptiveStreaming(true);
+            if (mLiveVideoBroadcaster == null) {
+                mLiveVideoBroadcaster = binder.getService();
+                mLiveVideoBroadcaster.init(LiveVideoBroadcasterActivity.this, mGLView);
+                mLiveVideoBroadcaster.setAdaptiveStreaming(true);
+            }
             mLiveVideoBroadcaster.openCamera(Camera.CameraInfo.CAMERA_FACING_FRONT);
         }
         @Override


Add Quicksearch Monitoring introduced in Jira Version 7.8.X; Use number input field for delay

See: https://confluence.atlassian.com/jiracore/jira-core-7-8-x-upgrade-notes-945531897.html
Also use a number input type instead of a text input type for the attachment calculation delay.

I will merge. Thank you. Just thinking today how to support for the older versions.


Failed loading '...': Check used architecture or existence of file.[Error: 126]

Describe the bug
I have a net5 wpf x32 program that references a dynamic link library where a function opens a dialog box with the current program as its owner, the first call without any problems, but when the dynamic library is re-instantiated later and the call throws an exception like the title
What code is involved

Expected behavior
Screenshots

126 (0x7E) means specified module could not be found. Including its dependencies and dependencies of each dependency... make sure.

but when the dynamic library is re-instantiated later and the call throws an exception like the title

Please provide some details how the mentioned "re-instantiated" was implemented there. And how is access going? DlR? lambda?
It would be better to see some demo project to reproduce same error for my machine.

Re-instantiation is to new class again inherited from ConariL, and the function of the dynamic link library is operated by the DLR, which, sorry, does not provide demo because it requires a specific environment to invoke the dynamic link library

Well I need at least know the hierarchy of the calling for the first requests and the second.

Re-instantiation is to new class

How? Did you release all resources before new trying?
Also keep this in your mind https://github.com/3F/Conari/issues/15#issue-527765868
btw,

inherited from ConariL, and the function of the dynamic link library is operated by the DLR,

I recommend ConariX if DLR is for primary use

Thanks for answering my question, now I use  singleton to solve this problem

Thanks for answering my question, now I use singleton to solve this problem

Good! Nice to hear that everything works well!
For everyone, please note the following

ConariX and ConariL implements IDisposable interface. Disposing must be processed with other objects to release all managed and unmanaged resources. 'Where' and 'How' depends on specific implementation. To clarify, please use MSDN and so on.

Starting with 1.4 Conari provides optional isolation between processes according to https://github.com/3F/Conari/issues/15#issue-527765868. Configure it through IConfig if you need. See our mta tests for details, for example:

https://github.com/3F/Conari/blob/7f37c6c5eb2aa28cd49ee7a2963ecdf6fcd44dd5/ConariTest/MtaTest.cs#L17
Also note that starting with 1.5 you will be able provide your custom isolation as needed, for example:
https://github.com/3F/Conari/blob/7f37c6c5eb2aa28cd49ee7a2963ecdf6fcd44dd5/ConariTest/Core/ModuleIsolationRecipeTest.cs#L20
Thanks for using!


Reintroduces Orbit.Source.

Orbit.Source provides a baseline for all sources and simplifies the
implementations of interfaces such as Transformable, Queryable, and
Fetchable.
Every source has a transformLog, transformed method, and implements
Evented.
This allows for a distinction between sources that can emit transform
events but not have transforms applied. The latter is still covered by
the Transformable interface, which now simply adds the transform
method and is more inline with the other *able mixins.

Sorry for the churn around the (Orbit/OC) Source classes. This change brings Orbit more inline with what @opsb and I settled on a while back, but still allows for the transform history tracking required by the recent refactor.

Seems like a good next step, LGTM

Thanks for the review @opsb


flickering screen - lesson 10

Hello,
When i am running 32bit_main in qemu, in lesson 10, is not working properly.
The screen is flickering, like is constantly rebooting. This only happens when i got 32 bit enabled.
When i comment out all 32bit code, and keep only 16bit is working properly.. any fixes for that?
I am sure that the problem is something about 32bit, but dont know exactly what.

I followed the lesson and it worked. If you post your code, I can try to help find the problems..

Fixed it!
i just messed up with the flags of gdt_data!


Reports errors that do not exist

What kind of issue is this?

[ ] Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use
our Community Forums http://community.platformio.org

[ ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.

[ ] PlatformIO IDE. All issues related to PlatformIO IDE should be reported to appropriate repository
https://github.com/platformio/platformio-atom-ide/issues

[X ] Bug report. If you’ve found a bug, please provide information below.

You can erase any parts of this template not applicable to your Issue.

Configuration
Windows 10
IDE 1.0.5, CLI 2.8.5 (platformio --version):
Description of problem
When I compile a program the IDE reports errors in the source code that are not emitted by the compiler in the build window. I am loading main.c from stm32-spl-blink. It compiles and runs just fine. But, the IDE says I have two errors where none exist.
Steps to Reproduce

New project for disco_l152rb with platformio.ini as below.
Set main.c to the source code below.
Build.

Actual Results
arm-none-eabi-gcc -o .pioenvs\disco_l152rb\src\main.o -c -g -Os -ffunction-sections -fdata-sections -Wall -mthumb -mcpu=cortex-m3 -nostdlib -DF_CPU=32000000L -DST
M32L152XB -DSTM32L1 -DSTM32L152xB -DSTM32L1XX_MD -DPLATFORMIO=020805 -I.pioenvs\disco_l152rb\FrameworkCMSIS -I.pioenvs\disco_l152rb\FrameworkCMSISVariant -I.pioen
vs\disco_l152rb\FrameworkSPLInc -I.pioenvs\disco_l152rb\FrameworkSPL src\main.c
No error is indicated. But, the IDE indicates:
GCC error variable or field 'simple_delay' declared void at line 22 col 19
GCC error 'uint32_t' was not declared in this scope at line 22 col 19
Expected Results
There should not be any error messages.
If problems with PlatformIO Build System:
The content of platformio.ini:
[env:disco_l152rb]
platform = ststm32
framework = spl
board = disco_l152rb

Source file to reproduce issue:
#ifdef STM32L1
	#include <stm32l1xx_gpio.h>
	#include <stm32l1xx_rcc.h>
	#define LEDPORT (GPIOB)
	#define LEDPIN (GPIO_Pin_7)
	#define ENABLE_GPIO_CLOCK (RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOB, ENABLE))
#elif STM32F3
	#include <stm32f30x_gpio.h>
	#include <stm32f30x_rcc.h>
	#define LEDPORT (GPIOE)
	#define LEDPIN (GPIO_Pin_8)
	#define ENABLE_GPIO_CLOCK (RCC_AHBPeriphClockCmd(RCC_AHBPeriph_GPIOE, ENABLE))
#elif STM32F4
	#include <stm32f4xx_gpio.h>
	#include <stm32f4xx_rcc.h>
	#define LEDPORT (GPIOD)
	#define LEDPIN (GPIO_Pin_12)
	#define ENABLE_GPIO_CLOCK (RCC_AHB1PeriphClockCmd(RCC_AHB1Periph_GPIOD, ENABLE))
#endif

/* timing is not guaranteed :) */
void simple_delay(uint32_t us)
{
	/* simple delay loop */
	while (us--) {
		asm volatile ("nop");
	}
}

/* system entry point */
int main(void)
{
	/* gpio init struct */
	GPIO_InitTypeDef gpio;
	/* reset rcc */
	RCC_DeInit();
	/* enable clock GPIO */
	ENABLE_GPIO_CLOCK;
	/* use LED pin */
	gpio.GPIO_Pin = LEDPIN;
	/* mode: output */
	gpio.GPIO_Mode = GPIO_Mode_OUT;
	/* output type: push-pull */
	gpio.GPIO_OType = GPIO_OType_PP;
	/* apply configuration */
	GPIO_Init(LEDPORT, &gpio);
	/* main program loop */
	for (;;) {
		/* set led on */
		GPIO_SetBits(LEDPORT, LEDPIN);
		/* delay */
		simple_delay(100000);
		/* clear led */
		GPIO_ResetBits(LEDPORT, LEDPIN);
		/* delay */
		simple_delay(100000);
	}

	/* never reached */
	return 0;
}

Additional info

Please see Issue 543: https://github.com/platformio/platformio/issues/543 What you are seeing is Intellisense errors, not compile errors. The solution is a bit finicky depending on your source code layout.

How did you create project? Try Menu: PlatformIO > Rebuild C/C++ Project Index.

I built the project using the instructions in the getting started, just changed the board type.
But, Rebuild C/C++ Project Index does appear to have fixed the problem. What does that do?

@charles-owen you have changed the board manually without rebuilding index. Code Linter didn't know about it. I've just opened an issue for that https://github.com/platformio/platformio-atom-ide/issues/70


Errors from time to time

curl_setopt(): The usage of the @filename API for file uploading is deprecated. Please use the CURLFile class instead
line 187:
curl_setopt($ch, CURLOPT_POSTFIELDS, $postBody);

I use it with Laravel 4.1 (up2date), PHP 5.5.11-3+deb.sury.org~precise+1

@Lechus we are indeed using CURLFile now (if supported by your system)


Add trouble shooting hint in mac os installation

1. Why is this change necessary?
A colleague just had this problem and it was pretty weird that it happened randomly. The error was easy to resolve but as shopware was the way to go one had a look into shopware repositories first. So there is now a hint on how to solve this bug. Great!
2. What does this change do, exactly?
Add a section about case sensitivity features of MacOs and used tools in shopware.
3. Describe each step to reproduce the issue or behaviour.

cd /users/foobar/code/production
bin/build.sh

4. Please link to the relevant issues (if any).
https://forum.shopware.com/discussion/64736/installationsprobleme-shopware-6-dev they could need that answer as well.
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-7145
Please use this issue to track the state of your pull request.

Can someone with a forum account please point the other guys to this change? https://forum.shopware.com/discussion/64736/installationsprobleme-shopware-6-dev

Yes sir!

Thank you very much :) 💙

thanks for your contribution @JoshuaBehrens :+1: :tada: :blue_heart:


Some issues with Google Calendar

Description
First off, love this project! I've been looking for something exactly like this.
Having some issues when importing to Google Calendar. Here's the unexpected behavior:

Events don't have details like the leekduck url or image (I see in the .ics file that these details are there)
Events are in the Universal Time zone and I am unable to change it

I'm not sure if these issues are purely Google's fault and there's no workaround. Just wanted to report my findings
Version
N/A - pulled ics file on Jan. 4 2023
Steps to Reproduce

go to google calendar settings
click "Add a Calendar" > "From URL" should be this screen
input the gocal url: https://github.com/othyn/go-calendar/releases/latest/download/gocal.ics

Screenshots
No response
Device
Macbook Pro, Samsung S22 Ultra
Operating System
mac, android
Additional Context
No response

Thank you! Glad its helped.

Events don't have details like the leekduck url or image (I see in the .ics file that these details are there)

So far, I've not yet used a calendar client that will render the images. Apple Calendar (the one I use) also doesn't, I think it may just be an Outlook thing.
As for the URL, I'm not sure. I don't really use Google Calendar and spending some time searching around the web for issues related to this doesn't yeild anything worth while.
I loaded the calendar into a spare Google account I have and I can see the issue with it not coming through. Given the lack of information online, I played around with it a little and I think its due to Google not having a 'URL' field. I'm only going on the fact that if I create a new event in Google Calendar, there is no specific field to enter a URL into like in Apple Calendar (for example).
As a work around I can simply add the URL to the event description so its located in multiple places for calendar clients that don't support the native URL field.

Events are in the Universal Time zone and I am unable to change it

Ah, this issue again haha. I've spent a long time trying to fix this.
In its current state, there is zero timezone information embedded in the generated ICS file, the TZID for the calendar and all associated timezone information at calendar level is omitted and the same goes for each event. This including the removal of Z as a suffix on event times to ensure the event isn't interpreted as UTC, that took a while to solve and resulted in commit f5b52438f84bbee3423da85a669a994c512161f4, at which point I thought the timezone issue was put to bed.
Given no timezone information, the calendar client is meant to interpret that as local time and act accordingly to place events into the calendar at the time stated in the event.
I've spend the last few hours debugging this between Apple Calendar and Google Calendar and trying various combinations of setting and omitting the timezone, and nothing seems to work consistently. Apple Calendar appears to follow what the RFC guidance states, reacting to local time if there is no timezone set. However, Google Calendar appears to do the same when importing a local .ics calendar file - this appears to change though when doing it from a URL, as it defaults to importing at UTC no matter how I try and skew it.
Searching around online, I can't really find anything that documents or proposes a work around to solve this behaviour.
Really sorry to say, but I think its a Google thing. All timezone information is stripped from the .ics files to allow for clients to interpret it as local time. Google Calendar seems to just yolo onto UTC no matter what though...
The only real fix for this (as it appears) is to generate timezone specific variants of each calendar alongside a local time one, for clients that struggle with interpreting local time events, which at this time appears to just be Google Calendar over 'URL Subscriptions' only. A really weird behaviour.

GH auto closed as I mentioned it in a commit, but I'll re-open it as I do want to explore timezone specific variants as a possibility.

Thanks for the response! It does look like a Google thing. I'll still use it and know that community days and such will just have the wrong time on my calendar until Google improves things on their side.
Thanks again for the great project!

🙌 I'm currently working on doing timezone variants of the calendars, will ping you once its released.

So... this is mainly just for future reference on this thread instead of a direct reply.
Looking at https://docs.github.com/en/repositories/releasing-projects-on-github/about-releases, there is no limit on release attachments or bandwidth.
The reason this is an issue is under the current generation, with the combination of all timezones and calendar types, its around ~4,000 calendar files that are generated.
I'm not sure on whether to release this or not, as its a lot of release assets to publish and I'm not sure if GitHub will even do it. Especially at a daily release frequency.
I'd love there to be a more generic and smaller timezone set to support, but I'm not having any luck on finding a compomise between a decent coverage vs. published timezones.

Hey, loving the service, and I'm just chiming in to say I'd also love to see the timezone fix coming in to play, and drop in my suggestion:
I can think of two approaches for the timezone problem:
Option 1, Easier: Ignore country specific timezones, and just use 24: UTC-12, ...,UTC-1, UTC, UTC+1, ..., UTC+12. This will reduce the calendar account to a couple hundred, instead of around 4k.
Option 2, A bigger solution, but I think this would be the next level service:
Have n endpoint on the site like /cal or something, that reads the current releases and adjusts them with two flags:
events (or filter, or any other name) and tz.
So actually only generate the everything calendar, have the endpoint read and cache that, and on request, filter the events and adjust the timezones.
Example:
https://gocalendar.info/cal -> return the everything calendar with UTC tz.
https://gocalendar.info/cal?tz=Pacific/Tahiti -> return the everything calendar with Pacific/Tahiti tz.
https://gocalendar.info/cal?tz=Pacific/Tahiti&events=community_day,elite_raids,event -> return only events from these categories + adjusted TZ to Pacific/Tahiti
I think it should be fairly straightforward to create a template file for the backend to quickly replace the values when serving the calendar data, and speed is not really a consideration because it's the calendar app that fetches it, not a user sitting and waiting for things to load.
I'm happy to collaborate on creating the second solution if this is something you would consider implementing!

Im not a Coder but a Pokemon Go player desperately looking for a Pokemon Go Calendar so idk cant say anything helpful, but i noticed that when i pull the calendar from gocalendar.info (only for communty day: https://github.com/othyn/go-calendar/releases/latest/download/gocal__community_day.ics) it puts the wrong time into the goole calendar, but when i add it from https://github.com/othyn/go-calendar/releases and insert the latest release .ics from there (https://github.com/othyn/go-calendar/releases/download/v2.3.57/gocal__community_day.ics) into google calendar, it adds it perfectly in local time. maybe there is kind of a problem? idk

@selfish awesome suggestion, I'm happy to accept PR's on this.
Option 2 is an awesome suggestion and was my original path. In typical systems design it would be the approach to take. One of the projects core requirements though is that its all hosted on GitHub Pages, as I wanted to keep it light weight, easily automated and free to run. As some form of backend will be required to ensure the calendar clients can use the endpoint, this approach cannot easily be taken without also overhauling the infra side too and breaking those requirements. So although its absolutely the right path to take, I'm not sure its perhaps the best one in this scenario.
Option 1 is a really good shout though, I hadn't considered that. When I have time I can look into that avenue and see what the results are, with some form of dropdown on the website that pairs off where you are to what timezone version you need for ease of UI. Or, happy to accept a PR! 😉


Recreate the manual testing of share encoding

Why is this test commented?
Originally posted by @SweeXordious in https://github.com/celestiaorg/celestia-app/pull/627#discussion_r948585018
When moving share encoding and decoding logic to the app, we had to comment out a test because it was designed around some tendermint logic, like creating votes and using methods instead of functions to encode shares. We should recreate this test by changing those requirements.

we have since added quite a few manual tests, and I'm not sure of the value that we would get from this anymore. I guess the manual tests would act as a forcing function to notify reviewers if anything changes the encoding, but that's about it.
Leaving this as a stretch issue for the epic, but I am also okay with closing this as unplanned. Will defer to @rootulp

This issue refers to the TestMakeShares test which was commented out and eventually removed. It appears to have tested
// 			got := tt.args.data.SplitIntoShares()

which no longer exists but it resembles https://github.com/celestiaorg/celestia-app/blob/0b85a0fabf07ff02bf03e9205638c6cd5462afba/pkg/shares/share_splitting.go#L25 which doesn't have unit tests so I think we can update this issue to add unit tests to that function.

That file has been migrated to the go-square repo: https://github.com/celestiaorg/go-square/blob/main/shares/share_splitting.go
SplitTxs has tests but SplitBlobs doesn't: https://github.com/celestiaorg/go-square/blob/main/shares/share_splitting_test.go


Plugin does not work

Hi guys, you do a great job and your plugin is one of my favourites plugins for Sketch, but about a month it doesn't work, I can't export my design a process freeze on a progress bar stage. Can you help to solve this problem?

@PaulMinami , please give this a try:
Remember to uninstall the plugin, restart sketch and install the downloaded version, just to make sure there's no cached plugin information.
Also, make sure you're using Sketch v53.2 (72643).
Get the Fork version (v2.8.0)
Download from sketch-measure-latest
Still not working?
Follow the steps below:

[ ] Uninstall the plugin
[ ] Quit Sketch
[ ] Follow the steps bellow to remove the PluginsWarehouse folder
[ ] Download the latest zip file from the fork
[ ] Reinstall the plugin
[ ] Test the Mark Properties / Spec Export again

Remove the PluginsWarehouse folder that lives in ~/Library/Application Support/com.bohemiancoding.sketch3/. This is where we cache plugin downloads, and if you’ve been testing different versions of your appcast, you probably have some old stuff there that’s worth cleaning.

Go to the Finder of Mac OS
From the “Go” menu navigate down to “Go to Folder” or...
Hit Command+Shift+G from the Mac OS X desktop or a Finder window
Paste the folder location ~/Library/Application Support/com.bohemiancoding.sketch3/
Remove/Delete the PluginsWarehouse folder

Let me know how it goes 😊
If after all this it still doesn't work, please share a small sketch example file (doesn't need to be your actual design file) that you can reproduce the bug and I'll take a look.

Hi,
Really love this plugin too, i just got the plugin to work by doing this:

Remember to uninstall the plugin, restart sketch and install the downloaded version, just to make sure there's no cached plugin information.
Also, make sure you're using Sketch v53.2 (72643).

Thanks @ellunium 🔥

I have the same problem, followed your suggestions and it didn't work. Sketch Measure used to work for me when I was on Sketch 53.1 and Sketch Measure 2.4. When I upgraded it stopped working. I use Sketch 54.1 and latest sketch measure.


Update ketcher toolbar UI styles

Solution
Update styles of ketcher toolbars and other visual elements so they fit the new design.

Update top, left and bottom toolbar styles
Update Right toolbar atom icon styles and icon arrangement
Update colors, including background color

Note keep in mind an ability to hide buttons via ketcher settings/api

Toolbars:

Colors and states:

Canvas background: #FCFCFC;
Toolbars drop-shadow(0px 2px 5px rgba(103, 104, 132, 0.15));
Top bar drop-shadow(0px 2px 5px rgba(103, 104, 132, 0.1));

Toolbar margins: 12px from the sides of the canvas

Atoms are not greyed out on hover and on select
https://github.com/epam/ketcher/assets/62464241/1c3db7c3-68a8-4ed9-945e-d011a5ed908a


Cross Entropy loss or NLL loss

Hi, Thanks for the nice repo!
I just wanted clarification about the NLL criterion in the code. Are the results in Birds.out using Cross Entropy criterion or NLL criterion?
Also, I tried running the code as is (using NLL criterion), train and test loss becomes nan.
Thanks once again.

The results in Birds.out are using criterion_NLLLoss.
And I re-run the code in this repo, all thing is fine.
Please check your environment and the data preparation.

Thank you!


Update osm-arc version to 1.2.5-beta-1.

[ ] I have ensured that required images are available in MCR:

osm-controller, osm-injector, osm-bootstrap, osm-crds and init images of the corresponding chart version
envoy image of version listed in osm-arc/oss values.yaml
grafana, grafana-image-renderer, prometheus, jaegertracing/all-in-one image of version listed in osm-arc/oss values.yaml

[ ] I have ensured that all the images mentioned in the previous step (aside from grafana-image-renderer) are available for both amd64 and arm64 architectures

[ ] I have updated the Helm chart:

Updated the chart version, app version and dependency version in charts/osm-arc/Chart.yaml
Made applicable updates to the osm-arc values.yaml if any were made in the upstream OSM chart

[ ] I have received 3 approvals from maintainers.
Description: Update chart version to 1.2.5-beta-1.

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Networking             [ ]
Metrics                [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

/azp run pr-e2e-job

/azp run upgrade-e2e-arc

/azp run pr-e2e-arc

/azp run upgrade-e2e-arc


Add webhook to prevent duplicate KataConfig instances

A KataConfig instance is a representation of Kata installation on cluster worker nodes therefore it'd be hard at this point to assign a meaningful semantics to multiple KataConfig instances in a single cluster.  Yet it's been possible so far to create multple Kata instances.
This PR fixes the issue by adding a validating webhook for KataConfig resources that prevents creation of additional instances if one already exists in a cluster.
The fix can be verified by attempting to create a duplicate KataConfig instance.  The expected result is an error message saying that the operation failed due to being denied by the webhook.
- Description for the changelog
Prevent creation of duplicate KataConfig instances.

@pmores There's some code in controllers/openshift_controller.go isOldestCR() that we used to prevent two kataconfigs being created before we had a webhook. I think we can (have to?) remove at least part of this code now. What do you think?

@jensfr I agree, this should be obsolete now although I'd prefer to remove it in a separate PR.  The obsolete code shouldn't interfere with webhook operation - the webhook runs before our reconciliation loop which means the old code guarding against duplicate KataConfigs should simply never trigger anymore.

We already have a test case to test duplicate KataConfig's. So I think we can merge this now.


Testing Vuex actions returns undefined

Hey there,
I couldn't find anything on SO so I'd like to ask here about. I recently moved from plain JS to TS and therefore started using vuex-class-modules. However, all of my tests using VueX that were working previously just stopped working with vuex-class-modules.
Is there anything special to take care about using vuex-class-modules?
I'm running tests on Jest.

After some more digging into vuex-class-modules I have to extend my question.
Using plain Vuex actions get triggered in my test and do what they are supposed to do. However, when I set up my test with vuex-class-module, by defining a VuexModule and putting in everything I need, my test still breaks. Is there by any chance some kind of 'how to' for working with vuex-class-module?

If the mocked module should return a value you need to define that with mockResolvedValue. See https://jestjs.io/docs/en/mock-functions.html#mocking-modules

i have the same problem. Is there any solution?

This issue was solved afaik. What additional problem do you have?


[Don't MERGE] CI 2.0, revamp ci scripts and dockerfiles, integrate IoT builds

Description
Checklist
Essentials

[ ] Passed code style checking (make lint)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

I created a new PR at https://github.com/apache/incubator-mxnet/pull/9946 as I don't have write access to pedros repo and I'll take care of the migration and integration


Last update, launch_gui.bat

Hi Boss!
I've tested last version and can't find some files:
call .bot_env\Scripts\activate.bat - where I can find folder Scripts with this bat file?

Hello,
Cool that you tested it! I tried to make sure the old notebook still works for now, so that's always an alternative and it works the same way.
The launch script just tries to run the Src/gui.py file in its own python environment. I'm guessing the issue is that you haven't run the install.bat, which is what creates and installs all the Python packages needed.
I can probably make it so the install file is run automatically if the environment is not created yet.
When you try to start the bot with the gui it will also try to install Scrcpy in the local folder, so the previous Scrcpy installation is not needed anymore.

I had, but it also trying to find directory script:
C:\Code\Rush-Royale-Bot>C:\Code\Rush-Royale-Bot\install.bat
C:\Code\Rush-Royale-Bot>python -m venv .bot_env
Python
C:\Code\Rush-Royale-Bot>call .bot_env\Scripts\activate.bat
Системе не удается найти указанный путь. (The system cannot find the specified path.)
If you want I can give you access to test

The python -m venv .bot_env command should create a virtual python environment located in the bot directory. Scripts\activate.bat is a base python script to activate the environment which will run the gui.

You also don't need to do this step, but it helps with compatibility if you are using python for other things on your computer.

but I can't run gui :(
And I don't have activate.bat file anywhere (in python directory scripts also)

Can you please copy this file for me?

The base python installation won't have a activate script, only venvs do https://docs.python.org/3/library/venv.html
I just did the whole installation process on my laptop probably had the same thing as you because Python was not added to path, or when I type python it goes to the Microsoft store. You can turn that off in windows settings under 'Manage app execution aliases', but I also just made it look in app data for python39.
%LOCALAPPDATA%\Programs\Python\Python39\python -m venv .bot_env

call .bot_env\Scripts\activate.bat

pip install -r requirements.txt

You can try run this in the bot folder.

Thanks a lot!


Add: Command that accesses WolframAlpha API

Add a command !math <math problem>. When the user executes this command, the bot must make a request to the WolframAlpha API using this package. The reply contains a set of "pods". Sort through these pods and return those that are useful for answering the question.
You may take a look at the website here

@z404 Please assign me this issue!

@z404 this somehow doesn't seem to work. Am I doing something wrong? The app id is functioning properly.
`var mathProb = strings.SplitN(m.Content, " ", 2)[1]
c := &wolfram.Client{AppID: "app-id"}
		//Get a result without additional parameters
		res, err := c.GetQueryResult(mathProb, nil)

		if err != nil {
			panic(err)
		}
		s.ChannelMessageSend(m.ChannelID, res)

		// Iterate through the pods and subpods
		// and print out their title attributes
		for i := range res.Pods {
			s.ChannelMessageSend(m.ChannelID, res.Pods[i].Title)

			for j := range res.Pods[i].SubPods {
				s.ChannelMessageSend(m.ChannelID, res.Pods[i].SubPods[j].Title)
			}
		}`

@z404 this somehow doesn't seem to work. Am I doing something wrong? The app id is functioning properly.
`                var mathProb = strings.SplitN(m.Content, " ", 2)[1]
c := &wolfram.Client{AppID: "app-id"}
	//Get a result without additional parameters
	res, err := c.GetQueryResult(mathProb, nil)

	if err != nil {
		panic(err)
	}
	s.ChannelMessageSend(m.ChannelID, res)

	// Iterate through the pods and subpods
	// and print out their title attributes
	for i := range res.Pods {
		s.ChannelMessageSend(m.ChannelID, res.Pods[i].Title)

		for j := range res.Pods[i].SubPods {
			s.ChannelMessageSend(m.ChannelID, res.Pods[i].SubPods[j].Title)
		}
	}`

@z404 this somehow doesn't seem to work. Am I doing something wrong? The app id is functioning properly.
`var mathProb = strings.SplitN(m.Content, " ", 2)[1]
c := &wolfram.Client{AppID: "app-id"}
	//Get a result without additional parameters
	res, err := c.GetQueryResult(mathProb, nil)

	if err != nil {
		panic(err)
	}
	s.ChannelMessageSend(m.ChannelID, res)

	// Iterate through the pods and subpods
	// and print out their title attributes
	for i := range res.Pods {
		s.ChannelMessageSend(m.ChannelID, res.Pods[i].Title)

		for j := range res.Pods[i].SubPods {
			s.ChannelMessageSend(m.ChannelID, res.Pods[i].SubPods[j].Title)
		}
	}`

@z404 this somehow doesn't seem to work. Am I doing something wrong? The app id is functioning properly.
            var mathProb = strings.SplitN(m.Content, " ", 2)[1]

            c := &wolfram.Client{AppID: "app-id"}
            res, err := c.GetQueryResult(mathProb, nil)

	if err != nil {
		panic(err)
	}
	s.ChannelMessageSend(m.ChannelID, res)

	// Iterate through the pods and subpods
	// and print out their title attributes
	for i := range res.Pods {
		s.ChannelMessageSend(m.ChannelID, res.Pods[i].Title)

		for j := range res.Pods[i].SubPods {
			s.ChannelMessageSend(m.ChannelID, res.Pods[i].SubPods[j].Title)
		}
	}

I'm assigning you to this issue!

I'm afk at the moment, I will test the API on my end and put up a post here when I have an update!

@z404 sure!

As is seen with most outdated APIs, this one is very broken.

I'm trying to get this to work directly using the API endpoint, as I did here: #31. I'll add a comment once that works.

@z404 I got the exact same output with this. Definitely broken. However I found another work around with the GetSpokentAnswerQuery function and this seems to work just fine.
                        res, err := c.GetSpokentAnswerQuery(mathProb, wolfram.Metric, 1000)
			if err != nil {
				panic(err)
			}
			s.ChannelMessageSend(m.ChannelID, res)

![image](https://user-images.githubusercontent.com/34034274/195894103-16656ab7-e631-4730-b239-a4b5d3c76dc1.png)

Nice! This works perfectly, though the output seems a little odd (as it is meant to be spoken, not shown as text). We can work with this for now, and if need be, correct it later. Proceed with using this work-around. Good job :D

@z404 yeah the output is a bit weird but def works! Since the search applies for generic queries as well, should we name the call the command !math or something else maybe?

If you have a better name in mind, feel free to change it!

@z404 alright thanks!


[GitHub] Downloads per week, month etc. missing

Like npm, packagist, hex.pm etc. have it.

Not even sure it would be possible for specific releases either.

Closing as i don't believe that either GitHub or a 3rd party service can provide us with the required information currently.
If this changes in the future feel free to reopen this issue and we can look at it providing this again :smile:


Replace requirements*.txt with pipenv

requirements.txt and test-requirements.txt should be replaced with Pipfile.
Then Travis-CI configuration should be changed.
More info about pipenv can be found there.

Hi @destag I'm on it

Hi @Dikaeinstein! There is #10 PR, but you can also try this. :relaxed:

Ok, I'll give it a try


Suppress Logs based on Flag [Suggestion]

I would really like to see a way to suppress the the output. Or even a way to filter some of the output.

doesn't belong in fancy-log, the implementing CLI should handle the flag and not log, this is how gulp handles it

@phated What I meant was, there is instanced where you do not want the "sync:group 0" logs to be shown but others yes. It would be able to filter if both the timestamp and the message where going through console.log but they are not.


⚠ ClamAV has degraded performance

In 4001915, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 76 ms

Resolved: ClamAV performance has improved in 2fb6fdc.


Install script failed -Did not find any OpenCV Libraries

I installed this last week,  it worked fine, I started over on a new install today , not working
Ideas?
pi@raspberrypi:~/allsky$ ./install.sh

*** Welcome to the Allsky Camera installer ***

Dependencies installation

make: Entering directory '/home/pi/allsky/src'
Makefile:42: *** Did not find any OpenCV Libraries, try 'make deps'.  Stop.
make: Leaving directory '/home/pi/allsky/src'
Installing dependencies failed!
Thanks

For anyone else having same issue,  installing libopencv-dev did the trick
apt install libopencv-dev
:)

Resolved after install libopencv-dev

I realize you've got this all taken care of now.. but I'm curious the flow you took that landed you into these errors ( and those from your other ticket(s)).
Did you run the install.sh script in the top level directory, as your regular user?  ( presumably the pi user ).
If so, can you provide the console output ( if it's still around ) from that run...
I ask, because the installer should have added those dependencies for you ( libopencv-dev ) and handled all of the permissions, etc.  So if that all failed, we'd like to know and fix it for future users.

I have a reproducer for this now.. and am working on it.


Error in zippy.hpp：: : Microsoft C++ exception: std::bad_alloc, Located at memory location 0x00EFD5C8.

When I tried to test demo6 and demo7, I encountered the above problem. In the zippy.hpp file, line 10047 showed an error. It happened in the save stage. My compiler is: Vs2019 32-bit. The memory of my host is 32G. Is there any way to solve it?

In the graph of your debugging environment, it shows only 2 GB of memory, with a steady increase in memory usage, eventually plateauing at the maximum. Going by this alone, I would assume that you have a memory leak in your program or environment, and the bad alloc is actually a bad allocation where the program fails to allocate memory for a trivial variable assignment.
Is it possible that you are having actual out of memory situation? Try increasing the limit for your program memory to 4GB and see if the same problem occurs, but elsewhere in the code.

Yes, I understand. As you said, for a 32-bit x86 program, the available space for the process under VS is 2G.

在您的调试环境图表中，它只显示 2 GB 的内存，内存使用量稳步增加，最终稳定在最大值。仅从这一点来看，我会假设您的程序或环境中存在内存泄漏，而错误的分配实际上是错误的分配，即程序无法为简单的变量赋值分配内存。 您是否有可能遇到实际的内存不足情况？
编辑：确实，正如您所说，您使用的是 32 位编译器，根据定义，您的进程内存被限制为 2GB。我非常确定您存在内存泄漏（或者为程序分配了太多内存）。这看起来不像是 zippy 中的错误。
编辑2：我刚刚看到它是 Demo6，它创建了一百万行，每行有 4 个值 - 这实际上可能需要超过 2GB 的内存，所以我很抱歉假设您可能有内存泄漏。但是，是的，在这种情况下，Demo5/Demo6/Demo7 程序不适合 32 位进程。

Yes, I understand. As you said, for a 32-bit x86 program, the available space for the process under VS is 2G.

Closing this issue then. I don't think it's important for the "load test" Demos to work under 32 bit :)


Coordinator logs full of warning message about aggregation key

Doing a smoke tests showed that most likely count query is run as count(*), which prints bunch of warnings like:
Aggregation query used without partition key (ks: jsonapi_smoke_test, tbl: docs_collection)

Not sure if #424 fixed this.. Maybe it can be simply closed..


realms: Create default system user groups for internal realm.

Since we include internal realms while creating system groups
in "0382_create_role_based_system_groups.py", we should do it
when creating new internal realms as well to be consistent.
This PR also modifies bulk_create_users to add the users
to the respective system groups.
And we would also need a migration to add the groups to internal
realms created after the previous migration.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@timabbott this is ready for your review.

@timabbott can you review this? Would be good to get this one completed as I will open the next PR for adding a new setting based on groups model tommorrow, and would be nice to get this one off the list.

This looks to me to be a correct implementation of fixing this inconsistency by adding these groups to the system bot realm.
I feel like it might be more correct to instead remove them from the system bot realm, if that realm doesn't contain users beyond the system bots... but I'm not sure how to code that in a safe fashion.
@mateuszmandera thoughts? I can see going either way on this, given our eventual goal to eliminate the system bot realm as a concept...

We currently define internal bots in settings.INTERNAL_BOTS so we can probably check through emails I think. Not think there would be many of them with users other than system bots.

I feel like it might be more correct to instead remove them from the system bot realm, if that realm doesn't contain users beyond the system bots... but I'm not sure how to code that in a safe fashion.

Hmm, what would the advantage be? Isn't it simpler to just have the clear property the system realm has system user groups rather than it being conditioned on whether the realm has additional human users or not, with all the extra code that would be needed for ensuring that conditionality?

Added the assert statement and a comment in migration as asked above.

Added commits to add constants for user group names wherever requried and using them and SYSTEM_USER_GROUP_ROLE_MAP to get user groups from names and in tests.

This is great, merged with a few changes:

The tweaks mentioned here: https://github.com/zulip/zulip/pull/22352#discussion_r941909110
Made sure the comment for the migration appears at the top of the file; that's more likely to be visible when looking for the information. I left a copy at the bottom as well.

@timabbott  This was not merge automatically because CI was failing due to linter error. I fixed it and also added commits for other group names. CI is not passing due to puppeteer error, which I think would be fixed by #22692.

Ohh, this got auto-merged as I repushed again for CI to pass.

Yeah, that's a possibly surprising interaction with how GitHub's "merge once CI passes" feature works -- basically it'll merge the next time the PR passes CI, regardless of whether there are actual changes or not.
In this case, I've reviewed the new commits and they look great, so not a problem :)


[Creator] Error running the Creator tool

Hi!
I´m not able to run the Creator tool. I don´t have any experience with .net but the error message sound like I cannot run the Creator with the SDK I have installed on my machine. Is there something I can do other then remove the SDK I have installed (3.1) and replace it with an older version (2.1)
Here is the message I got:
It was not possible to find any compatible framework version
The framework 'Microsoft.NETCore.App', version '2.1.0' was not found.
  - The following frameworks were found:
      3.1.0 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

You can resolve the problem by installing the specified framework and/or SDK.

Thanks in advance!

did you run "dotnet restore" under this project ? It shall automatically install the required packages, if not, you can install the 2.1 version package manually, it won't replace the newer version.

Hi!
Yes! I ran "dotnet restore" but the error remains.
I´ll try to find the 2.1 version package... here is what I´m getting now:
PS G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate> dotnet restore
  Restore completed in 54.57 ms for G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate\apimtemplate.csproj.
  Restore completed in 55.79 ms for G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate.test\apimtemplate.test.csproj.
PS G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate> dotnet run create --configFile G:\Getnet\Azure\DevSecOps\Git\api-management-test\echo-api\config.v1-rev1.yml
It was not possible to find any compatible framework version
The framework 'Microsoft.NETCore.App', version '2.1.0' was not found.
  - The following frameworks were found:
      3.1.0 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

You can resolve the problem by installing the specified framework and/or SDK.

The specified framework can be found at:
  - https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=2.1.0&arch=x64&rid=win10-x64

Thanks!

Hi!
I´ve installed 2.1 version and now I´m getting another issue but it is probably related with some problem on my config files!
PS G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate> dotnet restore
  Restore completed in 492.44 ms for G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate\apimtemplate.csproj.
  Restore completed in 492.48 ms for G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate.test\apimtemplate.test.csproj.
PS G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate> dotnet run create --configFile G:\Getnet\Azure\DevSecOps\Git\api-management-test\echo-api\config.v1-rev1.yml
Common\TemplateModels\ProductGroupsTemplateResource.cs(19,22): warning CS8632: The annotation for nullable reference types should only be used in code within a '#nullable' annotations context. [G:\azure\api-management\azure-api-management-devops-resource-kit-master\src\APIM_ARMTemplate\apimtemplate\apimtemplate.csproj]
API configuration is required

I´m closing this issue!
Thanks for your help!


Example code showed to Cache Composer Dependencies doesn't work with https://github.com/nektos/act because of the hyphen in step id

Describe the bug
Using the id composer-cache for the step Get composer cache directory creates the following error on the following step :
ERRO[0047] Unable to interpolate string '${{ steps.composer-cache.outputs.dir }}' - [ReferenceError: 'cache' is not defined]
see https://github.com/nektos/act/issues/104
Version

[ ] I have checked releases and the bug exists in the latest patch version of v1 or v2.
[X] v2
[ ] v1

Runners

[X] GitHub Hosted
[ ] Self Hosted

Operating systems
Ubuntu 16.04
PHP versions
7.2
To Reproduce
- name: Get composer cache directory
        id: composer-cache
        run: echo "::set-output name=dir::$(composer config cache-files-dir)"

      - name: Cache dependencies
        uses: actions/cache@v2
        with:
          path: ${{ steps.composer-cache.outputs.dir }}
          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
          restore-keys: ${{ runner.os }}-composer-

Then launch  act -P ubuntu-16.04=shivammathur/node:xenial
Expected behavior
The expression ${{ steps.composer-cache.outputs.dir }} the be correctly resolved
Additional context
I believe it has something to do with this : https://github.com/nektos/act/issues/104

@fpalamour Thanks for reporting this, I will update the documentation and examples in next release.

@shivammathur No worries, thanks for your great work.


Add kustomize as app delivery

We should add Kustomize as an app delivery approach.

By we you mean, that you are submitting an example :)

An example for kustomize has been added.


Intentionally leak our Application, so that we DON'T crash on exit

We've got a weird crash that happens terribly inconsistently, but pretty
readily on migrie's laptop, only in Debug mode. Apparently, there's some
weird ref-counting magic that goes on during teardown, and our
Application doesn't get closed quite right, which can cause us to crash
into the debugger. This of course, only happens on exit, and happens
somewhere in the ...XamlHost.dll code.
Crazily, if we manually leak the Application here, then the crash
doesn't happen. This doesn't matter, because we really want the
Application to live for the entire lifetime of the process, so the only
time when this object would actually need to get cleaned up is during
exit. So we can safely leak this Application object, and have it just
get cleaned up normally when our process exits.

[x] I discussed this with @DHowett-MSFT and we both agree this is mental
[x] I'm pretty sure there's not an actual bug on our repo for this
[x] I verified on my machine where I can crash the terminal 100% of the time on exit in debug, this fixes it
[x] I verified that it doesn't introduce a new crash in Release on my machine

@DHowett-MSFT should we selfhost this first as a sanity check? this seems like it can't be right, but hey, it works on my machine

Sure, I'll spin a selfghost build.

It doesn't seem to be causing issues on Release for me.

It doesn't seem to be causing issues on Release for me.

Good enough for me ¯\_(ツ)_/¯


[iOS/Android] Error “Incorrect Backup Key...” is not shown every time when user submit an incorrect backup key.

Frequency: 100%
App version: Test Flight build v1.5.1(14)
Repro on: iPhone 7 (13.1), iPhone 7+ (12.4)
Testing Account:<PHONE_NUMBER>/Backup Kay
Pre-condition: User is on Confirm your backup key screen.
Repro Steps:

Launch the app.
Tap on the Cog icon.
Tap on Backup and recovery options.
Tap on “Get your key” button.
Tap on the continue button.
Select randomly word tag and tap on the Submit button.
Make sure the error message is shown.
Select randomly work tag again and tap on the submit button.
Observe

Impact: Users won’t get the idea about the failure in the backup key.
Current Behavior: Error “Incorrect Backup Key...” is not shown every time
Expected Behavior: Error “Incorrect Backup Key...” should be shown every time
Attachment: iOS_ErrorMessageIsnotShownForSecondTime.mp4 (Bug observe at 22 sec)

Verified with the Android Play store build v1.5.1 (104694405) and observed the same issue: Error “Incorrect Backup Key...” is not shown every time when user submit an incorrect backup key.

Hi @i1skn @jmrossy I have verified this issue using Android internal build v1.2.0<PHONE_NUMBER>) and observed that issue is fixed and error massage is showing every time when user enter incorrect account key.


As an EVCM, I want to read some explanation of the scoring methodology, I understand how my community was or was not prioritized.

Description
We describe clearly to the EVCM user how the data indicators are combined into a cumulative score for the census
Solution
An explanation of how the data indictors are transformed into a scoring methodology for a census block group is shown as a pop up
Describe alternatives you've considered
Links to user research or other resources
Tasks

[ ] Understand/define how each data indicator is scaled (0-100)
[ ] Understand/define how multiple data indicators would be combined
[ ] Create user facing blurb of that definition
[ ] Wireframe for pop up
[ ] Put blurb in pop up

Definition of "Done"

[ ] pop up with explanation of scoring methodology blurb appears

I think explanation of the scoring methodology should be prioritized for Sprint 3, and having a pop up for this is lower priority than having the \methodology (or similar) page. I'll work with Design to rescope this.

Updated scope of this card to reflect latest discussion and designs, focusing on Methodology page for July 27th launch

@KameronKerger will update PDF today. @BethMattern has "Screening Tool Methodology - Content for Site" on shared drive - Jane will start using what's here in the meantime to start laying this out. We should make sure close to final designs are ready for Sprint 4 this Thursday

The sprint 4 designs Jane provided are up to date

Closing as duplicate of #144 and #351


What is the purpose of PROTECTION_PREFIX in default transformResponse?

This is question is about this regex replace().
I've looked in the issues, I've looked in the history and I've looked in the unit tests - but I couldn't find any clues - there doesn't seem to be an assert for it or any comments?
I'm happy to PR a config setting or a removal of this option - but for the use case I'm working with, it is not necessary, yet highly costly - Chrome seems to allocate memory for the copy of the whole response to do the regex replace/match. Considering, that the response I'm working with can go as high as 250Mb - doubling the RAM usage for just that one request becomes quite painful and occasionally leads to out of memory errors. I misinterpreted the profile results - despite the fact that the regex replace allocates a lot of memory, there's even more memory that needs to be allocated for the JSON.parse anyways.
As a workaround, we'll try to use a custom, simpler transform, yet I'm still wondering whether this should be the default behavior?
Thanks.

It's JSON vulnerability protection. You can read about it here. The vulnerability itself is discussed here.
Since the problem exists only in Firefox 2.0 or earlier, I think we can get rid of this code.


:sparkles: Deprecate dbt versions < 1.0.1

Context
dbt now has 5 minor versions in V1 so palm should support all five and deprecate support below V1.
Is your feature request related to a problem? Please describe.
Palm rejects the two newest dbt minor versions, which it should support. Also,  <= V0.19 is not supported by dbt any longer, so it doesn't need to be supported by palm either.
Describe the solution you'd like
Deprecate v0.19 and add support for dbt v1.4 and v1.5
Describe alternatives you've considered
N/A
Additional context
Is there an existing feature request for this?

[X]  I have searched the existing issues

Thanks for creating this issue. We will need to keep supporting dbt v0.19 for now since we have a legacy project running on that version. As soon as we are able to drop support internally, we will!
We are also using palm-dbt with dbt v1.4 and haven't run into any issues. Can you provide reproduction instructions for "Palm rejects the two newest dbt minor versions"

Specifically, palm containerize doesn't allow you to generate new boilerplate if you specify a version greater than 1.3. I am sure a manually created image would work just fine but the maximum_version property specifies 1.3 as the max supported version.

Would you mind creating a new issue to increase the maximum supported version for containerization? That can (and should) be done separately from deprecating support for <v1


How to unsubscribe from iframeTracker?

Hello, given that I have subscribed to some iframes with:
$('iframe.youtube').iframeTracker({
        blurCallback: function(){
            console.info('youtube');
        }
    });

What's now the recommended way to unsubscribe from clicks on these iframes?

Hello, there is currently no way to unbind an iframeTracker handler, but I think I will implement it soon since you are not the first user needing this feature.
Basically, to remove the handler, you have two things to do :

Unbind mouseover & mouseout event handlers bound on the tracked element ('iframe.youtube')
Remove the handler object from the internal index ($.iframeTracker.handlersList)

Otherwise, you can just try something like that (but this is pretty ugly) :
$.iframeTracker.disabled = [];
$.iframeTracker.disabled.push('iframe.youtube');

$('iframe.youtube').iframeTracker({
	blurCallback: function(){
		// Check if tracking is enabled
		if ($.iframeTracker.disabled.indexOf('iframe.youtube')) {
			return;
		}

		console.info('youtube');
	}
});

Hey, thank you for this detailed answer! I will stargaze the project to get notified when it's in. Thank you very much for your great plugin!

The untrack feature has been released (1.0.5) !
Use it like this :
$('iframe.youtube').iframeTracker(false);

Yaaaaay! I'll update my Bower dependency immediately. Thank you so much! :tada:


Closing the application disconnects the firewall

When you remove the recent app, the VPN service is disconnected, and you have to reopen the app to reactivate the firewall
I have seen that other apps do not close because they have a persistent notification that prevents accidental closing or force closing the system.

I do not see this behaviour on my test phones A40 and A54. When the app is closed the VPN continues to run as a background task.

The device I have is Xiaomi Poco F5.  I already tried several ways removing background restrictions and battery saving ways but it always closes

I confirm that on many, many phones (probably all the chinese ones and maybe even the Samsungs now) the only way to have a change of an app not getting killed, or actually to at least get relaunched when it gets killed, is to set a persistent notification (this might be harder to notice on phones with a lot of ram).

I also have the same problem with Version 64 on a Xiaomi Mi 9 Lite with Android 9, but the problem only happens after a reboot.
If you close the app from the list of recent used apps, the icon of VPN at top of screen disappears and the firewall is deactivated.
I have to open the app and re-enable it.
But if I use the phone several hours (maybe a day) after a reboot, this no longer happens: I can open and close the app without the firewall being deactivated.
It's strange.


question : activate/desactivate filter types

Hi,
I love your tool and I have a question:

when using the -W and -M options, is it possible to disable the -q, -u and -n options? I tried a -W -M -Q line, but all the options were disables, including the -W and -Q... (I want to use per the "read cutting by quality options" and disable the "quality filtering options" in the same time)

What's your command?
per read cutting by quality is disabled by default, although you specify -M and -W.
You have to set -5 to enable it on 5' end (front), or set '-3' to enable it on 3' end (tail), or set -5 -3 to enable both.

I tried something like this:
fastp -A -L -w 4 -j ffcf607a-7b70-4e16-a60b-c09197fa1601_report.json -h ffcf607a-7b70-4e16-a60b-c09197fa1601_report.html -W 20 -M 30 -Q \ -i input/ffcf607a-7b70-4e16-a60b-c09197fa1601_1.fastq.gz \ -I input/ffcf607a-7b70-4e16-a60b-c09197fa1601_2.fastq.gz \ -o output/ffcf607a-7b70-4e16-a60b-c09197fa1601_windows_W20M30_1.fastq.gz \ -O output/ffcf607a-7b70-4e16-a60b-c09197fa1601_windows_W20M30_2.fastq.gz
and nothing was filtered.
Then I removed the -Q, and some reads were filtered according to sliding window on the reads but it also enable the -q -n and -n filtering with the default values, no?

four points:
1, to enable per read cutting by quality, you have to set -5 to enable it on 5' end (front), or set -3 to enable it on 3' end (tail) . -W and -M options are only for the settings of this feature but actually not enabling them (I may add a WARNING or ERROR if -W or -M is specified but neither
-5 nor -3 is enabled).
2, this feature only cut the reads in the front or end, but will not discard them. So you saw nothing was filtered.
3, this feature has no any conflicts with quality filtering, which can be disabled by -Q.
4, this feature can make reads shorter since it cuts in front or end, so if length filter (can be disabled with -L) is enabled, then more reads may be filtered out since they may not pass the length filter.

Could you update? I am going to close this issue

Thanks for your help.


api: create new endpoint for access/users

implement bulk create grants
add grants list marshmallow schema
add a notification builder
closes https://github.com/inveniosoftware/invenio-app-rdm/issues/2593

Email preview:
Subject:

Main body:

In plain text:

May i ask for clarification "grant" is used here in terms of permissions? I know this word from another context (fundings) and that context seems not to match with the implementation.

Yes, it is used like "grant access", nothing to do with the fundings.

grant access

thank you for the clarification


Improve F# integration

Open a couple more default namespaces.
Add a couple more referenced assemblies

Adding namespaces has changed the errors returned in some of the tests, so you'll either have to update the tests or disallow namespace/type suggestions.

I'll update the tests, I like the suggestions.


⚠ Bofrost.lu has degraded performance

In 407dc92, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 969 ms

Resolved: Bofrost.lu performance has improved in d227f73 after 12 minutes.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/59020

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/59021


Release DnsRecords when failing to notify promise

Motivation:
We need to release the DnsRecords if we didnt transfer ownership to prevent leaks.
Modifications:
Release DnsRecords when trySuccess(...) returns false
Result:
Correctly release reference counted resources

Noticed this while working on something else... That said this is more of a "correct usage" fix. As we used wrapped ByteBufs  there would be no "real leak" even without this fix.


blog: number of comments cache is too aggressive

Caching is too aggressive. It should be around 1 minute at most (?). Right now it takes too long to wait to see the new value.
(it might be even that we don't reset it until the redeploy)

Making it p1 to at least check that we don't cache it infinitely. That would be a major p-0.

Looks like it's 900s (15m) right now. Let me get the github link to the exact point in comments.js

Let's test it on one of the older posts? to make sure that redeploy is not required at least

Fixed by #1250


Correct nested labels

This PR hardens our labels implementation, ensuring that it correctly handles arbitrarily nested items.
It incorporates a pre-release version of the bridge to use https://github.com/pulumi/pulumi-terraform-bridge/pull/1710. Without #1710, we see a name conflict from the bridge.
Note: expandPath should probably be in pulumi/pulumi, and I will open a PR to move it shortly. resource.PropertyPath.Reset works correctly on globed values, but resource.PropertyPath.Get and resource.PropertyPath.Set` do not.
I have been unable to reproduce https://github.com/pulumi/pulumi-gcp/issues/1387. This PR rewrites nested label application, which is where the #1387 is broken. I assume that this will fix it, but in absence of a repro the only proof will be the absence of new problems.
Fixes https://github.com/pulumi/pulumi-gcp/issues/1387

I have a pretty good idea of what is going wrong here. I'm working on a fix now.

What was going wrong in https://github.com/pulumi/pulumi-gcp/issues/1387?
Can you please summarize the issue which must be happening?

I have a pretty good idea of what is going wrong here. I'm working on a fix now.

What was going wrong in #1387?
Can you please summarize the issue which must be happening?

In #1387, we were inserting "pulumiLabels.%" as a key into the labels field, which errored as . and % are invalid labels keys. The process wasn't deterministic, so it would work most of the time,.
There are a couple of issues here:

Since tfbridge.SchemaPathToPropertyPath was being handed the wrong contexts, it could generate conflicting path names. I believe that this was the source of the nondeterminism.

Due to https://github.com/pulumi/pulumi-terraform-bridge/pull/1710, we weren't actually translating paths correctly at runtime. That is how the .% remained in the path name. We didn't notice, since (1) meant that we rarely inserted labels for nested types.

(1) and (2) are responsible for #1387. When I fixed those, I noticed that we didn't correctly handle glob based paths, which is where the glob handling comes in.

Thanks for the explanation, looks sensible to me


Ceph-calamari docker image

I'd be good to get calamari working inside docker.
I tried to once get it working by following all the instructions but got stuck in the ruby frameworks and couldn't get it going.

Not sure the benefit we get from Calamari at the moment but it's probably a good thing to add.
IMHO ceph-rest-api does the job just fine.

ceph-rest-api is deprecated and we want to remove it eventually :)

Yes I've heard that but the direction of Calamari is not really clear at the moment. :)
But right we shouldn't bother doing anything with ceph-rest-api then.

@ktdreyer - I didn't realise ceph-rest-api has been deprecated.  What replaces it?

@hookenz Calamari API.


init_by_shape does not work on pytrees

init_by_shape only supports a list of arrays as lazy arguments.
Instead it would be better to support arbitrary pytrees.
The easiest way to support this is by using the ShapeDtypeStruct in Jax similar to jax.eval_shape.

Quick question: by arbitrary pytrees do you mean "support a list of pytrees as lazy arguments"?I.e., update _parse_spec flatten, turn into ShapeDtypeStruct, unflatten

Marking as on-hold as this is likely to become much easier once the FLIPs land.

We no longer have `init_by_shape1 in Linen, so I'll close this issue.


Cross compiling

It would be nice to provide guides for cross compiling.
None of the core writers of this book has experience here.
Contributions would be very welcome.
The most important directions are

linux aarch64 to linux arm64
linux aarch64 to windows aarch64

I haven't worked with it, but this Docker container might help: https://github.com/MGlolenstine/gtk4-cross

I personally find the directions
linux x86_64 -> Window x86_64
&
linux x86_64 -> Macos aarch64
really importaant, to allow distrebuting a gtkrs app build on linux to people on modern macbooks and people on modern windows notebooks as well.


New-ChartLegend is not showing labels on Bar/Pie Chart

I have the following code that receives a PSObject with three fields.
What I get is a pie chart that obeys colors either with the theme or directly assigned on the legend.  But i get no labels on the legend on the right.
Dashboard -Name 'Bounce Chart' -FilePath C:\Scripts\DashboardChartsBar00.html {
    TabOptions -slimtabs
    Tab -Name 'Bounce Chart by Type' {
        Panel {
            Chart {
                ChartTheme -Palette palette6
                ChartLegend  -FontSize 14 -FontFamily 'Arial'
                foreach ($Bounce in $Bounces) {
                    ChartPie -Name $Bounce.BounceTypeName -Value $Bounce.Total
                }
            }
        }
        Section -Invisible {
            Table -DataTable $BounceList -HideFooter {
                Table -title 'Email Address'
                TableHeader -names 'EmailAddress' -alignment Left
                TableHeader -names 'Bounce Type','Date' -alignment Center
                TableContent -ColumnIndex 2,3 -alignment center
            }
        }
    }
} -Show

I have tried adding -names either by $Bounces.BounceTypeName or by @('Hard Bounce','Soft Bounce') or not even including it.
I am not sure what i am missing as I get the color dots of the legend and no text.  Nor can I get labels on the pie other than %.

$Bounces = @(
    [PSCustomObject] @{ BounceTypeName = 'Area1'; Total = 5 }
    [PSCustomObject] @{ BounceTypeName = 'Area2'; Total = 13 }
    [PSCustomObject] @{ BounceTypeName = 'Area3'; Total = 18 }
    [PSCustomObject] @{ BounceTypeName = 'Area4'; Total = 1 }
)

Dashboard -Name 'Bounce Chart' -FilePath C:\Scripts\DashboardChartsBar00.html {
    TabOptions -slimtabs
    Tab -Name 'Bounce Chart by Type' {
        Panel {
            Chart {
                ChartTheme -Palette palette6
                ChartLegend  -FontSize 14 -FontFamily 'Arial'
                foreach ($Bounce in $Bounces) {
                    ChartPie -Name $Bounce.BounceTypeName -Value $Bounce.Total
                }
            }
        }
        Section -Invisible {
            Table -DataTable $Bounces -HideFooter {
                Table -title 'Email Address'
                TableHeader -names 'EmailAddress' -alignment Left
                TableHeader -names 'Bounce Type','Date' -alignment Center
                TableContent -ColumnIndex 2,3 -alignment center
            }
        }
    }
} -Show

That's what I get

Seems to work?


Tcd wip

Feedback so far:

We need the VM image to cache postgres:15-alpine and testcontainers/ryuk:0.5.1.
(TBC) I think we should show a Testcontainers ASCII logo instead of AtomicJar, as people might not know how it relates.
On my machine, running ./mvnw test (in the terminal only) gives me the following warning:

[WARNING]
[WARNING] Some problems were encountered while building the effective model for cloud.testcontainers:java-example:jar:1.0-SNAPSHOT
[WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must be unique: org.testcontainers:postgresql:jar -> duplicate declaration of version (?) @ line 46, column 21
[WARNING]
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
[WARNING]

(nice-to-have) For the content of the database, let's include: https://testcontainers.com/getting-started/, and Getting started with Testcontainers for %LANG guides, with the name and URL in separate cols.
(nit) This INFO log line is not actionable:

12:23:31.497 [main] INFO  org.testcontainers.utility.ImageNameSubstitutor - Image name substitution will be performed by: DefaultImageNameSubstitutor (composite of 'ConfigurationFileImageNameSubstitutor' and 'PrefixingImageNameSubstitutor')

@mtremsal The Maven warning don't appear for me.
12:23:31.497 [main] INFO  org.testcontainers.utility.ImageNameSubstitutor - Image name substitution will be performed by: DefaultImageNameSubstitutor (composite of 'ConfigurationFileImageNameSubstitutor' and 'PrefixingImageNameSubstitutor')

This is expected INFO logging from tc-java, there is no action associated with this..


Disable one more test when contrib ops are disabled.

One more test failed as per https://aiinfra.visualstudio.com/Lotus/_build/results?buildId=79109. Not sure why this didn't fail earlier.

Maybe I know. There was a known issue in onnx_test_runner. It was introduced in Pranav's PR #639, that it may access uninitialized memory when retrieving test config. It should get fixed in #751

@snnn thanks for the the links above. The changes from #639 should be available in rel-0.3.1 branch. The failing tests seem to be transient. Merging this PR into  rel-0.3.1 is building successfully.

I don't think this PR is good. Because, you just exclude one test failure you have seen, but you may get more random test failures in future runs.

It'll be fixed in the master branch.

If you have more changes in rel-0.3.1, you may hit this issue again.

rel-0.3.1 is sealed now.


Update IPv6 support in M365 documentation with latest information

This page is more than 2 years old, so likely needs to be updated to reflect changes in how M365 supports IPv6. Customers are asking for detail, but don't trust a 2 year old document.

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

ID: 41916ef7-635a-80f9-914a-207040a3829e
Version Independent ID: 1f372ebc-774b-a1f5-5e43-73b42f74b198
Content: IPv6 support in Office 365 services - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/ipv6-support.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@kelleyvice-msft  Please add your comment and help us in resolving this issue. Thanks.

Engineering initiated updates are pending.