Add basic "dvc add" workflow to DVC tracked panel

For simpler onboarding (and tutorial) it would be handy to have a button like this.
It can cover only new files (no complicated scenarios like adding files into a directory and such).
+ Icon and a "wizard" is enough
It should cover a scenario when data is tracked by Git (it's needed to the onboarding tutorial). If this is too complicated we can postpone the whole feature since it won't move the needle at the moment most likely.

We already have the ability to add untracked files via the SCM panel:
https://github.com/iterative/vscode-dvc/assets/37993418/e155b812-a04e-4d4a-864b-ab61aa0dc4aa
and we do have "Add Data" via the tracked tree:
https://github.com/iterative/vscode-dvc/assets/37993418/4042158e-fe7d-4d60-affe-fe3bfa92f5b3
It does seem like a couple of days work to add this to the file explorer context menu as we will have to run some git commands up front in order to be able to run dvc add for files tracked by Git:
dvc add .github
ERROR:  output '.github' is already tracked by SCM (e.g. Git).

    You can remove it from Git, then add to DVC.

        To stop tracking from Git:

            git rm -r --cached '.github'

            git commit -m "stop tracking .github"

We also know how unstable things start getting when we chain commands together.
Other things to consider:

show message if resource is already tracked by DVC.
do not try to git rm files that are not tracked by Git.
multi-project workspace with subprojects, need to work out which project to add to.


add keepalives attribute

for JIT clients(https://sensuapp.org/docs/latest/clients#jit-clients)

:+1:

Thanks for the PR @negachov. This provider is not meant to configure JIT clients, only the node's client.json, so this isn't the appropriate place to include keepalives as that config key isn't relevant for the client definition.
This does make me think we could add a provider to help with JIT clients, though. Tracking that idea in #417


[GPU] Add NVTX to XLA

This is CUPTI and xprof compatible. So the NVTX marker are human readable, but also intended to be read by machine.
Doesn't work with TF without XLA.
Here is a screen shot:

@cliveverghese @trisolaran

I added a commit to this PR as the CI failed. There was a missing file.

This PR broke the build on CPU. I'll fix it locally and update it. Not ready to be approved.

I pushed a new commit that should resolve the build on CPU.
At least, it worked locally. You only need to review the last commit.

@trisolaran The ARM CI build failed:
https://source.cloud.google.com/results/invocations/d49adac5-f0b4-4337-b09e-52508c5d0136/log
The error:
./tensorflow/tsl/profiler/lib/nvtx_utils.h:24:10: fatal error: nvtx3/nvToolsExt.h: No such file or directory
   24 | #include "nvtx3/nvToolsExt.h"
      |          ^~~~~~~~~~~~~~~~~~~~

Any idea why NCCL_ENABLED is defined, on ARB but the dependency isn't available?

I updated this PR with another commit to fix the build on some platform.
I mixed NCCL and NVTX macro.

ARM and AMD CI failure shouldn't be related to this PR. They have the same error ARM one:
FATAL: bazel crashed due to an internal error. Printing stack trace:
java.lang.RuntimeException: Unrecoverable error while evaluating node 'File:[/tmpfs/bazel_output/_bazel_root/eab0d61a99b6696edb3d2aff87b585e8/external/pybind11_abseil[source]]pybind11_abseil/LICENSE'

@trisolaran @akuegel Can one of you approve again this PR?
It was already approved. I only fixed some builds issues after the approval.

It is merged now. Thanks!
Out of curiosity, I looked at the changes you did and understand them except 2:
+++ b/tensorflow/compiler/xla/service/gpu/BUILD
@@@ -368,6 -371,7 +368,8 @@@ cc_library
          "//tensorflow/tsl/platform:logging",
          "//tensorflow/tsl/platform:human_readable_json",
          "//tensorflow/tsl/platform:status",
+         "//tensorflow/tsl/profiler/lib:nvtx_utils",
++        "//tensorflow/tsl/protobuf:dnn_proto_cc_impl",

diff --cc tensorflow/compiler/xla/service/gpu/ir_emitter_unnested.cc
index 6fb55dd78c7,6be9f77890a..b32e42524cc
--- a/tensorflow/compiler/xla/service/gpu/ir_emitter_unnested.cc
+++ b/tensorflow/compiler/xla/service/gpu/ir_emitter_unnested.cc
@@@ -130,6 -130,7 +130,8 @@@ limitations under the License
  #include "tensorflow/tsl/platform/errors.h"
  #include "tensorflow/tsl/platform/human_readable_json.h"
  #include "tensorflow/tsl/platform/logging.h"
+ #include "tensorflow/tsl/profiler/lib/nvtx_utils.h"
++#include "tensorflow/tsl/protobuf/dnn.pb.h"

Why you needed to add dnn?

This was suggested by a ClangTidy check. ClangTidy is a blocker for submitting changes. I didn't check how it derived that this include is needed, but this check usually can be trusted. I just checked it now, the usage is pre-existing:
stream_executor::dnn::ActivationMode is used in this file.


StructureProvider interface

This adds a StructureProvider interface in core. The Structure class implements this interface. The StructureRenderer now only requires an object of type StructureProvider.
This makes it easier to provide blocks from a different source for rendering to the Renderer. Without this interface any class extending Structure includes blocks array which might not be needed.

Thanks for the contribution! This is indeed a good idea.


[8.2.x] ISPN-7652 Align River marshaller version

JBoss Marshalling's River jar version needs to be aligned with the
JBoss Marshalling API jar.

https://issues.jboss.org/browse/ISPN-7652

This should fix most of the test failures on 8.2.x

merged, all failures in the server testsuite fixed.


Only support new import syntax

Is your feature request related to a problem? Please describe.
Playground v2 should not have to deal with address translation. Currently on v1, there is an issue in address translation where all hexadecimal values are translated (regardless of being an import or not).
Solution
The best solution for this is to only use the new import syntax in v2 and have no address translation at all.

The frontend should show accounts as names rather than addresses
Maintain a flow.json file to resolve imports rather than translating
Use flowkit for supporting the new import syntax

This will also allow for a smoother transition from Playground to local development as the flow.json file would also be exported and so imports will work the same.
Fixes this issue for v2: https://github.com/onflow/flow-playground-api/issues/201

Problem is; it is not only imports. getAccount(0x2) still needs translation, likewise for example if script is getting an Address parameter.

@bluesign we can still show addresses for each account but they won't be as important (beside few edge cases which you mention). This is not just about translation but also about making the playground experience more similar to local.


Assess adding TRI data from RSEI model data

Source - EPA
Add to: Legacy pollution
Pull in the air version of the data.
Here's one set of data https://gaftp.epa.gov/rsei/Census_XWalks
and here's another that includes MDB files https://www.epa.gov/sites/production/files/2015-07/rsei_2010_census_table_0.zip.
This is probably the best way to read in MDB on a mac: https://stackoverflow.com/a/42610107.
Data overview at https://www.epa.gov/rsei/ways-get-rsei-results#microdata.

Sindhu has requested this data and we are awaiting it.

Information from Mitch suggests this analysis is using "wrong" information. To redo, using files from Mitch et all

suggest not to include.


[QUESTION]How to use kerberos authentication in JDBC on Windows

Question
I am trying to use Kerberos authentication in JDBC on Windows:
Java Version: Java 17
mssql-jdbc version: com.microsoft.sqlserver:mssql-jdbc:12.6.1.jre11
I can login successfully in Microsoft SQL Server Management Studio with Windows Authentication, but can not access database in my Java application, here is my java code:
public static void main(String[] args) {
        try (
                Connection connection =
DriverManager.getConnection("jdbc:sqlserver://localhost:1433;integratedSecurity=true;authenticationScheme=JavaKerberos;encrypt=true;trustServerCertificate=true;", "", "");
                Statement statement = connection.createStatement();
                ResultSet rs = statement.executeQuery("select 'abc'")
        ){
            if(rs.next()){
                System.out.println(rs.getString(1));
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

After I run my application on this Windows machine, I got below error:
**java -Dsun.security.krb5.debug=true -jar kerberos-1.0-SNAPSHOT.jar**

Java config name: null
LSA: Found Ticket
LSA: Made NewWeakGlobalRef
LSA: Found PrincipalName
LSA: Made NewWeakGlobalRef
LSA: Found EncryptionKey
LSA: Made NewWeakGlobalRef
LSA: Found TicketFlags
LSA: Made NewWeakGlobalRef
LSA: Found KerberosTime
LSA: Made NewWeakGlobalRef
LSA: Found String
LSA: Made NewWeakGlobalRef
LSA: Found Ticket constructor
LSA: Found PrincipalName constructor
LSA: Found EncryptionKey constructor
LSA: Found TicketFlags constructor
LSA: Found KerberosTime constructor
LSA: Finished OnLoad processing
Native config name: C:\Windows\krb5.ini
Loaded from native config
>>>KinitOptions cache name is C:\Users\<***>\<***>\<***>
>> Acquire default native Credentials
Using builtin default etypes for default_tkt_enctypes
default etypes for default_tkt_enctypes: 18 17 20 19.
LSA: Found KrbCreds constructor
LSA: Got handle to Kerberos package
LSA: Response size is 1843
LSA: TICKET SessionKey KeyType is 18
LSA: Valid etype found: 18
LSA: Principal domain is ***
LSA: Name type is 1
LSA: Name count is 1
LSA: Principal domain is ***
LSA: Name type is 2
LSA: Name count is 2
LSA: Session key all zero. Stop.
>>> Found no TGT's in native ccache
com.microsoft.sqlserver.jdbc.SQLServerException: Kerberos Login failed: Integrated authentication failed. ClientConnectionId:c6f18883-621c-4647-9369-adb0578b406a due to javax.security.auth.login.LoginException (Cannot get any of properties: [user, USER] from con properties not available to garner  authentication information  from the user)
        at com.microsoft.sqlserver.jdbc.KerbAuthentication.initAuthInit(KerbAuthentication.java:130)
        at com.microsoft.sqlserver.jdbc.KerbAuthentication.generateClientContext(KerbAuthentication.java:238)
        at com.microsoft.sqlserver.jdbc.SQLServerConnection.sendLogon(SQLServerConnection.java:6587)
        at com.microsoft.sqlserver.jdbc.SQLServerConnection.logon(SQLServerConnection.java:5402)
        at com.microsoft.sqlserver.jdbc.SQLServerConnection$LogonCommand.doExecute(SQLServerConnection.java:5334)
        at com.microsoft.sqlserver.jdbc.TDSCommand.execute(IOBuffer.java:7739)
        at com.microsoft.sqlserver.jdbc.SQLServerConnection.executeCommand(SQLServerConnection.java:4384)
        at com.microsoft.sqlserver.jdbc.SQLServerConnection.connectHelper(SQLServerConnection.java:3823)
        at com.microsoft.sqlserver.jdbc.SQLServerConnection.login(SQLServerConnection.java:3348)
        at com.microsoft.sqlserver.jdbc.SQLServerConnection.connectInternal(SQLServerConnection.java:3179)
        at com.microsoft.sqlserver.jdbc.SQLServerConnection.connect(SQLServerConnection.java:1953)
        at com.microsoft.sqlserver.jdbc.SQLServerDriver.connect(SQLServerDriver.java:1263)
        at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:681)
        at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:229)
        at com.mssql.Kerberos.main(Kerberos.java:8)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:568)
        at org.springframework.boot.loader.MainMethodRunner.run(MainMethodRunner.java:49)
        at org.springframework.boot.loader.Launcher.launch(Launcher.java:108)
        at org.springframework.boot.loader.Launcher.launch(Launcher.java:58)
        at org.springframework.boot.loader.JarLauncher.main(JarLauncher.java:65)
Caused by: javax.security.auth.login.LoginException: Cannot get any of properties: [user, USER] from con properties not available to garner  authentication information  from the user
        at jdk.security.auth/com.sun.security.auth.module.Krb5LoginModule.promptForName(Krb5LoginModule.java:853)
        at jdk.security.auth/com.sun.security.auth.module.Krb5LoginModule.attemptAuthentication(Krb5LoginModule.java:689)
        at jdk.security.auth/com.sun.security.auth.module.Krb5LoginModule.login(Krb5LoginModule.java:597)
        at java.base/javax.security.auth.login.LoginContext.invoke(LoginContext.java:755)
        at java.base/javax.security.auth.login.LoginContext$4.run(LoginContext.java:679)
        at java.base/javax.security.auth.login.LoginContext$4.run(LoginContext.java:677)
        at java.base/java.security.AccessController.doPrivileged(AccessController.java:712)
        at java.base/javax.security.auth.login.LoginContext.invokePriv(LoginContext.java:677)
        at java.base/javax.security.auth.login.LoginContext.login(LoginContext.java:587)
        at com.microsoft.sqlserver.jdbc.KerbAuthentication.initAuthInit(KerbAuthentication.java:107)
        ... 22 more

Is there any configuration I am missing? Thanks,

@Gong-Allen
Just some basic initial kerberos troubleshooting questions first.

Did you get a successful kinit to get a ticket on the client machine on the domain?

Was a SPN properly registered on the server? You can check the error log text file in SQL Server for this. If you see The SQL Server Network Interface library successfully registered the Service Principal Name (SPN) [ MSSQLSvc/MySqlServerHostName:1433 ] for the SQL Server service then it's working. You can find the error log text file on SSMS in the "Object Explorer > Management > SQL Server Logs".

If the above 2 points are working for you, then maybe for some reason it can't find the ticket cache. Then you can try explicitly setting the path to the ticket using the KRB5CCNAME environment variable.

Also, incase you haven't seen this yet. We have a doc for using the driver with kerberos found here:
https://learn.microsoft.com/en-us/sql/connect/jdbc/using-kerberos-integrated-authentication-to-connect-to-sql-server?view=sql-server-ver16

Hi @tkyc ,
Thanks for your information, after I call the kinit.exe under the bin folder of my JRE, I can connection to SQL Server with Kerberos authentication successfully, but here comes a question for me: is the manual calling  kinit a mandatory step? My application would be a windows backend service and there is no user interaction, in this case is it possible to use kerberos authentication?

@Gong-Allen

is the manual calling kinit a mandatory step? My application would be a windows backend service and there is no user interaction, in this case is it possible to use kerberos authentication?

I'm not an expert on kerberos and and vaguely recall this (and I can't say for sure if this is appropriate, so you'll need to do more research), but I believe you'll need a kerberos keytab file which can be used to get a ticket without entering a password. Then you can schedule some sort of cron to run the kinit with the keytab file to renew.
I'm sure there are better and more appropriate ways to do this, but this was my immediate thought.

Hi @tkyc ,
Would you please tell me how to use the kerberos keytab file? Is there any document for it? Thanks.

@Gong-Allen
Personally I don't recall exactly how, but you can refer to this relevant doc here:
https://learn.microsoft.com/en-us/windows-server/administration/windows-commands/ktpass

hi @Gong-Allen
As @tkyc mentioned we are not experts in Kerberos nor can we provide any support on how to setup Kerberos in your environment. We can only offer you support on issues on our driver only.  As @tkyc mentioned you will have to verify that your Kerberos environment is setup correctly before you can successfully use the driver.

Hi @lilgreenbird
Thanks for your clarification, I will close this question.


"Scăești" , "Valea lui Pătru" are listed in the drop-down menu

Description:
The villages are listed in the drop-down menu alongside its corresponding county and city, according to the law
Preconditions:  the  website is up and runing. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Scăești"

Expected results
The location appears in the drop-down menu as "Sat Scăești, SĂLAJ (Scăești)"

Step 2
Write in the search bar "Valea lui Pătru"

Expected results
The location appears in the drop-down menu as "Sat Valea lui Pătru, SĂLAJ (Scăești)"

ENVIRONMENT: ui.orase
URL: https://peviitor-ro.github.io/ui.orase/
DEVICE: LAPTOP
OS: Windows
STEPS TO REPRODUCE:
Open url in browser

Write in the search bar "Scăești"

Write in the search bar "Valea lui Pătru"

EXPECTED RESULTS:
1.The location appears in the drop-down menu as "Sat Scăești, SĂLAJ (Scăești)"
2.The location appears in the drop-down menu as "Sat Valea lui Pătru, SĂLAJ (Scăești)"
ACTUAL RESULTS:
1."Sat Scăești, SĂLAJ (Scăești)" doesn't exist in the drip-down menu
2."Sat Valea lui Pătru, SĂLAJ (Scăești)" doesn't exist in the drip-down menu


excludeFilter doesn't work

I'd like to have an ability to exclude some protofiles from code generation. I tried to add:
excludeFilter in unmanagedSources := "filename.proto"

to build.sbt, but it is ignored.

The next version of sbt-scalapb (0.6) invokes sbt-protobuf which picks up the proto files. This can be fixed once https://github.com/sbt/sbt-protobuf/pull/29 is merged.

Looks like changes in sbt-protobuf are merged now, and new version (0.5.0) is released.


[!] No more data! Scraping will stop now.

Command Ran

twint -s "AUD/USD" --since 2016-01-01 --json -o AUDUSD.json

Description of Issue

Hi, I managed to get around 296946 tweets from the above command from Jan 2021 till August 2019 then got the prompt [!] No more data! Scraping will stop now. Surely there are more tweets beyond this point of date as well.  Does anyone know what's causing the script to stop?

Environment Details

Windows | Command Prompt

If it says no more data then there won't be more than what you already fetched.

If it says no more data then there won't be more than what you already fetched.

Hey, thanks for getting back.
I had also ran the command twint -s "AUD/USD" --year 2019 to fetch all tweets from 2018 and it scrapped more data which means there were more tweets. I scrapped all the way heading back to 2015.  I think the program stops when it's not able to find tweets around that month perhaps? Not sure whats happening..

Hey, thanks for getting back.
I had also ran the command twint -s "AUD/USD" --year 2019 to fetch all tweets from 2018 and it scrapped more data which means there were more tweets. I scrapped all the way heading back to 2015.  I think the program stops when it's not able to find tweets around that month perhaps? Not sure whats happening..

Hi, yeah that could happen, maybe you can write a script with predefined month and year. You can fetch the data using for loop. If there is any month which doesn't contain the data you can just raise an exception/or just continue with the loop.

Hi, yeah that could happen, maybe you can write a script with predefined month and year. You can fetch the data using for loop. If there is any month which doesn't contain the data you can just raise an exception/or just continue with the loop.

Yeahh, that does sound like a possibility to try to get the script working.  Thanks!

Yeahh, that does sound like a possibility to try to get the script working.  Thanks!

Hi,
Can I ask again the same question. But for me it stops and says no more data, then I put it in a loop like this :
`st = datetime.strptime('2021-09-13','%Y-%m-%d')
for i in range(356):
c= twint.Config()
c.Search=" كاتشوف OR كيضحك OR زوينة OR كتبكي OR كولشي OR  مزيان OR داكشي OR كيشوف OR كتشوف OR واخا OR كيزيدو OR دابا OR ديال OR الجلاخة OR تبوگيصة OR تخربيق"
stdt=st-timedelta(days=i+1)
c.Until=stdt.strftime('%Y-%m-%d')
c.Store_csv= True
#c.Hide_output= True
c.Filter_retweets= True
c.Output="C:\dabaList.csv"
c.Resume="C:\Resume_dabaList.csv"
twint.run.Search(c)`

It scapes but with a lot of "No more data! Scraping will stop now" messages.
What I did is going to Twitter advanced research and there is existing tweets on the days twint says no more data.
Can you help with this please?


Assertion message improvements when using .not

Some of the messages for matchers are misleading when using .not.
For example, whether you do expect(this.subject.find(...)).toBePresent() or expect(this.subject.find(...)).not.toBePresent(), upon failure, the error message is "Expected contents to not be empty, but it is".
I think it would be more useful if the message changed depending on whether the assertion passes. For example, instead of this:
      compare(enzymeWrapper:Object) : Object {
        return {
          pass: enzymeWrapper.length !== 0,
          message: 'Expected contents to not be empty, but it is',
        };
      },

how about:
      compare(enzymeWrapper:Object) : Object {
        var passes = enzymeWrapper.length !== 0;
        return {
          pass: passes,
          message: passes ? 'Expected contents to not be empty, but it is' : 'Expected contents to be empty, but it wasnt',
        };
      },

I think a similar conditional could be applied to some of the other matchers as well.

Thanks for the post! This was recently brought up in #31 and has a PR up for review in #34 that will be merged soon. I'm going to close this as duplicate. Be on the look out for a new version soon

Wow my reading comprehension skills are limited. thanks

This is now released in 1.2.0 🎉


error while loading model

Hi Yue Linn Chong,
Thanks for your reply on the previous issue.
I am running "generate_mit_latents.py" file to get the latent variable w and the image that the StyleGAN2 generates with said w.
For this I am using saved model provided along with repo as "stylegan2_weights.pt".
But I am getting error as follows
command:  python generate_mit_latents.py --size 256 --sample 1 --pics 500 --ckpt /home/workspace/stylegan2_weights.pt --out_path ./styleinversion_trainset
error:

I think Reason for error may be:
model architecture changes, model compatibility issue, model partial loading and to solve this issue we need access to model code.
Also, For training F(EN) encoder, where can I find the "train.py" file.

Hi Rohit,
thanks for the detailed issue.
The issue with the script generate_mit_latents.py should not happen. I will try to replicate the problem and fix it next week.
If you are looking for the training script for the weights of stylegan2_weights.pt, the training script is in stylegan2-pytorch/train.py.
Similarly, the training for  F(EN) encoder is in the pixel2style2pixel/scripts/train.py but I checked this is not the latest version for the training of F(EN). Sorry about that, I will upload the correct train script for the F(EN) next week as well.

Hi Rohit,
I notice you are using the weights I uploaded. This model outputs image size of 512 so you should run with the correct image size. For example :
python generate_mit_latents.py --size 512 --sample 1 --pics 10 --ckpt stylegan2_weights.pt --out_ path ../artifacts


trident error, no animation, no damage and console error

Spark link
https://spark.lucko.me/tSzaxg8mtc
Expected behavior
the trident will deal damage and an animation will play
Observed/Actual behavior
when using a trident on a dragun there is no animation and damage to entities is not dealt and an error is written to the console
Steps/models to reproduce

[12:41:36 ERROR]: Failed to handle packet net.minecraft.network.protocol.game.ServerboundPlayerActionPacket@4634dca8, suppressing error
java.lang.IllegalArgumentException: Bound must be positive
at net.minecraft.world.level.levelgen.BitRandomSource.nextInt(BitRandomSource.java:19) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.world.entity.Entity$RandomRandomSource.nextInt(Entity.java:236) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.Util.getRandom(Util.java:688) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.world.item.TridentItem.releaseUsing(TridentItem.java:141) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.world.item.ItemStack.releaseUsing(ItemStack.java:1048) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.world.entity.LivingEntity.releaseUsingItem(LivingEntity.java:4402) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.network.ServerGamePacketListenerImpl.handlePlayerAction(ServerGamePacketListenerImpl.java:1932) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.network.protocol.game.ServerboundPlayerActionPacket.handle(ServerboundPlayerActionPacket.java:51) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.network.protocol.game.ServerboundPlayerActionPacket.handle(ServerboundPlayerActionPacket.java:20) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.network.protocol.PacketUtils.lambda$ensureRunningOnSameThread$0(PacketUtils.java:55) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.TickTask.run(TickTask.java:18) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.util.thread.BlockableEventLoop.doRunTask(BlockableEventLoop.java:154) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.util.thread.ReentrantBlockableEventLoop.doRunTask(ReentrantBlockableEventLoop.java:24) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:1619) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:207) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.util.thread.BlockableEventLoop.pollTask(BlockableEventLoop.java:128) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.MinecraftServer.pollTaskInternal(MinecraftServer.java:1596) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.MinecraftServer.pollTask(MinecraftServer.java:1589) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.util.thread.BlockableEventLoop.managedBlock(BlockableEventLoop.java:138) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.MinecraftServer.managedBlock(MinecraftServer.java:1540) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.MinecraftServer.waitUntilNextTick(MinecraftServer.java:1550) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1386) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:342) ~[purpur-1.21.3.jar:1.21.3-2340-db09358]
at java.base/java.lang.Thread.run(Thread.java:1583) ~[?:?]
Purpur version
[12:42:48 INFO]: Current Purpur Version: 1.21.3-2340-db09358 (MC: 1.21.3)*

You are running the latest version
Previous: 1.21.3-2337-ca668ab (MC: 1.21.3)

Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] I ticked all the boxes without actually reading them
[X] My version of Minecraft is supported by Purpur.

Other
No response

Hi! Could you try to reproduce on Paper and one time without any plugins?
Consider joining our Discord!  https://purpurmc.org/discord
Also:

[x] I ticked all the boxes without actually reading them

Couldn't reproduce in build 2340

I tried it on a completely clean build, everything is the same
https://github.com/user-attachments/assets/230a9f42-2ea9-4379-90af-af25f83e31c9

Oh, sorry! I was thinking that you meant "dragon"

I apologize for the video quality, at 0:17 you can see the kernel version

Could reproduce on Purpur now.
Upstream (Paper) is not affected.


Propsal: Drop "Variable x should be longer than 3 characters" warning

For example, popular CFML MVC framework fw/1 uses the variable rc (short for Request Context) heavily.
If the intention is to stop people from using single letter variables for iterators, e.g. in for (var i = 0; then I think I could get onboard with that... I would just say it should be a minimum of two, not three. If that wasn't the intention of the rule, I'd be curious to hear what the intention was.
Here's a list of all of the places where variable name lengths are being checked.
https://github.com/cflint/CFLint/search?utf8=✓&q=longer+than+characters

This like all rules this is configurable, you can exclude the rule altogether or you can set the minimum number of characters.

I'm using CFLint through SublimeLinter. I guess I'll get in touch with them to find out how to configure it. Thanks.

I've not used SublimeLinter can you pass any parameters through to the jar?

I've not used SublimeLinter can you pass any parameters through to the jar?

Guess we'll find out. :)

So yes, I can include a cflintrc.xml file in my project to set rules, but they don't document the supported XML.
See you have a wiki page with some samples on it, but one thing that's not represented there is setting a value like minimum or maximum variable length.

Include the rule, copied from cflint.definition.json
to .cflint        (json form of the config file).
put it in the "rule" array.
 {
     "rule": [
     {
       "name": "VariableNameChecker",
       "className": "VariableNameChecker",
       "message": [
         {
           "code": "VAR_INVALID_NAME",
           "severity": "INFO"
         },
         {
           "code": "VAR_ALLCAPS_NAME",
           "severity": "INFO"
         },
         {
           "code": "SCOPE_ALLCAPS_NAME",
           "severity": "INFO"
         },
         {
           "code": "VAR_TOO_SHORT",
           "severity": "INFO",
           "messageText": "Variable ${variable} should be longer than ${MinLength} characters."
         },
         {
           "code": "VAR_TOO_LONG",
           "severity": "INFO"
         },
         {
           "code": "VAR_TOO_WORDY",
           "severity": "INFO"
         },
         {
           "code": "VAR_IS_TEMPORARY",
           "severity": "INFO"
         },
         {
           "code": "VAR_HAS_PREFIX_OR_POSTFIX",
           "severity": "INFO"
         }
       ],
       "parameter": [
         {
           "name": "MinLength",
           "value": "4"
         },
         {
           "name": "MaxLength",
           "value": "20"
         },
         {
           "name": "MaxWords",
           "value": "4"
         }
       ]
     }
   ]
 }

As easy as it is for users to change this, I'm in favor of having the default min be 2 characters, as I think there are too many common 2 char things like "rc", or valid things like "id", and thus a lot of folk will want to edit/disable this rule, versus being able to use the default.

closing for now.

Your up
Late
Sent from my iPhone

On Jun 17, 2016, at 10:15 PM, ryaneberly<EMAIL_ADDRESS>wrote:
closing for now.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[Docs] withCheckedThrowingContinuation body fix

withCheckedThrowingContinuation's body says that it takes an UnsafeContinuation... seems like a copy/paste error.

I hope you won't mind this ping, seeing as it's been a while and it seems nobody noticed this - it's a comment change... as I understand it, almost zero chance of breaking code.
@airspeedswift

Sorry again for tagging, but seeing similar doc fixes appearing, I thought I'd try again with @ktoso

Indeed a copy paste error I think -- thank you!
Feel free to ping me or @amartini51 about docs issues, we try to be pretty responsive 👍

@swift-ci please smoke test and merge

@sjavora Thanks for the fix!


Where is nuscenes_kmeans900.npy?

I can't find it. Thank you

https://github.com/linxuewu/Sparse4D/releases/download/v0.0/nuscenes_kmeans900.npy

All links provided for downloading files are here. https://github.com/linxuewu/Sparse4D/releases/tag/v0.0

All links provided for downloading files are here. https://github.com/linxuewu/Sparse4D/releases/tag/v0.0

OK, thank you so much for the excellent work you do!

how to generate nuscenes_kmeans900.npy? Is there any python script?

how to generate nuscenes_kmeans900.npy? Is there any python script?

We do not provide the script for anchor kmeans. You can use sklearn.cluster.KMeans to cluster the center points of the gt boxes, then set w h l yaw vx vy vz  to fixed values, refer to the anchor file

Hi, super great work, and thanks for the explanation! I still have a question. w h l yaw vx vy vz is set to fixed values, but objects with different classes have different values of those properties. Can you help to give more insights? thanks


[Typings] Allow undefined apiKeyOrConfig

apiKeyOrConfig === undefined is being handled at https://github.com/bugsnag/bugsnag-react-native/blob/master/src/Bugsnag.js#L21

Thanks for opening a PR @danilobuerger - what would the use-case be for passing undefined into the constructor?

You shouldn't pass undefined in here so the types are correct. I don't think the fact that we cope with undefined being passed in when you aren't using a type system imply that the types should also allow undefined.

@snmaynard so why does the code explicitly handle this case (as I already linked):
if (typeof apiKeyOrConfig === 'string' || typeof apiKeyOrConfig === 'undefined') {
  this.config = new Configuration(apiKeyOrConfig)
}

If I shouldn't pass undefined why the explicit check for it here? Why allow it then and create a working Configuration?

Also why handle it in the native module (undefined in js is null in java):
          if (apiKey != null) {
              client = Bugsnag.init(this.reactContext, apiKey);
          } else {
              client = Bugsnag.init(this.reactContext);
          }

There are a lot more samples I could post here that show that a undefined api key is actually properly being handled. And if that isn't enough, here the use case:

Your Integration API Key, available on your Bugsnag dashboard under “Settings”. If not set, it will be loaded from your AndroidManifest.xml or Info.plist if available on Android and iOS respectively.

By passing undefined here (which again even the Docs say is allowed), the api key will be loaded from the manifest of plist.


Unresolved reference: cpgLanguageCxx

Hi,
when trying to build cpg-neo4j, I get the following error message:
 ../gradlew clean
Type-safe project accessors is an incubating feature.

> Configure project :
Java frontend is enabled
C/C++ frontend is disabled
Go frontend is disabled
Python frontend is disabled
LLVM frontend is enabled
TypeScript frontend is disabled

> Configure project :cpg-analysis
e: <cpg>/cpg-analysis/build.gradle.kts:46:33: Unresolved reference: cpgLanguageCxx

FAILURE: Build failed with an exception.

* Where:
Build file '<cpg>/cpg-analysis/build.gradle.kts' line: 46

* What went wrong:
Script compilation error:

  Line 46:     testImplementation(projects.cpgLanguageCxx)
                                           ^ Unresolved reference: cpgLanguageCxx

1 error

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

* Get more help at https://help.gradle.org

BUILD FAILED in 815ms
12 actionable tasks: 12 up-to-date

git bisect tells me that it was commit 2b89676a6f6f4e8894e040b9a658ffa78735e614 that caused the problem.

Indeed, that fixed it. Thanks!


Remove verification page & link

Currently, the verification email includes a link, which mark the user as verified when opened. This is problematic for security reasons:

GET request should not cause side effects
User should log-in before verification to ensure a typo in email would not cause account takeover

Therefore, to support a secure verification page, we need to require manual action & authentication before verification. However, due to complex login behaviors & methods, and the use of header session transport, it is impractical to implement a generic login page at the moment.
For now, we'll mitigate the existing problem by:

remove verification GET endpoint
remove the link in email and show only the code

remove the link in email and show only the code
This is a very weird default behaviour?

remove verification GET endpoint
This could be mitigated with a GET page that requires another click?

Instead of removing them and make the default experience so weird, actually are we using cookies on JS SDK by default? If yes maybe it is just easier if we provide a default URL that respects the cookies, show a button for a click to verify the code, and redirect users to error page if not logged in (or even better, create a redirect URL setting for where to redirect if logged in is needed)
Then the default experience is more complete...
For users use auth token, we shall explain in our guides and user portal that the link doesn't work for auth token and they should implement the URL on their own (assuming auth token is not the default option, so people switched it knows what they're doing?)
That seems way better than removing the whole thing...

BTW, for big feature changes... please put it on feature repo instead of in individual repo so it would update our spec and I don't look at every issues here (it just happen we discussed this today so...)


Class 'Twilio\Rest\Fax' not found

I am trying to send SMS using the example, however when running my PHP file I get this error: Class 'Twilio\Rest\Fax' not found.
I am using the latest v3.0.0 package, installed via composer.
Any ideas?

Hi @shmaltz,
could you share your composer.lock?

Thanks! It seems you are using version 2.3.10 for signalwire-community/signalwire; could you try requiring v3.0.0?
composer.json:
    "require": {
        "signalwire-community/signalwire": "^3.0"
    }

Thank you, that worked!
It's interesting why composer installed the old version.


Add header section with logo

Value description
As a user
I want to see the shop's logo at the top of the screen, so I know I am in the right shop.
Description

Acceptance Criteria

[ ]  the logo of the shop should be shown in the header.
[ ]  the header should always stay in the foreground and stay in position.

Tasks

[ ]  Create a new branch
[ ]  Create a header navigation element
[ ]  Creae a logo container with maximum sizes
[ ]  Place the logo centered

Complexity
low

I would replace header-navigation with header-element in the second task. You only make a headline in the header and no navigation.


Updated node version

Fixes #270

@saniales Can you help with testing the action from @main. Updated the build with node16 updates

Yes I can what do you need?

If you are using this action, I guess it must be at v2.0.0. Can you test with @main? I have few test cases running with @main, and they are all green, but want to be sure before releasing v2.1.0 from main branch

Yes, will do

@saniales Did you get a chance to test action from @main?

No but i will tomorrow

I tested it and it works, you can release new version

Thank you.


build: update wasmvm and update contracts for tests

Motivation and context
This PR updates a test and test data. This is for following https://github.com/line/cosmwasm/pull/234 and https://github.com/line/wasmvm/pull/74.

How has this been tested?
with x/wasm/dynamic_link_test.go

Screenshots (if appropriate):
Checklist:

[x] I followed the contributing guidelines and code of conduct.
[x] I have added a relevant changelog to CHANGELOG.md
[x] I have added tests to cover my changes.
[x] I have updated the documentation accordingly.
[ ] I have updated API documentation client/docs/swagger-ui/swagger.yaml (not needed)

This status is draft, can I review it?

This status is draft, can I review it?

Yes. sorry, it was missed.


blade create -t servicebuilder inside liferay-workspace doesn't create workspace build.gradle files

fixed in commit 81701ae1a96892995b2f0cb76dd3ffb4e2b26612


Game crashes when placing down train track

Describe the Bug
After updating to 0.5c, placing a train track causes the game to crash for the user who places the track.
Other players on the server are not affected unless they get within a certain distance of the placed track.
Other new items from update like the steam whistle don't seem to trigger a crash.
Reproduction Steps

Launch 1.18.2 Fabric with 0.5c Create
Place train track on ground
Game will go into a "Saving World" screen briefly before it crashes and prompts error message: java.lang.IllegalStateException: Wrapping function stack not empty!
...

Expected Result
Track to be placed on ground.
Screenshots and Videos
No response
Crash Report or Log
https://pastebin.com/v2Wxtpyv
Operating System
Windows 10
Mod Version
0.5.0c
Minecraft Version
1.18.2
Other Mods
additionaladditions: Additional Additions 4.0.5
adorn: Adorn 3.5.0+1.18.2
advancementplaques: Advancement Plaques 1.4.5
advdebug: Advancements Debug 2.3.0
appleskin: AppleSkin 2.4.0+mc1.18.2
architectury: Architectury 4.5.76
badpackets: Bad Packets 0.1.2
balm-fabric: Balm 3.1.0+0
betteranimalsplus: Better Animals Plus 1.18.2-11.0.5
bewitchment: Bewitchment 1.18-22
bosses_of_mass_destruction: Bosses of Mass Destruction (Beta) 1.3.7-1.18.2
byg: Oh The Biomes You'll Go 1.4
cardinal-components: Cardinal Components API 4.2.0
citresewn: CIT Resewn 1.1.1+1.18.2
comforts: Comforts 1.1.2-1.18.2
continuity: Continuity 2.0.0+1.18.2
create: Create 0.5.0c-708
croptopia: Croptopia 2.0.5
ecologics: Ecologics 1.7.2
explorerscompass: Explorer's Compass 1.18.2-2.1.0-fabric
fabric: Fabric API 0.57.0+1.18.2
fabric-language-kotlin: Fabric Language Kotlin 1.8.1+kotlin.1.7.0
fabricloader: Fabric Loader 0.14.8
forgottengraves: Forgotten Graves 1.18.2-2.6.0
friendsandfoes: Friends&Foes 1.4.5
geckolib3: Geckolib 3.0.35
iceberg: Iceberg 1.0.38
indium: Indium 1.0.6+mc1.18.2
infusion_table: Infusion Table 1.1.0
inmis: Inmis 2.6.0-1.18.2
iris: Iris 1.2.5
journeymap-fabric: Journeymap 5.8.2
lazydfu: LazyDFU 0.1.2
lithium: Lithium 0.7.10
mavm: More Axolotl Variants Mod 1.0.7a
mcda: MC Dungeons Armors 2.0.6
mcdar: MC Dungeons Artifacts 1.4.8
mcdw: MC Dungeons Weapons 4.4.0
medievalorigins: Medieval Origins 1.18.x-4.0.3
memoryleakfix: Memory Leak Fix 1.18.2-0.7.0
minecraft: Minecraft 1.18.2
modmenu: Mod Menu 3.2.3
morevillagers: More Villagers 3.3.2
mostructures: Mo' Structures 1.4.2+1.18.2
naturescompass: Nature's Compass 1.18.2-2.0.3-fabric
origins: Origins 1.4.1
patchouli: Patchouli 1.18.2-71.1-FABRIC
pehkui: Pehkui 3.3.3+1.14.4-1.19.1
playerabilitylib: Pal 1.5.1
reeses-sodium-options: Reese's Sodium Options 1.4.3+mc1.18.2-build.42
roughlyenoughitems: Roughly Enough Items 8.3.501
seethroughlava: SeeThroughLava 2.3
skylorlib: SkyLib 1.3.35
sodium: Sodium 0.4.1+build.15
sodium-extra: Sodium Extra 0.4.5+mc1.18.2-build.42
starlight: Starlight 1.0.2+fabric.89b8d9f
terrablender: TerraBlender <IP_ADDRESS>
trinkets: Trinkets 3.3.1
twilightforest: The Twilight Forest 4.1.251
untitledduckmod: Untitled Duck Mod 0.5.0
vanilla-hammers: Vanilla Hammers 3.4.0-1.18.2
waystones: Waystones 2.5.4
wthit: wthit 4.11.0
Additional Context
No response

I have noticed that Iris 1.2.5 was causing the crashing with the track and when I updated Iris to 1.2.6 it started working as intended.

Your iris version is too old

Thanks! Updating Iris did the trick.


[3] add home page

add general api (WIP)
add api and models
update changelog
Autogenerate API
add docs + openapi models
Autogenerate API
refactoring
adjust compilation flags
add cfg flags + refactoring
cargo fmt
Autogenerate API
add cfg flags
refactoring
update .gitignore
Autogenerate API
post rebase fix
postrebase fix + better logging
add general api (WIP)
add api and models
update changelog
Autogenerate API
add docs + openapi models
Autogenerate API
refactoring
adjust compilation flags
add cfg flags + refactoring
cargo fmt
Autogenerate API
add cfg flags
refactoring
update .gitignore
Autogenerate API
update API
add homepage

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

Current dependencies on/for this PR:

main

PR #9

PR #15
PR #10

PR #12

PR #13

PR #16

PR #18

PR #19

PR #20


Hald CLUT to shader

Hi, i want to apply hald clut on a video. But this lib is limited to images only. So, i want to write Hald 64x64x64 LUT into opengl shader but i am stuck in logic behind it. Can you please explain, how to read data from halt clut.

@dntks any update??

@Zer0bee
First of all; using a Color LUT means you will most probably use less color depth than the original image/video. If you check the LUTImage.java in the code it has the rgbDistortion property, that stores how much less colors you will have correlated to the 8 bit (256256256) color depth. If you have a 4096x4096 LUT image you won't have the distortion (as 40964096 = 256256*256).
There is the identity LUT which has all the colors in the correct order: if you check the RGB(256,0,0) = pure red color, when you get this on the identity LUT, you will get the same red color. First step using a LUT is to check the original color's coordinate on the Identity LUT. Identity LUT-s differ by the type of the LUT, which can be square LUT or Hald LUT. In the EasyLUT library this is called  LutAlignment.
Identity LUTs can also vary in how they use the 3 color coordinates, the x y and z axes represent a color each, so X can be the red, Z the blue, Y the green, or any other combination. That's why it's important to define this before using the LUT, you need to define how the identity LUT looks like. This is handled by CoordinateToColor.java class and the CoordinateToColor.Type, and the DistortedColor class's methods get the coordinates of the given color on the identity LUT.
LUT images can differ in their size as well, they can have 256x16,  64x64 or any variation of pixel dimension that's why the rowdepth is important, it stores how the 3d cube is represented in the 2d LUT image (how many 2d layers are in one row).
Then comes the LutAlignment, which returns the 2d coordinate representing the color in the Identity LUT. The library gives the rowDepth and the 3d coordinates to the Alignement, which returns the 2d coordinates of the color of the identity LUT. For Hald this looks like:
class Hald implements LutAlignment {
        @Override
        public int getX(int rowDepth, int sideSize, int x, int y, int z) {
            int red = x;
            int green = y % rowDepth * sideSize;
            return red + green;
        }
        @Override
        public int getY(int rowDepth, int sideSize, int x, int y, int z) {
            int green = y / rowDepth;
            int blue = z * rowDepth;
            return green + blue;
        }
}

Then you have the 2d coordinates for the color, then you just need to get the same coordinate on you distorted LUT image.

@dntks Thanks for the explanation. It really helps. 👍

@Zer0bee any update on GLSL shader ?


clarification needed for creator in FAQ

"FAQA7":
I suggest to note, that creator only needs funds for fees, I guess it is not clear for new-comers

"FAQA7": Also I suggest to delete excuses about possible issues and project youth

Let's see if this is cleared up @cryptodamsky ?


feat: browser detector added

Description 📜
Browser detector added
Fixes #831

Type of change 📝

[] Add a new project

Domain of Contribution 📊
Please delete options that are not relevant. And in order to tick the check box just put x inside them for example [x] like this

[] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant.And in order to tick the check box just but x inside them for example [x] like this

[] I have performed a self-review of my own code or work.
[] I have commented on my code, particularly in hard-to-understand areas.
[] My changes generates no new warnings.
[] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸

Hello @gurjeetsinghvirdee, sorry I had an exam yesterday. Could you kindly review this PR.


[FEATURE] Support new Qianfan model enum such as ERNIE_Speed and ERNIE_Lite.

Is your feature request related to a problem? Please describe.

Qianfan has been published some new model such as ERNIE_Speed and ERNIE_Lite that is free to use.

It would be easily to use if LangChain4j-qianfan can directly using model name to call the model.
Describe the solution you'd like

It would be easily to use if LangChain4j-qianfan can directly using model name to call the model, without pass the endpoint parameter, such as:
        ChatLanguageModel chatModel = QianfanChatModel.builder()
                .modelName(QianfanChatModelNameEnum.ERNIE_SPEED_8K.getModelName())
                // or
                // .modelName("ERNIE-Speed-8K")
                .apiKey(apiKey)
                .secretKey(secretKey)
                .build();

Describe alternatives you've considered

Additional context

currently, if user want to call ERNIE_Speed or other model, they should pass 'endpoint' parameter, which looks like:
return QianfanChatModel.builder()
                .modelName("ERNIE-Speed-8K")
                .endpoint("ernie_speed")
                .apiKey(System.getenv("QIANFAN_API_KEY"))
                .secretKey(System.getenv("QIANFAN_SECRET_KEY"))
                .build();

I can work on it tomorrow~


[embedded] Re-enable executable trap tests on Linux

After https://github.com/apple/swift/pull/72971, they should pass now.

@swift-ci please test

@swift-ci please test


when i use rpc server , always appear server has gone away

Laravel Version: 5.4
PHP Version: 7.1
Database Driver & Version: mysql 5.6

Description:
when i use rpc server , always appear server has gone away.
Steps To Reproduce:

Do you really think that this issue has enough information for us to reproduce the problem and fix it?

mysql server has gone away.
my rpc server based on swoole.


Firebase Functions Support

Hello,
I wish to use firebase for my functions deployment. I can currently run the functions, without functions_framework, with
firebase emulators:start
This works fine but I cannot use this for local debugging (say I'm in VSCode).
I can try;
firebase emulators:start --inspect-functions but unfortunately I get an error
--inspect-functions only supported for Node.js runtimes.
My next option is to use functions_framework for local debugging. This is great, but it seems it will only work with a single function (which is incomplete for development needs), unless I install minicube or some other product. However, this is out of sync with my firebase functions, which do not use minicube.
My question is, is there a way to run multiple functions for firebase functions that I can debug, using functions_framework?

It would be great if we could fix this basic ask.  There really isn't a better way to unit test python Firebase functions.  At least if someone on the team has a workaround that would be great.


Replace "" by "\"

Description
What steps will reproduce the problem?

Tried to replace all "" by "\"
Crash
my code :
"
import pandas as pd

a = pd.read_csv("C:\Users\badia\Desktop\PSB\s1\Python\Exercices\data.csv")
"
What is the expected output? What do you see instead?
Please provide any additional information below
File "C:\Users\badia\Anaconda3\lib\site-packages\spyder\widgets\findreplace.py", line 504, in replace_find_all
self.replace_find(focus_replace_text, replace_all=True)
File "C:\Users\badia\Anaconda3\lib\site-packages\spyder\widgets\findreplace.py", line 449, in replace_find
if len(re.findall(pattern, cmptxt2)) > 0:
File "C:\Users\badia\Anaconda3\lib\re.py", line 222, in findall
return _compile(pattern, flags).findall(string)
File "C:\Users\badia\Anaconda3\lib\re.py", line 301, in _compile
p = sre_compile.compile(pattern, flags)
File "C:\Users\badia\Anaconda3\lib\sre_compile.py", line 562, in compile
p = sre_parse.parse(p, flags)
File "C:\Users\badia\Anaconda3\lib\sre_parse.py", line 847, in parse
source = Tokenizer(str)
File "C:\Users\badia\Anaconda3\lib\sre_parse.py", line 231, in init
self.__next()
File "C:\Users\badia\Anaconda3\lib\sre_parse.py", line 245, in __next
self.string, len(self.string) - 1) from None
sre_constants.error: bad escape (end of pattern) at position 0
Version and main components

Spyder Version: 3.2.3
Python Version: 3.6.2
Qt Versions: 5.6.2, PyQt5 5.6 on Win

Thanks for reporting. We'll fix this problem is a future release.

Note: This happens if the user has activated regular expressions. \ is of course not a valid expression to search for.
One part of the problem is that the user was probably not aware of this fact. This is addressed by the above PR [#5614].
The second aspect is that acually performing the search on an invalid regexp raises the above error. One can debate how to handle this. In the simplest case, one could catch re.error for the call of re.findall() in FindReplace.replace_find(). Alternatively, one could deactivate the replace buttons for invalid regular expressions. Personally, I'd think that the former solution is enough if the user has already feedback through PR [#5614].

@timhoffm, I agree that catching re.error is the best that we can do. Could you implement that in another PR, please?


ResolveFully not resolving extensions of child schemas

At line 462 of ResolveFully class, extensions of only source schemas are being added to target schema. Shouldn’t it copy extensions of resolved schemas as well, similar to the way properties and required of resolved schemas are handled?
https://github.com/swagger-api/swagger-parser/blob/2db8c628cbbfdd8afb972965698e011c93747b15/modules/swagger-parser-v3/src/main/java/io/swagger/v3/parser/util/ResolverFully.java#L462

Here's a simple case
`openapi: "3.0.2"
info:
title: "Extension not inherited"
components:
schemas:
ResponseModel:
x-extension-ref: 1
type: object
required:
- data
properties:
data:
type: "string"
paths:
/test/extensions/not/inherited:
get:
responses:
200:
content:
application/json:
schema:
allOf:
- $ref: "#/components/schemas/ResponseModel"
- type: "object"
required:
- "data"
properties:
data2:
type: "string"
`

fixed in #1681 and #1837


Using require('node-fetch').Headers causes issues

When matching headers that have been created with new Headers(), fetch-mock checks if the object is an instance of Headers: https://github.com/wheresrhys/fetch-mock/blob/master/src/compile-route.js#L32
The problem is that this Headers is hard coded to be node-fetch's impl: https://github.com/wheresrhys/fetch-mock/blob/master/src/compile-route.js#L4
However, we are using whatwg-fetch, so this check fails and therefore the route isn't matched. It seems to me that compile-routes should be given the FetchMock.Headers, which contains the implementation that the user is using, and can be overwritten by them.
Does that make sense, and if so would you be happy with a PR that did the above?

It may be better to try to duck-type if something is a Header by looking for a suite of functions. Also currently we rely on Headers.prototype.raw which is not standard and seems node-fetch specific.

HI Jack,
There is a way to pass in custom implementations of all the fetch classes - fetchMock.setImplementations(). Hopefully this meets your requirements. And hopefully the recent changes to fix other bugs (https://github.com/wheresrhys/fetch-mock/releases/tag/v5.10.1), which moved further away from duck-typing, won't have affected too many users

Hey Rhys,
From what I see though setting a custom Header implementation it still doesn't affect the check in that code I linked to earlier - I did try setting a custom Header impl and it didn't work. I'll see if I can reproduce in a small test case.

It does look that 5.10.1 might fix my issue though - I'll try it at work soon!

Hi Jack. There was also a recent bug todo with Headers which will be fixed in a minute


Docker Machine Support

In what area(s)?

/area runtime
What version of Dapr?

0.1.0
Expected Behavior
Dapr runtime should be able to connect to redis when using docker-machine.  localhost doesn't work when making connections to redis.  Is there an override property we can use to set the redis url?
 Status: Running.    11 minutes ago      Up 11 minutes       <IP_ADDRESS>:6379->6379/tcp     hopeful_hertz
fd0d3adfa6b9        daprio/dapr                  "./placement"            11 minutes ago      Up 11 minutes       <IP_ADDRESS>:50005->50005/tcp   dapr_placement

Steps to Reproduce the Problem
(Install docker-machine https://docs.docker.com/machine/install-machine/)
# Create new docker-machine
docker-machine create dapr-test
docker-machine env dapr-test
eval $(docker-machine env dapr-test)
# Init dapr
dapr init
# Try sample project
git clone https://github.com/dapr/samples.git
cd samples
cd 1.hello-word
npm install
dapr run --app-id mynode --app-port 3000 --port 3500 node app.js

Found what I'm looking for here: https://github.com/dapr/docs/blob/master/howto/setup-state-store/setup-redis.md


quote filename for edit

Some text editor treat the argument which contains space as multiple arguments. ex: gvim.exe

Thank you


⚠ Bofrost.de has degraded performance

In 6ee13b5, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1325 ms

Resolved: Bofrost.de performance has improved in 5b22aed.


android clipboard issue

Hello
I am a novice in the development of South Korea
I had a copy & paste the Korean in FreeRDP, but displays garbled
Do I have to change where the part?
Please Help

Clipboard issues fixed on current master 3c7cc555a5a5f02366918be7eb935bb1f22115ef


 Just prompt come back that's all
Application details

FreeRDP version (xfreerdp /version)
The last build today, 2023/01/11

Command line used
wfreerdp /?
wfreerdp /h
wfreerdp --help
wfreerdp -h

Output of xfreerdp /buildconfig
nothing

OS version connecting to (server side)
debian, bullseye xrdp

If available the log output from a run with /log-level:trace 2>&1 | tee log.txt
NA

If you built it yourself add some notes which tag/commit/branch you have used, also your cmake parameters and
compiler can help
NA

Environment (please complete the following information):

OS: Windows 10

windows bug, stdout and stderr are detached for console applications.
try with wfreerdp.exe <yourargs> >log.txt 2>&1 to get the output

@G3G4X5X6 use cmd.exe or look up how to redirect stdout and stderr on powershell.

@akallabeth Thank you, with your reminder, my doubts have been resolved. The issue lies in the program not providing any feedback when it receives incorrect parameters.

Correction, it's not that there was no feedback, but rather it was not displayed in the terminal. After redirecting it to a file, I found that there was correct output.

can not reproduce with our nightly builds, entering the command line you have given I get the help output.
This is FreeRDP version 3.0.0-dev5 (13b44b8b7)

sha256 E3AF24ED1DAA949A986C876D3BEC4A21BCA3AC56730FF326E195E7A427176BBF *wfreerdp.exe

I want to express that when using the command line in the terminal, there is no output displayed in the terminal itself. However, when redirecting the command line output to a file, you can open the file to see the normal output.
# no output in cmd (Terminal)
wfreerdp.exe /version

# output redirected in log.txt
wfreerdp.exe /version  >log.txt 2>&1

log.txt
[10:38:34:15] [24872:00005b80] [WARN][com.freerdp.client.common.cmdline] - [freerdp_client_warn_deprecated]: [deprecated] wfreerdp.exe client has been deprecated
[10:38:34:15] [24872:00005b80] [WARN][com.freerdp.client.common.cmdline] - [freerdp_client_warn_deprecated]: As replacement there is a SDL based client available.
[10:38:34:15] [24872:00005b80] [WARN][com.freerdp.client.common.cmdline] - [freerdp_client_warn_deprecated]: If you are interested in keeping wfreerdp.exe alive get in touch with the developers
[10:38:34:15] [24872:00005b80] [WARN][com.freerdp.client.common.cmdline] - [freerdp_client_warn_deprecated]: The project is hosted at https://github.com/freerdp/freerdp and  developers hang out in https://matrix.to/#/#FreeRDP:matrix.org?via=matrix.org - dont hesitate to ask some questions. (replies might take some time depending on your timezone)
[10:38:34:12] [24872:00005b80] [INFO][com.freerdp.client.windows] - [wf_has_console]: Detected stdin=-1 -> gui mode
This is FreeRDP version 3.0.0-dev5 (70a360bf3)

@G3G4X5X6 well, read the explanation in my first reply...


Feature/aoloso/hybrid openmp

Description

Several changes to support component level hybrid MPI/OpenMP.
Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[ ] I have tested this change with a run of GEOSgcm (if non-trivial)
[ ] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[ ] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

I also have some suggestions for changes. I'll write a separate comment after I finish my lunch

Uh oh. Need to fix up the GEOSadas-for-MAPL-develop branch...a moment please...

The UFS failure might need @tclune help:
2/7 Test  #8: MAPL.profiler.tests ..............***Failed  Error regular expression found in output. Regex=[Encountered 1 or more failures/errors during testing]  1.13 sec
................ Timer <timer_1> does not match start timer <timer_2>
F....................
Time:         0.005 seconds

Failure
 in:
test_TimeProfiler_suite.test_stop_wrong_meter
  Location:
[test_TimeProfiler.pf:41]
Failed to throw exception: <Timer <timer_1> does not match start timer <timer_2>>

 FAILURES!!!
Tests run: 36, Failures: 1, Errors: 0
, Disabled: 0
*** Encountered 1 or more failures/errors during testing. ***

Note: There still seems to be an issue as the CI does not want to build MAPL:
/root/project/MAPL/base/MaplGrid.F90:46:9:

   46 |     use ESMFL_Mod, only : ESMFL_UnitsRadians
      |         1
Fatal Error: Cannot open module file 'esmfl_mod.mod' for reading at (1): No such file or directory
compilation terminated.

I'm a bit baffled by this as it looks like both MaplGrid.F90 and ESMFL_Mod.F90 are both in base/
@tclune Could this be a weird dependency issue? MaplGrid depends on ESMFL, ESMFL depends on MAPL_Base, and MAPL_Base depends on MaplGrid?

Yep. Ninja lays it out:
ninja: build stopped: dependency cycle: include/MAPL.base/mapl_abstractgridfactorymod.mod -> include/MAPL.base/mapl_basemod.mod -> include/MAPL.base/mapl_maplgrid.mod -> include/MAPL.base/esmfl_mod.mod -> include/MAPL.base/mapl_abstractgridfactorymod.mod.

OK - presumably something in this PR triggered this.   Presumably it is MaplGrid.  Looking...

OK - this is perverse.    The "units" argument is not even used by ESMFL, so there is no reason to specify it!
In any even the named constant should be moved to ./shared/Constants/InternalConstants.F90 if it is truly needed.
ESMFL should be treated as deprecated, but we can't delete it just yet.

I tested this (along with #1683) with the GCM and it seems zero-diff and good to go.


Iterations clean

This should be much cleaner, I apologize for the trouble.

Reviewed in #35 but still need to sanity check this


Enable custom pod labels for helm chart

Hey, we leverage tooling that requires certain labels to be present. While deploying into our ecosystem I noticed the pods don't have the capability to have custom labels. This PR is meant to include that annotation and feature for others that require specific pods labels

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

Hi @MarkDPierce, why customLabels cannot be used? They should be applied to Deployment as well, here.

When I use a custom label of environment: test and generate the manifest I get the following.
# Source: k6-operator/charts/k6-operator/templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: release-name-k6-operator-controller-manager
  namespace: release-name-system
  labels:
    control-plane: "controller-manager"
    app.kubernetes.io/component: controller
    helm.sh/chart: k6-operator-1.2.0
    app.kubernetes.io/name: k6-operator
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/version: "0.0.11"
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/part-of: k6-operator
    environment: test
  annotations:

spec:
  replicas: 1
  selector:
    matchLabels:
      app.kubernetes.io/name: k6-operator
      app.kubernetes.io/instance: release-name
  template:
    metadata:
      labels:
        app.kubernetes.io/name: k6-operator
        app.kubernetes.io/instance: release-name
    spec:
      containers:
        - name: kube-rbac-proxy
          image: "europe-docker.pkg.dev/dbk-ecom-cicd/docker-hosted/kubebuilder/kube-rbac-proxy:v0.8.0@sha256:34e8724e0f47e31eb2ec3279ac398b657db5f60f167426ee73138e2e84af6486"
          imagePullPolicy: IfNotPresent
          args:
            - --secure-listen-address=<IP_ADDRESS>:8443
            - --upstream=http://<IP_ADDRESS>:8080/
            - --logtostderr=true
            - --v=10
          ports:
            - containerPort: 8443
              name: https
        - name: manager
          image: "europe-docker.pkg.dev/dbk-ecom-cicd/docker-hosted/k6-operator/controller:v0.0.11@sha256:d80c85a54d3687b346cba8e44421364f87d54f89b6a7adace839d49e0b50c6d8"
          imagePullPolicy: Always
          resources:
            limits:
              cpu: 50m
              memory: 50Mi
            requests:
              cpu: 50m
              memory: 50Mi
          command:
            - /manager
          args:
            - --enable-leader-election
            - --metrics-addr=<IP_ADDRESS>:8080
      serviceAccount: k6-operator-controller
      terminationGracePeriodSeconds: 10

These customLabels are not included in the template metadata. Due to security reasons we need labels at the pod level. So every templated pod will get these custom labels.
Thanks for actually asking because I totally forgot to add {{- include "k6-operator.customLabels" . | nindent 8 }} to the deployment.yaml template.

Sorry, that was a long winded explanation to just say deployment.yaml only has
spec:
  replicas: 1
  selector:
    matchLabels:
      {{- include "k6-operator.selectorLabels" . | nindent 6 }}
  template:
    metadata:
      labels:
        {{- include "k6-operator.selectorLabels" . | nindent 8 }}
    spec:

So customLabels wouldnt work in this scenario.

Thanks for full explanation. Now after some looking around, I'm not fully certain we should have customLabels at all :confused:  E.g. these charts don't seem to propagate user-defined labels at the level of Deployments or StatefulSets:
https://github.com/grafana/mimir/blob/main/operations/helm/charts/mimir-distributed/values.yaml
https://github.com/grafana/loki/blob/main/production/helm/loki/values.yaml
In k6-operator case, there's only one Deployment which makes it easier; though I suspect in the future we might need to make further breaking changes here.
I'd appreciate some feedback from community here, about how everyone uses labels in their charts :slightly_smiling_face:

It's a feature change and other charts we've worked on within the open
source community prefer a major bump. I also attempted to find any docs you
might have had towards the process. So went with best guess, I'm happy to
change or conform to what ever you deem appropriate.
On Tue, Oct 31, 2023, 11:35 Olha Yevtushenko @.***>
wrote:

@.**** commented on this pull request.
In charts/k6-operator/Chart.yaml
https://github.com/grafana/k6-operator/pull/326#discussion_r1377381591:

@@ -2,7 +2,7 @@ apiVersion: v1
appVersion: "0.0.11"
description: A Helm chart to install the k6-operator
name: k6-operator
-version: 1.2.0
+version: 2.0.0

Why do you think it should be a breaking change? This seems like a "pure"
addition: old deployments should continue to work as is. I.e. why not 1.3.0?
FYI, I'll need to test it a bit and probably return to this PR ~ tomorrow.
—
Reply to this email directly, view it on GitHub
https://github.com/grafana/k6-operator/pull/326#pullrequestreview-1705970544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK56WNSNSJ57K32AQ5HWDVLYCDH6JAVCNFSM6AAAAAA6V4LZRCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMBVHE3TANJUGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey sorry for the delay. Work had prio.
I took a stab at the line breaks in your labels and annotations but it might require a bit of refactoring. At least with both you have | default "" | which means it's by design. So I took a stab at a bit of refactoring and managed to achieve the following
---
# Source: k6-operator/templates/namespace.yaml
apiVersion: v1
kind: Namespace
metadata:
  name: release-name-system
  labels:
    app.kubernetes.io/name: release-name-k6-operator
    control-plane: "controller-manager"
    foo: bar
  annotations:
    bar: foo
---
# Source: k6-operator/templates/serviceAccount.yaml
apiVersion: v1
kind: ServiceAccount
metadata:
  name: k6-operator-controller
  namespace: release-name-system
  labels:
    app.kubernetes.io/component: controller
    helm.sh/chart: k6-operator-2.0.0
    app.kubernetes.io/name: k6-operator
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/version: "0.0.11"
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/part-of: k6-operator
    foo: bar
  annotations:
    bar: foo

But I do think the changes would be out of scope for this change.

@MarkDPierce yes, the line break would be out of scope, but thanks for taking a look. If you see the solution, please feel free to open a PR :slightly_smiling_face: ATM, I don't think default "" is a must so long as the values are passed correctly or are omitted correctly :thinking:
Either way, the podLabels change looks good, thank you for this!


[WIP] importers: migrate code from invenio-rdm-records

partially closes https://github.com/inveniosoftware/invenio-rdm-records/issues/822

closing in favour of https://github.com/inveniosoftware/invenio-vocabularies/pull/100


HSEARCH-4498 Fluent definition of composite projections

https://hibernate.atlassian.net/browse/HSEARCH-4498

I'll submit another PR with a different API, more appropriate if we want to use it in nested projections (HSEARCH-3943).


Inquiry on your dataset

I am interested in your work and I really appreciate your efforts in retraining all the baseline models on the same datasets.  I would like to know that would you publish your datasets as well as the codes? Thank you very much!

Thank you for the interest!
The code of Matformer and corresponding dataset will be released early 2023 according to our plan.

Thank you for the interest! The code of Matformer and corresponding dataset will be released early 2023 according to our plan.

Thank you very much! And I guess that this is the same dataset used in your work? https://figshare.com/collections/ALIGNN_data/5429274 Thanks again!

Yes, for the formation energy and band gap tasks in MP and all five tasks in Jarvis, they are the same as ALIGNN, with the same splits. For the other two tasks in MP, they are the same as another baseline method.

Thanks for the reply!

Sorry, but I couldn't find the original split of datasets used by ALIGNN. Did you use random_seed 123 to shuffle the MP and JARVIS data? (Also, did you use random seed 456 in that subset same as GATGNN?) This would be very helpful! Thank you very much!

You can use directly the same data preprocessing and loading code without changing the random seed to obtain the original splits used by ALIGNN. And this can be found in their official code in GitHub.  For the GATGNN, we use their released datasets. If you want to use these splits now, it is better to do the above.

Now you can use the official code, thank you.


gem push TAB raises error

I get the following error when typing gem push with a tab following:
_arguments:comparguments:325: doubled rest argument definition: *:gem:_files -g "*.gem(-.)"

gem + TAB works fine. I'm not sure whether this is a bug, filing anyway since this may be someone else's problem too.
P.S. I initially posted this to zsh-autosuggestions but according to folks over there this is not their issue.

Try reset the completion cache: rm ~/.zcompdump* and restarting the shell.

@mcornella I did as you suggested and the error persists.

I suppose you're not using the gem plugin, in which case the issue is within zsh's official gem completion. What zsh --version are you using?

@mcornella yes, I'm not using the gem plugin. I noticed that by enabling the gem plugin the errors go away. I don't get any suggestions for gem push however.

This was fixed 11 days ago: https://github.com/zsh-users/zsh/commit/191e05b1083d8c8ac8b2858c9278a7bd5414881f. I guess it will be in the next zsh version. In the meantime, you can download the _gem file and put it in some folder that's in $fpath. Remember to clean up the zcompdump files afterwards to reset the completion: rm ~/.zcompdump*.
Related: https://github.com/rubygems/rubygems/issues/2758
Related: https://github.com/zsh-users/zsh/pull/35

@mcornella thanks for your support and constructive comments. The workaround worked like charm. Much appreciated.

Thank you for your kind feedback!


xunsafe.Field.Offset may be wrong in struct with embedded struct

https://github.com/golang/go/issues/61495

is this actual problem ?


Add Hacktoberfest Python Projects #4

Added Hacktoberfest Python Projects #4
Fix to  Issue #
===============================

===============================
Checklist

[x] I read the contributing guidelines
[x] I structured my project as stated in the contributing guidelines
[x] My project uses dependencies
[x] I starred the repo
[x] I had fun contributing!

@siddhesh-coder
Howdy?, you haven't responded to the review.

I have make the new PR


问题：lowerCamel变量命名匹配有误（myNAME匹配有误、$name在编码规约中明确指出是反例）

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.hWX619444 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


The Metadata type from Storage is exposing a type that is meant to be private in public surface area via type alias

Public APIs that accept Metadata are leaking a type from the Details:: namespace, specifically CaseInsensitiveComparator because of the type alias. Given Metadata is an alias for a custom templated map, the user ends up relying on a private implementation detail, which goes against the guidelines, which means we wouldn't be able to rename or otherwise break the type.
https://github.com/Azure/azure-sdk-for-cpp/blob/76db013528d60e3f54919628747dd990a9c36600/sdk/storage/azure-storage-common/inc/azure/storage/common/storage_common.hpp#L69
For example, see:
https://github.com/Azure/azure-sdk-for-cpp/blob/90430040d5b446ade88caf2400f344040be16cb3/sdk/storage/azure-storage-files-shares/inc/azure/storage/files/shares/share_directory_client.hpp#L163-L165
https://github.com/Azure/azure-sdk-for-cpp/blob/90430040d5b446ade88caf2400f344040be16cb3/sdk/storage/azure-storage-files-datalake/inc/azure/storage/files/datalake/datalake_file_system_client.hpp#L143-L146
https://github.com/Azure/azure-sdk-for-cpp/blob/90430040d5b446ade88caf2400f344040be16cb3/sdk/storage/azure-storage-files-datalake/inc/azure/storage/files/datalake/datalake_options.hpp#L110-L128
cc @Jinming-Hu, @JeffreyRichter

My recommendation here is for Storage to create a Metadata class that has a private field of this type: std::map<std::string, std::string, Details::CaseInsensitiveComparator>.
Then, replicate the map's methods on this Metadata type (or whichever methods make sense for Metadata - maybe not all methods make sense). You could hard code strings for the keys/values. since metadata must always be string/string pairs.

My recommendation here is for Storage to create a Metadata class that has a private field of this type: std::map<std::string, std::string, Details::CaseInsensitiveComparator>.
Then, replicate the map's methods on this Metadata type (or whichever methods make sense for Metadata - maybe not all methods make sense). You could hard code strings for the keys/values. since metadata must always be string/string pairs.

I believe your recommendation works, but that sounds to me like an overkill.
What's the worst thing that could happen if we just keep things as is?

Anyway, I know core sdk team is working on a case-insensitive map, so I'm gonna switch to that some time in the future.

Yes, I guess we could have one and we could use it for headers as well as metadata. And, maybe even query parameters.


Fix KeyError: 'environmentName' on 'az account list'

When CLI telemetry attempts to get installation id, it would modify the user ~/.azure/azureProfile.json saving incorrect data into it.
We fix this by returning deep copies of subscriptions to prevent later modification (e.g. by telemetry).
https://github.com/Azure/azure-cli/issues/2031

Codecov Report

Merging #2358 into master will not change coverage.
The diff coverage is 80%.

@@           Coverage Diff           @@
##           master    #2358   +/-   ##
=======================================
  Coverage   72.94%   72.94%
=======================================
  Files         425      425
  Lines       19307    19307
  Branches     2740     2740
=======================================
  Hits        14083    14083
  Misses       4322     4322
  Partials      902      902

Impacted Files
Coverage Δ

...rofile/azure/cli/command_modules/profile/custom.py
22.72% <0%> (-1.16%)
:x:

src/azure-cli-core/azure/cli/core/_profile.py
83.92% <100%> (+0.04%)
:white_check_mark:

Continue to review full report at Codecov.

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


Reorganise evaluation

In a previous phase, the model evaluation is decoupled from the model building phase. In the later, only one metric was used for optimisation where in the former, many metrics can be used to evaluate the final model. After adding the options to use different algorithms (SVM, linear regression with continuous target, ...) for building your model, there should also be different options for evaluation as there are different types of models:

Binary target with binary scoring: basic evaluator class for binary model (contains accuracy, F1, precision, recall, ...)
Binary target with probabilistic scoring: basic evaluator (with threshold on probability) + AUC, lift, gains and lift curves, roc curve, ...)
Multi class model
Ordinal regression models
continuous target
...

All of these types of models can have different Evaluator classes, so it is important to use inheritance as much as possible, as well as having the proper utility functions split off for plotting.

New version will have ClassificationEvaluator and RegressionEvaluator classes, with separate metrics and plotting. Inheritance would be overkill, but might be useful if we add more model types.


Install fails while adding leveldown dependency

Having some trouble installing orbit-db to my working directory. npm i orbit-db fails while trying to add leveldown as a dependency. yarn add orbit-db fails in a similar fashion. Gist of verbose log
https://gist.github.com/Annorstades/9663c945ae48c9fc012a61cc253a4d58
Things apparently start to go south once prebuild-install || node-gyp rebuild is invoked.

I can also confirm that the package installation does not work at node v10 and I have to fall back to node v8.

you could override the leveldown dependency of orbit-db-cache using selective dependency resolution in yarn for now (or setting the leveldown version in the package-lock.json file and installing via npm ci).
Hopefully orbitdb/orbit-db-cache#7 will be merged soon

Can confirm this is a problem @haadcode

Note! We don't officially support Node.js v10 yet as it's not yet LTS. Once it is, we'll officially support it. That said, I think this is really important to fix and get working and we might as well do it before v10 LTS.
We've now updated the LevelDB related modules to v3.x and the updated dependencies are merged to master. The tests are passing locally with v10 and on CI with v8, so I'm assuming this is fixed. Note that this fix hasn't been published to npm yet, but we're expecting to make and publish a new release soon.
I'll leave the issue open until we have a new version of orbitdb in npm. Meanwhile, would highly appreciate if you'd test this and let us know if you're still experiencing problems! 🙏

Node 10 is stable now - this should be working for folks. Feel free to re-open or create a new issue if it's not.


Generate bech32-invalid checksum

Per @kirtaner. Makes address unspendable without ridiculous levels of effort unreasonable to expect naïve users to be able to accomplish.

Superseded by #2


Prepare brandkit

I think current logo padding is too much, and I want to have logo without text too.
This is used for header logo too.

I only prepared the small no padding logo.


Enhancement: Assert InvalidArgumentExceptions are thrown

This PR

[x] increases coverage for ZfModule\Mapper\ModuleHydrator by asserting that an InvalidArgumentException is thrown when objects are passed to hydrate() or extract() which do not implement ZfModule\Entity\ModuleEntityInterface

@localheinz thanks!


TB Management - Update cough duration [6h]

Issue
As for TB management, an individual becomes a TB suspect if, through acute illness, cough persists as a symptom for a period of 2 weeks or more.
As such we need to change the cough duration field to reflect a diagnosis for TB:

Less than 2 weeks
2 Weeks or more

Task

Add the new field and deprecate the old field
Create infrastructure to handle the new field
Update the UI on the forntend

@adamhstewart what do we do about progress report then?

@adamhstewart

@adamhstewart what do we do about progress report then?

This is what I've implemented, so we can make use existing if fields and get all needed info:

Will this do?

@anvmn ^^ Unfortunately, no - we need to convert to the two options as described, I assume this means a new field and infrastructure.
For the purposes of the progress report, we can assume less than 2 weeks to be 7 days and  more than 2 weeks to be 15 days.

@adamhstewart

For the purposes of the progress report, we can assume less than 2 weeks to be 7 days and more than 2 weeks to be 15 days.

It's 2 weeks or more, so I'll make it 14. This way we maintain consistency with max duration of other signs.

As discussed at catch up meeting:
Periods are:

2 Weeks or less
More than 2 weeks

QA Review
Found issues and questions

None

Checked and approved

Options on UI are updated


[NETBEANS-54] Module Review options.editor

I'm not entirely sure these files need license headers. Aren't they displayed to the user?

Note: related to #164, which was closed.


Add isGalleryImportAllowed to iOS

Hello all,
I think it would be nice to also add isGalleryImportAllowed to iOS as a potential feature.

Hey Any updates of this ^ ?


VCS_info broken in zsh 5.2, workaround?

Any theme using nvcsformats (VCS_info) to display different information depending on if current working directory is a repository or not, such as the pure theme, is broken in zsh 5.2.
It has been noted upstream already, but for those of us stuck at 5.2 (Arch users for example), is there any easy workaround for the time being?

Well, this is an old post but this still isn't fixed and I've figured something out and I thought it might help someone. Now, I don't use oh-my-zsh myself but I don't see why this shouldn't work with that as well.
You can use the start-up hook to specify the prompt (or whatever you use vcs_info for) you want in VCS directories and then use the no-vcs hook to overwrite that setting if you're in a non-vcs environment. Like this:
zstyle ':vcs_info:*+start-up:*' hooks vcs_prompt
zstyle ':vcs_info:*+no-vcs:*' hooks no_vcs_prompt
function +vi-vcs_prompt() {
    PROMPT='the-prompt-you-want-in-vcs-directories'
}
function +vi-no_vcs_prompt() {
    PROMPT='the-prompt-you-want-in-non-vcs-directories'
}

Of course, you can also put something completely different in those functions.

I fixed "refined" or "pure" using the following hack. If anyone has something more elegant, please provide it.
refined.zsh-theme
repo_information() {
    inside_git_repo="$(git rev-parse --is-inside-work-tree 2>/dev/null)"
    if [ "$inside_git_repo" ]; then
        echo "%F{blue}${vcs_info_msg_0_%%/.} %F{8}$vcs_info_msg_1_`git_dirty` $vcs_info_msg_2_%f"
    else
        echo "%F{blue}%~%f"
    fi
    # echo "%F{blue}%~%F{blue}${vcs_info_msg_0_%%/.} %F{8}$vcs_info_msg_1_`git_dirty` $vcs_info_msg_2_%f"
}


Old PR: feat(robot-server): add maintenance run resource and routes

(#12529 is taking over for this PR. Will close this PR once pending reviews on this PR are posted. They will be addressed in the new PR)
Closes RSS-199
Overview
This PR is taking over from previous #12392 to reduce merge conflicts between @sanni-t 's and @TamarZanzouri 's branches while working on this.
Adds a /maintenance_runs resource to run maintenance procedures like LPC and instrument attach/detach/calibration.
Test Plan

[ ] Use the /maintenance_runs endpoints to create a maintenance run, issue commands to the run, delete the run.
[ ]  Create protocol run while a maintenance run is active, it should delete the maintenance run and create the protocol run successfully.
[ ] Send a new maintenance run creation request while a maintenance run is already active; the old one should get replaced by the new one.
[ ] Start running a protocol run and issue a maintenance run creation request; it should throw an error and not allow creating the maintenance run.
[ ] While a maintenance run is active when a protocol run is idle, issue a PLAY action on the protocol run; the maintenance run should get removed and the protocol run should start playing.
[ ] Test uploading a protocol, running LPC and running the protocol via the app. The changes in this PR should not affect any existing behavior of the /runs resource.

Changelog

Adds a new maintenance runs resource that implements a base router, a commands router and a labware router over /maintenance_runs route
Adds supporting classes of MaintenanceEnginestore and MaintenanceRunDataManager that are very similar to the ones used by the /runs resource but without the run store and protocol management.
Adds maintenance run deletion when PLAY action is issued over runs endpoint.

Review requests

Make sure code looks good and has good unit test coverage
Make sure the above tests in the plan succeed as expected
The intention of approach we took in this PR - duplication of a lot of code between the runs & maintenance runs resource- was done on purpose to do a first pass of run split without affecting any code related to protocol runs. We expected that we would eventually be able to compress some of the code into a shared resource and I think this PR proves that we can, and we should, but I propose we work on that refactor in a follow-up PR. This PR establishes a base for splitting the run and builds client-server expectations on managing two separate runs. The follow-up will work on code-management and finer details of the client-server interactions while using such separate runs.

Follow-up tickets:

RSS-223
RSS-224
RSS-225

Risk assessment
Low-None for existing infrastructure since this adds a new resource that isn't used by anything yet.

Tested all scenarios on OT-3 and it works great! will review the code as well

[x] Use the /maintenance_runs endpoints to create a maintenance run, issue commands to the run, delete the run.

[x] Create protocol run while a maintenance run is created and send it (the protocol run) a play action, it should delete the maintenance run and start running the protocol run successfully.

[x] Send a new maintenance run creation request while a maintenance run is already active; the old one should get replaced by the new one.

[x] Start running a protocol run and issue a maintenance run creation request; it should throw an error and not allow creating the maintenance run.

[x] Test uploading a protocol, running LPC and running the protocol via the app. The changes in this PR should not affect any existing behavior of the /runs resource.

Superseded by #12529