User Registration - "email address already in use" when it is unique

Hello,
I started with just devise and now am integrating this gem and ng-token-auth.  I got sign in working, but for sign up, the API constantly returns email is not unique, when I can create a new user in the console with exact credentials.
UserRegistrationsController registration form submit function
$scope.submitRegistration = function (credentials){
      credentials.password_confirmation = reg_form.password;
      credentials.provider = 'email';

      $auth.submitRegistration(credentials).then(function(resp) {
      console.log(resp);
    }).catch(function(resp) {
      // handle error response
      console.log(resp);
    });
}

I am customizing the registration controller because I need to gate users with a code for private beta release, so I set a custom route to point to the controller:
routes.rb
namespace 'api' do
    scope 'v1' do
      mount_devise_token_auth_for 'User', at: 'auth', defaults: {format: :json}, controllers: {
        ....
        registrations:      'token_auth_override/registrations', #CUSTOM
        ....
      }
    end
end

I am setting the strong params in the application_controller, as recommended in the docs both here and devise:
application_controller.rb
class ApplicationController < ActionController::Base
  include ActionController::MimeResponds
  protect_from_forgery

  respond_to :html, :json
  before_action :configure_permitted_parameters, if: :devise_controller?

  protected

  def configure_permitted_parameters
     devise_parameter_sanitizer.for(:sign_up) { |u| u.permit(:alias, :invite_code, :first_name, :last_name, :email, :password, :password_confirmation, :bio, :youtube_url, :personal_url, :twitter_url, :facebook_url, :user_profile_pic, :provider) }
  end
end

registration request from the client hit this controller in
token_auth_override/registrations_controller.rb
class TokenAuthOverride::RegistrationsController < DeviseTokenAuth::RegistrationsController
	include DeviseTokenAuth::Concerns::SetUserByToken
	before_action :configure_permitted_parameters, if: :devise_controller?

  respond_to :json
  def create

    invite_code = SignUpCode.find_by_code_string(params[:invite_code])
		if invite_code && invite_code.active?
			@user = User.new(
				email: params[:email],
				first_name: params[:first_name],
				last_name: params[:last_name],
				provider: params[:provider],
				password: params[:password],
				password_confirmation: params[:password_confirmation]
			)
			@user.level = invite_code.level

			if @user.save
				# sign user in
				sign_in(@user)
				invite_code.increment
				respond_to do |format|
					format.json { render json: @user }
				end
			else
				render json: {
          status: 'error',
          errors: @user.errors,
          data: @user,
          valid: @user.valid?
        }, status: 403
			end
		else
			# message that code is invalid
			respond_to do |format|
					format.json { render json: {message: 'Invite code invalid'}, status: :precondition_failed }
			end
		end
  end
end

json response looks something like this:
data: {id: null, email<EMAIL_ADDRESS>created_at: "2015-05-01T23:21:04.492Z",…}
errors: {email: ["This email address is already in use"]}
status: "error"
valid: false

Oddly, if i remove include DeviseTokenAuth::Concerns::User in my user.rb model, I do not run into this error.  I'd be happy to provide any additional snippets.
Thanks in advance for any help/thoughts.  Been struggling to get this working for a while now...

going to close this since I've gone with the path of building oauth functionality from scratch

$scope.submitRegistration = function (credentials){
credentials.password_confirmation = reg_form.password;
credentials.provider = 'email';
  $auth.submitRegistration(credentials).then(function(resp) {
  console.log(resp);
}).catch(function(resp) {
  // handle error response
  console.log(resp);
});

}

<EMAIL_ADDRESS>


fix: add more prisma keywords to denylist Fixes #12332

Fixes - https://github.com/prisma/prisma/issues/12332
Adding more keywords to the deny list
Related PR - https://github.com/prisma/prisma-engines/pull/2788

Closing as the underlying issue is fixed since Prisma 5.1.0 https://github.com/prisma/prisma/issues/12332#issuecomment-1640446135


Mil-Std-2525C Tasks (Isolate and Retain) are not rendering correctly in World Wind

The isolate and retain graphics are being rendered incorrectly in World Wind. I tested them in Cesium and they render correctly. The graphics are being rendered in a square like shape instead of a circular shape.

Closing this issue, See #48 for resolution and testing


Handle tables with no primary keys

It it quite common to have tables with no primary key, here are a few use cases:

Time series data (specially with TimescaleDB)
One-to-many where individual elements are never references

It would be nice it was supported by sea-orm. At first I would be totally fine by just rendering the Column so we can use it with sea-query and later down the line do a better integration. Currently it is a bit annoying because we cannot ignore it in the CLI so effectively either you remove the entity after each run or you add a primary key.

Currently it is a bit annoying because we cannot ignore it in the CLI so effectively either you remove the entity after each run or you add a primary key.

Hey @Sytten, do you mean generating entities of table without primary key will cause errors?

That might be a bit tricky. I always have an assumption that for a table without primary key, the DB engine implicitly adds one and hides it from you anyway.
A table without primary key breaks many assumptions about the concept of "Entity", and hence methods like find_by_id is not possible too.
If we need to support it within SeaORM, we might have to invent a new concept, Datum I guess? (just some random name poping up in my mind)

You are partially right. Databases in fact have a row ID no matter if you have a primary key or not.
Yeah it is probably hard to support in an ORM. I think my wish would mostly be to be able to still generate some entities for it (at least the columns and models) so they are usually with sea-query and dont break the build.
This would fit with my other issue on providing something in between the raw query and the orm.

Hey @Sytten, you can select the Datum like below.
#[derive(Debug, Clone, FromQueryResult)]
struct JoinedResult {
    ...
}

let builder = self.db.get_database_backend();

let mut stmt = sea_query::Query::select();

stmt.columns([stuff::Column::Name, stuff::Column::CreatedAt])
    .column(status::Column::Status)
    .from(stuff::Entity)
    .join(JoinType::InnerJoin, status::Entity, Cond::any());

JoinedResult::find_by_statement(builder.build(&stmt))
    .all(&self.db)
    .await

Adapted from a Discord message by jochen#5064

Good idea! We could simply generate a Model struct and Column enum.

That might be a bit tricky. I always have an assumption that for a table without primary key, the DB engine implicitly adds one and hides it from you anyway.

In my case it was very easy to create a Sqlite table without a pk using sqlite-utils and then it would still have a ROWID. It's also easily fixed, but there's nothing in sqlite that requires you to explicitly have a primary key.

I'm trying to produce a better error message at compile time.

https://github.com/SeaQL/sea-orm/pull/1020

It would still fail to build when compiling the codegen code directly without any manual modification. Maybe we could add settings in the codegen CLI to either ignore the tables or treat them differently (like only build the columns enum).

+1 for this solution, as I ran into the same issue today.
Should I open a new issue to track the cli changes or can this be reopened? It is unclear to me why this was closed by #1020.

what was the solution here? i would like to avoid creating another index if possible, is it possible to tell the entity that it should treat some other index as a "primary" key?

Hey @databasedav, every table is assumed to have a primary key. For table without a primary key, I would suggest doing one of the following:

Adding a autoincrement primary key column (if it's possible to alter the schema)
Or, let all column in the table be a composite primary key, i.e.

#[derive(Clone, Debug, PartialEq, DeriveEntityModel, Eq)]
#[sea_orm(table_name = "cake_filling")]
pub struct Model {
    #[sea_orm(primary_key, auto_increment = false)]
    pub cake_id: i32,
    #[sea_orm(primary_key, auto_increment = false)]
    pub filling_id: i32,
}

Hey @databasedav, every table is assumed to have a primary key. For table without a primary key, I would suggest doing one of the following:

Adding a autoincrement primary key column (if it's possible to alter the schema)
Or, let all column in the table be a composite primary key, i.e.

#[derive(Clone, Debug, PartialEq, DeriveEntityModel, Eq)]
#[sea_orm(table_name = "cake_filling")]
pub struct Model {
    #[sea_orm(primary_key, auto_increment = false)]
    pub cake_id: i32,
    #[sea_orm(primary_key, auto_increment = false)]
    pub filling_id: i32,
}

Although this works, what about for users who have hundreds of Models.
Is there scope to add this into sea-orm-cli

let all column in the table be a composite primary key

@vablings thank you for the suggestion! actually it seemed to be a very smart way, and the only drawback probably is the max. number of columns is 12

I am trying to make use of https://github.com/pgpartman/pg_partman/blob/master/doc/pg_partman_howto.md#simple-time-based-1-partition-per-day which wants me to create a parent table without having a primary key.
But in that case I am getting a lot of
 the trait `sea_orm::IdenStatic` is not implemented for `entities::transfer::PrimaryKey`
``

errors even for other entities that have primary keys :(


Fixing Broken Link / Updating Information

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/2423

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@tonysmit  PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Gzip decompress Unexpected EOF with small buffer + fix

Steps to reproduce

Gziped 38KB file

Trying to ungzip it supplying into GZipInputStream.Read buffer with the size 10KB (smaller than gzipped array itself). Receive "Unexpected EOF".

If to use buffer 50KB (more than gzipped array) - all works.

It started to work with smaller buffer when the part in GZipInputStream.Read was remarked:

Code for decompressing

public static byte[] Decompress(this byte[] data)
{
    if (data == null)
        return null;

    byte[] unzipped = new byte[10000];
    int readOut = 0;
    byte[] res = null;

    try
    {
        using (Stream stream = new GZipInputStream(new MemoryStream(data)))
        {
            using (MemoryStream memory = new MemoryStream())
            {
                while((readOut = stream.Read(unzipped, 0, unzipped.Length))>0)
                    memory.Write(unzipped, 0, readOut);

                res = memory.ToArray();
                memory.Close();
            }
            stream.Close();
        }
    }
    catch
    {
        res = null;
    }
    return res;
}

Expected behavior
Unzip
Actual behavior
Unexpected EOF
Version of SharpZipLib
Obtained from (only keep the relevant lines)

Compiled from source, commit: 6f245b3

It looks like the ReadWriteThroughtput GZip unit test fails with the same exception (that test is marked as explicit, so it isn't run by default).
Looks like that throw was only added in https://github.com/icsharpcode/SharpZipLib/pull/225/commits/a35ce40d7784b408ac0c4108e77bf897bd3df90a, don't know if removing it reverts part of that fix.

Yeah, this needs to be resolved before 1.2, something is clearly not behaving as expected. Why would a smaller buffer make that condition return false?

Possibly it's incorrectly throwing in cases where the inflator has used up its own internal buffer, but the input stream still has data left? (maybe it should only be doing that check if bytesRead is 0?)

Yes, that could be it. Due to confusion about what inf.RemainingInput refers to...

Changing it to throw if inf.RemainingInput and bytesRead are both 0 fixes the test failure (e.g.
else if (inf.RemainingInput == 0 && bytesRead == 0)

Not certain if that is the right fix though (I hadn't previously realised that it explicitly tries to not throw in cases where there is garbage in the stream after the end of the valid data).

Okay, I finally know whats going on!

There is no more Input to read, since all the bytes have been read, but no more bytes will be written to the output buffer since it's full. So a simple fix would be to check if the buffer is full (bytesRead == count). If the buffer sizes exactly matches up with the needed bytes I think the next read should be 0 bytesRead, but some more testing is needed.


Add futures::stream::unfold

This adds an adapter, a bit similar to Stream::fold, which allows to go from the "world of futures" to the "world of streams".
I felt such adapter was missing (or maybe I just missed it) when trying to use tokio-core, because most helper functions (like io::write_all or io::read_exact) return Futures and didn't work well with streaming protocols (using recursion with .and_then() finished by overflowing the stack).
This adapter allows to write code like:
fn handle_client<T: Read>(reader: T, handle: &Handle) -> Receiver<Vec<u8>> {
    let (tx,rx) = std::sync::mpsc::channel();
    let stream = repeat(reader, |reader| {
        // Read a length-prefixed message
        io::read_exact(reader, [0u8;8]).and_then(|(reader, buffer)| {
            let next_msg_size = buffer.read_u64::<LittleEndian>().unwrap() as usize;
            io::read_exact(reader, vec![0; next_msg_size])
        })
    });
    let future = stream.for_each(|message| {
        tx.send(message).unwrap();
    }).map_err(|e| println!("Error: {}", e));

    handle.spawn(future);
    rx
}

I am not too happy with the naming (repeat could mean something else) and with the example in the documentation, so I will be happy to change that.
We could also try to have a way to get back ownership of the internal state (right now it is possible by integrating it the Error type for example, but it looks ugly)

Thanks for the PR! I wonder if we could perhaps think of a different name for this? It seems similar to std::iter::repeat but prety different as well? Right now it looks similar-ish in functionality at least to Iterator::scan

My apologies for the delay, I just realized that I actually forgot to send my reply to you ...
I agree the name isn't great because it is close to other functions with different semantics.
I tried to see if I could find a similar functionality in other programming languages for inspiration, but I didn't find anything.
In essence, this functions creates a Stream out of thin air by repeating the same action (future) and waiting for it to complete, hence my first name idea repeat. I had also thought of generate_stream, but I didn't find it good either. I will keep thinking about names, but if you have any suggestions please let me know.
Apart from the name, do you think the functionality offered is useful (and has a place in the futures crate)?

Oh no worries!
This feels similar to an infinite stream with a map or perhaps something that's just FnMut() -> Option<Future<T>>. Could you elaborate on why T is passed in explicitly? I'm also still a little undecided on the name, but I think we'd want to reserve the name repeat at the very least.
I'm also totally fine adding this to this crate, seems like quite appropriate functionality to me!

The idea was to be able to pass ownership of a (potentially unclonable) object (such as a socket). The user provides the initial T, that we give to the closure, which returns a Future. The returned Future must resolve in a tuple (T, Item). T is then fed back into the closure, and Item is yielded by the Stream.
I took this approach because it fits well with some tokio functions, like tokio::io::read_exact<A, T>(a: A, buf: T) which returns a future that will resolve to (a, buf), thus giving back ownership of the socket/buffer.
I think a major disadvantage is that it may result in another "level" of tuple, already commonly found in tokio. For example, if the user wants the stream to yield (A, B) tuples, the Future created by the closure will need to return a (T, (A, B)).
As you pointed out, another design could be not to pass that state around (and have FnMut() -> Option<Future<T>>), but this may force users to use Rc<RefCell<T>> or TaskRc<RefCell<T>> in places where it may not be necessary, going a bit further from zero-cost abstractions.

Hm the state here is a little different than read_exact though where you never get it back out. That is, the internal state is locked into the stream forever (not yielded as an item). In that sense it could just be captured and closed over by the FnMut, right?
That may require an Option in some places to deal with take and unwrap, but that's what this implementation is doing anyway. I don't think it'll require Rc and/or internal mutability.

But if the state was closed over by the FnMut (as an Option as you suggest) how would the Future returned by the FnMut put it back into the closure's environment?
I figured I would end up having something like (pseudocode):
struct Repeat {
    f: F,
    future: Future,  // Future contains a reference to a part of f environment
}

Which would be rejected by Rust (can't have self-referencing structures). This is why I made the future explicitely return back the ownership of T.

Aha! I see what you mean now, that is indeed quite subtle. I wonder if perhaps this is almost more worth it as a combinator rather than a base construction of a stream. That is, an and_then like combinator which plumbs state through the various futures and such. What do you think?

Yes, I guess that should work! I could then have the functionality I originally wanted with something like futures::stream::iter(std::iter::repeat(())).and_then_state(initial_state, |state, _| { ... });
I'll try to work on the implementation tonight (and try to find a good name, although I am very bad at naming).
As a side note, wouldn't your previous comment about the state possibly being captured in the closure's environment apply to the std::iter::scan() iterator as well?

I'll try to work on the implementation tonight (and try to find a good name, although I am very bad at naming).

Ok! We can figure out naming afterwards, that's fine.

As a side note, wouldn't your previous comment about the state possibly being captured in the closure's environment apply to the std::iter::scan() iterator as well?

Indeed! I was never a fan of scan as an iterator...

OK the implementation is mostly done and revealed an interesting edge-case that I overlooked before. In case of error returned by the user-returned future, we would lose the internal state with no way to get it back (the future should have returned it in the Ok() variant). So I chose the option of returning Ok(Async::Ready(None)) because it is the only sane thing to do. This edge-case would disappear if polling an error meant the end of the stream, as suggested in #206.
I also added a few tests.
The Travis failure looks unrelated to this PR.

@aturon you wouldn't happen to recognize this combinator would you? I can't quite put my finger on it but it seems like something we've seen before...

@alexcrichton This combinator is sometimes known as "unfold", where you take a seed and a transformation, and produce from it a stream of values.

Oh that sounds pretty nifty! @Vaelden want to add this as Stream::unfold and rename appropriately?

@aturon I would just like to double-check, because since I started this PR I changed the implementation but not the accompanying docs (which I will do ASAP): is "unfold" an adapter, taking a stream of values, a seed and a transformation and creating a new stream; or does it produce a stream from nothing (well ... seed + transformation)?

@Vaelden It produces a stream from nothing, just as you have it here. Here's a similar function in Haskell. You can think of it as the "reverse" of fold.

Ah ok so given that, @Vaelden I wonder if it might be best to go back to what you had originally, not as a combinator on the Stream trait? Sorry for the runaround!

@aturon Thanks a lot for the link! It helped me a lot for the documentation.
@alexcrichton That's fine no worries! I changed the name, rewrote the doc and rebased on top of master so everything should be fine now (except the name of my base branch that I cannot change from repeat_adapter without creating another PR).

Oh and I also changed the order of the item in the tuple to match Haskell's implementation: the Future must return (item, next_state) instead of (next_state, item) as in the beginning of this PR.

Thanks @Vaelden!


feat(#60): Defect.version

In this pull I've added new method to the Defect interface: version(), that returns version of the JAR from pom.xml file.
closes #60
History:

feat(#60): maven-model
feat(#60): passes
feat(#60): clean for pmd

@yegor256 take a look, please

@h1alexbel this won't work in production, because pom.xml doesn't exist in the JAR file. See how we've implemented this functionality in another place recently: https://github.com/objectionary/jeo-maven-plugin/pull/899

@yegor256 thanks, I will try it

@yegor256 updated pull request. Take a look again, please

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here.

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 3min).


Add tags_to_ignore flag to map_operations and merge_operations transformer primitives

As requested in https://github.com/quantumlib/Cirq/issues/4923

Should also be useful to simplify https://github.com/quantumlib/Cirq/pull/4933
cc @daxfohl


Self identify requests coming from Fairing SDK

We need to add some instrumentation to identify requests/job submissions to KF and other backends to enable debugging and monitoring on the backend side. An example would be setting the user-agent in the http requests to "fairing". This enable the ops team to find if there is a specific version of Fairing causing any issues and also to deprecate older versions.

CC @abhi-g @kramachandran


Fixed music controller always giving error

Also took the time to refactor musicController to use an API library, making the controller entirely focus on the controlling logic.

I think you referenced it with musicController while the actual name is MusicController?

It's called musicController in my project files, and not MusicController... 🤷♂


Storage: Refine logging about reading snap, place index, index to wait

What problem does this PR solve?
Issue Number: ref https://github.com/pingcap/tiflash/issues/8864
Problem Summary:
What is changed and how it works?

Add the index to wait in learner read logging
Add logging about place index result and snapshot rows/deletes

[2024/04/21 00:33:40.417 +08:00] [DEBUG] [Segment.cpp:2422] ["Begin segment getReadInfo <segment_id=1 epoch=1 range=[-9223372036854775808,9223372036854775807)>"] [thread_id=1]
[2024/04/21 00:33:40.418 +08:00] [DEBUG] [StableValueSpace.cpp:487] ["start_ts:<PHONE_NUMBER>3709551615, enable_handle_clean_read: false, is_fast_mode: false, enable_del_clean_read: false"] [source="segment_id=1 epoch=1"] [thread_id=1]
[2024/04/21 00:33:40.418 +08:00] [DEBUG] [DMFilePackFilter.cpp:118] ["RSFilter exclude rate: 0.00, after_pk: 2, after_read_packs: 2, after_filter: 2, handle_ranges: {[1,9223372036854775807)}, read_packs: 0, pack_count: 2"] [thread_id=1]
[2024/04/21 00:33:40.454 +08:00] [DEBUG] [Segment.cpp:2682] ["Finish segment ensurePlace, read_ranges={[-9223372036854775808,9223372036854775807)} placed_items=1 shared_delta_index=<placed_rows=0 placed_deletes=0 tree_entries=0 tree_inserts=0 tree_deletes=0> my_delta_index=<placed_rows=8192 placed_deletes=0 tree_entries=8192 tree_inserts=8192 tree_deletes=0> <segment_id=1 epoch=1 range=[-9223372036854775808,9223372036854775807)>"] [thread_id=1]
[2024/04/21 00:33:40.455 +08:00] [DEBUG] [Segment.cpp:2449] ["Finish segment getReadInfo, my_delta_index=<placed_rows=8192 placed_deletes=0 tree_entries=8192 tree_inserts=8192 tree_deletes=0> fully_indexed=true read_ranges={[-9223372036854775808,9223372036854775807)} snap={stable_rows=16384 persisted_rows=8192 persisted_dels=0 persisted_cfs=1 mem_rows=0 mem_dels=0 mem_cfs=0} <segment_id=1 epoch=1 range=[-9223372036854775808,9223372036854775807)>"] [thread_id=1]
[2024/04/21 00:33:40.455 +08:00] [DEBUG] [Segment.cpp:2461] ["Segment updated delta index, my_delta_index=<placed_rows=8192 placed_deletes=0 tree_entries=8192 tree_inserts=8192 tree_deletes=0> <segment_id=1 epoch=1 range=[-9223372036854775808,9223372036854775807)>"] [thread_id=1]

Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests


Liquibase doesn't recognize Database "SAP IQ" on update

Environment
Liquibase Version: 4.4.3
Liquibase Integration & Version: CLI
Database Vendor & Version: Sybase SAP IQ
Operating System Type & Version:
Windows, Linux.
Description
It appears that Liquibase doesn't recognize Database "SAP IQ".
Steps To Reproduce
Have the following properties setup with jconn4.jar jdbc driver
url: jdbc:sybase:Tds:localhost:2638/iqdemo
username: dba
password: password
classpath: path/to/jconn4.jar
driver: com.sybase.jdbc4.jdbc.SybDataSource

Actual Behavior
The following message appears.
instance liquibase.database.core.UnsupportedDatabase
[2021-08-18 15:38:08] FINE [liquibase.database] Properties:
[2021-08-18 15:38:08] FINE [liquibase.database] Key:'user' Value:'user1'
[2021-08-18 15:38:08] FINE [liquibase.database] Key:'password' Value:'**********'
[2021-08-18 15:38:08] FINE [liquibase.database] Connecting to the URL:'jdbc:sybase:Tds:hostname:2638/devdb' using driver:'com.sybase.jdbc4.jdbc.SybDataSource'
[2021-08-18 15:38:08] FINE [liquibase.servicelocator] Loaded liquibase.database.DatabaseConnection instance liquibase.database.jvm.JdbcConnection
[2021-08-18 15:38:08] FINE [liquibase.servicelocator] Loaded liquibase.database.DatabaseConnection instance com.datical.liquibase.ext.database.jvm.ProJdbcConnection
[2021-08-18 15:38:08] FINE [liquibase.database] Connection has been created
[2021-08-18 15:38:09] WARNING [liquibase.database] Unknown database: SAP IQ

This later causes issues with setting the DBCL LOCK table with incompatible bool datatype instead of bit type since the default fallback database type is the "UnsupportedDatabase".
Expected/Desired Behavior
Liquibase should be able to identify the SAP IQ database and handle the tracking tables correctly.

Hi @szandany Thanks for writing up this issue.  We will add it the list of issues that we are processing.

I am also facing similar issue. What is plan to rollout fix for this.  Please let me know if there is tentative timeline.
Also please help to know if there is workaround till permanent fix is available.

I am also facing similar issue.
@prashantsawant06 do you find workaround fix this problem?
Starting Liquibase at 16:53:25 (version 4.28.0 #2272 built at 2024-05-16 19:00+0000)
Liquibase Version: 4.28.0
Liquibase Open Source 4.28.0 by Liquibase
ERROR: Exception Details
ERROR: Exception Primary Class:  SybSQLException
ERROR: Exception Primary Reason:  SQL Anywhere Error -131: Syntax error near 'BOOLEAN' on line 1
ERROR: Exception Primary Source:  SAP IQ SAP IQ/<IP_ADDRESS>12/14315/P/SP05.12/Linux/Linux64 - x86_64 - 3.10.0-327/64bit/2024-03-04 01:37:20
Unexpected error running Liquibase: SQL Anywhere Error -131: Syntax error near 'BOOLEAN' on line 1  [Failed SQL: (102) CREATE TABLE gestaocontabil.DATABASECHANGELOGLOCK (ID INT NOT NULL, LOCKED BOOLEAN NOT NULL, LOCKGRANTED datetime, LOCKEDBY VARCHAR(255), CONSTRAINT PK_DATABASECHANGELOGLOCK PRIMARY KEY (ID))]


https://aws-otel.github.io/ copyright need to be updated to 2023 year.

Hi! As of today copyright at the bottom of the page is: "© 2020–2021 Amazon Web Services" - makes sense to update to "© 2020–2023 Amazon Web Services".

Closing this issue as the relevant PR has been merged: https://github.com/aws-otel/aws-otel.github.io/pull/569


[Bug] libtvm4j.so is not created in android project.

Thank you for everyone!
Finally, I success build. but i got new problem.
Please help me.
🐛 Bug
I restored "tvm_runtime.h" and success android build.
But i got android runtime error.
My project is not created 'libtvm4j.so'.
W/System.err: [WARN] Couldn't find native library tvm4j.
W/System.err: java.lang.UnsatisfiedLinkError: Couldn't find the resource libtvm4j.so
To Reproduce
Steps to reproduce the behavior:

Try to build android project.

Expected behavior
Working android app.
Environment
Platform (e.g. WebGPU/Vulkan/IOS/Android/CUDA): cuda
Operating system (e.g. Ubuntu/Windows/MacOS/...): ubuntu
Device (e.g. iPhone 12 Pro, PC+RTX 3090, ...) GTX 1080 TI
How you installed MLC-LLM (conda, source): source
How you installed TVM-Unity (pip, source): source
Python version (e.g. 3.10): 3.10.9
GPU driver version (if applicable): latest ( Iinstalled 1 month ago.)

What was the resolution for it?  I'm facing the same issue, @junrushao

What was the resolution for it? I'm facing the same issue too. @junrushao @chensinit

same here @junrushao @chensinit


A bit confusing regarding TLS 1.2

Hello!
Been troubleshooting the better part of today on why TLS 1.2 is not working when forcing only TLS 1.2 in the my.cnf file, we have followed the guide you have "Deploy highly available MySQL Databases" which uses Bitnami and their templates which is using Mysql Server version: 5.7.22-log MySQL Community Server. When enabling TLS 1.2 and only  TLS 1.2 you get the warning message in the mysql log: Failed to set up SSL because of the following SSL library error: TLS version is invalid
I found someone who had a similar problem here: https://github.com/docker-library/mysql/issues/567 which states:
This is a limitation in yaSSL (which is what the community edition of 5.7 uses currently); It only supports 1.1... So if following the "Deploy highly available MySQL databases" guide that you have the: Ensure that all hosting servers are configured for communication using TLS 1.2.  part is not applicable. Maybe needs some clarification :)?

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

ID: 9da7c95f-1749-8818-f96f-c4ae02aa28a6
Version Independent ID: db6e924d-d8c0-72bc-b0e3-382373ee7588
Content: Add MySQL hosting servers in Azure Stack
Content Source: azure-stack/operator/azure-stack-mysql-resource-provider-hosting-servers.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@azscloudpro Thanks for the feedback. I am following up now to get more information.

@azscloudpro We are looking further into this. TLS 1.1 will work in this scenario.

Thank you for following up!

@azscloudpro I am updating the doc. Thanks! Matt
#please-close


No module named "excel_ui"

HI!
I installed FlowCal following the directions on the website but I get the following error once I try to run it:
ImportError                               Traceback (most recent call last)
 in ()
----> 1 import FlowCal
/Users/mwehrs/anaconda/lib/python3.5/site-packages/FlowCal-1.1.3-py3.5.egg/FlowCal/init.py in ()
10
11 import io
---> 12 import excel_ui
13 import gate
14 import transform
ImportError: No module named 'excel_ui'
It seems to be installed tho.
mwehrs-mr:FlowCal-master mwehrs$ python setup.py install
running install
running bdist_egg
running egg_info
writing top-level names to FlowCal.egg-info/top_level.txt
writing requirements to FlowCal.egg-info/requires.txt
writing dependency_links to FlowCal.egg-info/dependency_links.txt
writing FlowCal.egg-info/PKG-INFO
reading manifest file 'FlowCal.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'FlowCal.egg-info/SOURCES.txt'
installing library code to build/bdist.macosx-10.6-x86_64/egg
running install_lib
running build_py
creating build/bdist.macosx-10.6-x86_64/egg
creating build/bdist.macosx-10.6-x86_64/egg/FlowCal
copying build/lib/FlowCal/init.py -> build/bdist.macosx-10.6-x86_64/egg/FlowCal
copying build/lib/FlowCal/excel_ui.py -> build/bdist.macosx-10.6-x86_64/egg/FlowCal
copying build/lib/FlowCal/gate.py -> build/bdist.macosx-10.6-x86_64/egg/FlowCal
copying build/lib/FlowCal/io.py -> build/bdist.macosx-10.6-x86_64/egg/FlowCal
copying build/lib/FlowCal/mef.py -> build/bdist.macosx-10.6-x86_64/egg/FlowCal
copying build/lib/FlowCal/plot.py -> build/bdist.macosx-10.6-x86_64/egg/FlowCal
copying build/lib/FlowCal/stats.py -> build/bdist.macosx-10.6-x86_64/egg/FlowCal
copying build/lib/FlowCal/transform.py -> build/bdist.macosx-10.6-x86_64/egg/FlowCal
byte-compiling build/bdist.macosx-10.6-x86_64/egg/FlowCal/init.py to init.cpython-35.pyc
byte-compiling build/bdist.macosx-10.6-x86_64/egg/FlowCal/excel_ui.py to excel_ui.cpython-35.pyc
byte-compiling build/bdist.macosx-10.6-x86_64/egg/FlowCal/gate.py to gate.cpython-35.pyc
byte-compiling build/bdist.macosx-10.6-x86_64/egg/FlowCal/io.py to io.cpython-35.pyc
byte-compiling build/bdist.macosx-10.6-x86_64/egg/FlowCal/mef.py to mef.cpython-35.pyc
File "build/bdist.macosx-10.6-x86_64/egg/FlowCal/mef.py", line 22
except ImportError, e:
^
SyntaxError: invalid syntax
byte-compiling build/bdist.macosx-10.6-x86_64/egg/FlowCal/plot.py to plot.cpython-35.pyc
File "build/bdist.macosx-10.6-x86_64/egg/FlowCal/plot.py", line 62
except ImportError, e:
^
SyntaxError: invalid syntax
Any idea on how to fix it?

Hmmmm. My initial guess is that you are using Python 3 instead of Python 2. Can you check that you are using Python 2.7?

Hi !
yes I am using python3. Is there a way to make it compatible with python3?
Thanks
Maren

On May 30, 2017, at 11:24 AM, John T. Sexton<EMAIL_ADDRESS>wrote:
Hmmmm. My initial guess is that you are using Python 3 instead of Python 2. Can you check that you are using Python 2.7?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/taborlab/FlowCal/issues/256#issuecomment-304965495, or mute the thread https://github.com/notifications/unsubscribe-auth/AbuQ0HVztvz6C3K6bE_X7XZkixPsdZECks5r_F7HgaJpZM4NqpIb.

Python 3
FlowCal is currently built on top of Python 2.7. There has been talk of supporting Python 3 (see #100), but I don't think that will happen any time soon.
There may be tools available online that can port Python 2.7 code to Python 3 code that you could try, but I can't vouch for any of them.
Virtual Environments
If you're using Anaconda, as recommended in the FlowCal documentation, but you need Python 3 for other applications, I would recommend that you take advantage of Anaconda's virtual environments to install Python 2.7 alongside Python 3 and run FlowCal out of the Python 2.7 environment.
Specifically, I use the Anaconda command prompt to install and run FlowCal:

On my Windows machine, in an Anaconda command prompt, I do the following:
# list current Anaconda virtual environments
> conda info -e

# create new Anaconda virtual environment for FlowCal version 1.1.3
> conda create -n FlowCalv1.1.3 python=2.7 anaconda

The -n FlowCalv1.1.3 names the virtual environment, python=2.7 instructs the Anaconda environment manager to set up a new environment on top of Python 2.7, and anaconda instructs the Anaconda environment manager to install all of the default packages that come with Anaconda in the new virtual environment (which includes widely used packages like numpy, matplotlib, and pandas).
Once the new Anaconda virtual environment has been created (which can take a few minutes), I download the most recent release of FlowCal, I unzip the downloaded file and navigate my Anaconda command prompt to the unzipped FlowCal/ folder (which should contain a setup.py file), I activate the new Anaconda virtual environment, and I manually perform the commands from the Install FlowCal (Windows).bat / Install FlowCal (OSX) scripts:
# activate new Anaconda virtual environment
> activate FlowCalv1.1.3

# install packages upon which FlowCal depends
(FlowCalv1.1.3) > python -m pip install -r requirements.txt --no-cache-dir

# install FlowCal
(FlowCalv1.1.3) > python setup.py install

If the install proceeds without errors, I can now use FlowCal from an Anaconda command prompt via either the native Python interpretor:
> activate FlowCalv1.1.3

(FlowCalv1.1.3) > python

>>> import FlowCal

or via the FlowCal Excel UI:
> activate FlowCalv1.1.3

(FlowCalv1.1.3) > python -m FlowCal.excel_ui -v -p -H

Thank you so much!
I think I got it to work now =)

On May 30, 2017, at 2:18 PM, John T. Sexton<EMAIL_ADDRESS>wrote:
Python 3
FlowCal is currently built on top of Python 2.7. There has been talk of supporting Python 3 (see #100 https://github.com/taborlab/FlowCal/issues/100), but I don't think that will happen any time soon.
There may be tools available online that can port Python 2.7 code to Python 3 code that you could try, but I can't vouch for any of them.
Virtual Environments
If you're using Anaconda, as recommended in the FlowCal documentation http://taborlab.github.io/FlowCal/getting_started/install_anaconda.html, but you need Python 3 for other applications, I would recommend that you take advantage of Anaconda's virtual environments https://conda.io/docs/using/envs.html to install Python 2.7 alongside Python 3 and run FlowCal out of the Python 2.7 environment.
Specifically, I use the Anaconda command prompt to install and run FlowCal:
https://cloud.githubusercontent.com/assets/2179825/26605262/fa4aa198-4552-11e7-9c92-067dfa29ea78.png
On my Windows machine, in an Anaconda command prompt, I do the following:
list current Anaconda virtual environments

conda info -e

create new Anaconda virtual environment for FlowCal version 1.1.3

conda create -n FlowCalv1.1.3 python=2.7 anaconda
The -n FlowCalv1.1.3 names the virtual environment, python=2.7 instructs the Anaconda environment manager to set up a new environment on top of Python 2.7, and anaconda instructs the Anaconda environment manager to install all of the default packages that come with Anaconda in the new virtual environment (which includes widely used packages like numpy, matplotlib, and pandas).

Once the new Anaconda virtual environment has been created (which can take a few minutes), I download the most recent release of FlowCal https://github.com/taborlab/FlowCal/releases, I unzip the downloaded file and navigate my Anaconda command prompt to the unzipped FlowCal/ folder (which should contain a setup.py file), I activate the new Anaconda virtual environment, and I manually perform the commands from the Install FlowCal (Windows).bat / Install FlowCal (OSX) scripts:
activate new Anaconda virtual environment

activate FlowCalv1.1.3

install packages upon which FlowCal depends
(FlowCalv1.1.3) > python -m pip install -r requirements.txt --no-cache-dir
install FlowCal
(FlowCalv1.1.3) > python setup.py install
If the install proceeds without errors, I can now use FlowCal from an Anaconda command prompt via either the native Python interpretor:

activate FlowCalv1.1.3

(FlowCalv1.1.3) > python

import FlowCal
or via the FlowCal Excel UI:

activate FlowCalv1.1.3

(FlowCalv1.1.3) > python -m FlowCal.excel_ui -v -p -H
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/taborlab/FlowCal/issues/256#issuecomment-305011422, or mute the thread https://github.com/notifications/unsubscribe-auth/AbuQ0KZjVMbLOJ8KyQOFsWW71D6kLGxQks5r_IeygaJpZM4NqpIb.

Great! I'm gonna go ahead and close this issue then. Feel free to open it back up, though, if you have any other installation issues.


Error in console

Dark theme works fine but I get this kind of invisible error in console, this appears only when a change is made to the editor like importing a script or build. I even tried reinstalling unity and other methods too, but still this occurs. I'm using unity version 2019.3.14f1 .

I solved this issue actually its a common unity error and not with this tool ;)


SAA-1885 bug fix, date range is wrong for adjudications

code was always passing 2 days range for some reason to prison api, inherited by adjudications now

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.19%. Comparing base (cf340ef) to head (ab9e3b0).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #975   +/-   ##
=========================================
  Coverage     84.19%   84.19%
  Complexity     3656     3656
=========================================
  Files           379      379
  Lines         12200    12200
  Branches       1064     1064
=========================================
  Hits          10272    10272
  Misses         1446     1446
  Partials        482      482

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


Add onKeyPress for Link component to respond Enter and Space by default for accessibility

Version
react-router-dom: 4.3.1
Test Case
https://codesandbox.io/s/24mwr7w1p
Steps to reproduce
I am trying to add accessibility. After I use the package reactjs/react-a11y, it gives some warnings for Link (NavLink) component.
Open the codesandbox, and click refresh first (otherwise will show error), then click Home link, you will see error

[react-a11y]: Link - You have an onClick handler but did not define an onKeyDown, onKeyUp or onKeyPress handler. Add it, and have the "Space" key do the same thing as an onClick handler. See 'https://www.w3.org/WAI/GL/wiki/Making_actions_keyboard_accessible_by_using_keyboard_event_handlers_with_WAI-ARIA_controls' for more info.

I know developer can always add onKeyPress by themselves.
But I am thinking does it make sense to add onKeyPress in Link component to handle Enter (default behavior) keyboard by default? And if developer provides its own onKeyPress, let it overwrites the default one.
If it makes sense, I can create a pull request.

Adding to this- Link has support for Enter keypresses but does not handle space, which goes against the ARIA guidelines for buttons.

Oh, this is still open.
@Hongbo-Miao:
I don't think this issue is going to be addressed. To mirror the comment I got on my PR:

Hey StringEpsilon, thanks for the PR. However, I do not believe that this is necessary because an anchor already works with the Enter key. The react-a11y warning seems more geared towards non-accessible elements. If the link has an onClick prop, it will be called when the user clicks navigates using the Enter key.

You can also pass a custom onKeyPress handler through Link to the underlying anchor, if you should need to:
<Link onKeyPress={() => { /* do accessible stuff */ }/>
<NavLink onKeyPress={() => { /* do accessible stuff */ }/>

NavLink and Link both pass down all props not specific to them down to the rendered anchor element.


spec_unix.go: remove unneeded TODO (moby/moby#35442)

Unmasking /sys/firmware was considered to be needed for LCOW, but it appeared to be not needed now moby/moby#35442
Signed-off-by: Akihiro Suda<EMAIL_ADDRESS>

Codecov Report

Merging #1745 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1745   +/-   ##
=======================================
  Coverage   48.58%   48.58%
=======================================
  Files          28       28
  Lines        4250     4250
=======================================
  Hits         2065     2065
  Misses       1750     1750
  Partials      435      435

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 17093c2...79e3b97. Read the comment docs.

LGTM


Enable running stress test for fetching binary snapshot in odsp driver.

Describe the bug
Enable running stress test for fetching binary snapshot in odsp driver.

Duplicate of https://github.com/microsoft/FluidFramework/issues/9011


[UI]

Description
Changes proposed
Made the navbar sticky on the y-axis scroll by adding the sticky class to the outermost div element and setting top-0 and bg-black classes for styling.
Updated the structure of the navbar by moving the image and text into a div with the flex and items-center classes to align them horizontally.
Added a hover effect to the navbar links by modifying the className of the Link component to include text-white hover:text-[#194fbc] duration-500.
Added the "Join Us" button to the list
Please assign it to me
Screenshots

Additional Information
No response

Hey there this is not required in the design
someone is already working on the underline effect in the navbar check this out #98


Move rake tasks into gem lib directory

Rather than have to vendor the gem to get access to the rake tasks, I wonder if you might follow a pattern I've seen with other gems (delayed_job and thinking_sphinx, specifically).  They have the actual rake tasks inside of a directory underneath lib, so that they're available once the gem has been loaded.  There is still a tasks file in the root of the gem directory, but that file simply loads the tasks definition from the lib directory.  With the rake tasks loaded, our application can then "require" them, without having to separately load a file containing those tasks.
Something to consider.  Vendoring the gem isn't ideal because it is so easy to update the gem but then not update the vendored version.  The approach that delayed_job and thinking_sphinx took avoids that issue.

Without this, whats the work around to make sure the rake tasks are available ?

Not an issue any more as they are under lib/ now.


Ability to render deterministic HTML (every time)

Previously it was impossible to test the HTML output in unit tests,  since attributes keys order was random every time.

Ok, I see the work is already in progress here: https://github.com/TokamakUI/Tokamak/pull/346  :)

Ok, I see the work is already in progress here: https://github.com/TokamakUI/Tokamak/pull/346  :)

Let's keep it open until the PR is merged 🙂

Let's keep it open until the PR is merged 🙂

Sorry for the confusion, while checking this from the GitHub mobile client I thought this was an issue, not a PR. Closing as a duplicate then 😅

Sorry for the confusion, while checking this from the GitHub mobile client I thought this was an issue, not a PR. Closing as a duplicate then 😅


import Stipple causes UndefVarError: @with_kw not defined

import Stipple
import StippleUI
import StipplePlotly

import Clustering
import RDatasets: dataset
import DataFrames

data = DataFrames.insertcols!(dataset("datasets", "iris"), :Cluster => zeros(Int, 150))
@Stipple.reactive mutable struct IrisModel <: Stipple.ReactiveModel
        iris_data::Stipple.R{import Stipple
import StippleUI
import StipplePlotly

import Clustering
import RDatasets: dataset
import DataFrames

data = DataFrames.insertcols!(dataset("datasets", "iris"), :Cluster => zeros(Int, 150))
@Stipple.reactive mutable struct IrisModel <: Stipple.ReactiveModel
        iris_data::Stipple.R{StippleUI.DataTable} = StippleUI.DataTable(data)
        credit_data_pagination::StippleUI.DataTablePagination = StippleUI.DataTablePagination(rows_per_page=50)

        features::Stipple.R{Vector{String}} = ["SepalLength", "SepalWidth", "PetalLength", "PetalWidth"]
        xfeature::Stipple.R{String} = ""
        yfeature::Stipple.R{String} = ""

        iris_plot_data::Stipple.R{Vector{StipplePlotly.PlotData}} = []
        cluster_plot_data::Stipple.R{Vector{StipplePlotly.PlotData}} = []
        layout::Stipple.R{StipplePlotly.PlotLayout} = StipplePlotly.PlotLayout(plot_bgcolor = "#ccc")

        no_of_clusters::Stipple.R{Int} = 3
        no_of_iterations::Stipple.R{Int} = 10
       end
ERROR: LoadError: UndefVarError: @with_kw not defined
in expression starting at Mixers.jl:79.DataTable} = StippleUI.DataTable(data)
        credit_data_pagination::StippleUI.DataTablePagination = StippleUI.DataTablePagination(rows_per_page=50)

        features::Stipple.R{Vector{String}} = ["SepalLength", "SepalWidth", "PetalLength", "PetalWidth"]
        xfeature::Stipple.R{String} = ""
        yfeature::Stipple.R{String} = ""

        iris_plot_data::Stipple.R{Vector{StipplePlotly.PlotData}} = []
        cluster_plot_data::Stipple.R{Vector{StipplePlotly.PlotData}} = []
        layout::Stipple.R{StipplePlotly.PlotLayout} = StipplePlotly.PlotLayout(plot_bgcolor = "#ccc")

        no_of_clusters::Stipple.R{Int} = 3
        no_of_iterations::Stipple.R{Int} = 10
       end
ERROR: LoadError: UndefVarError: @with_kw not defined
in expression starting at Mixers.jl:79

Could you please indicate your Stipple versions?

The latest (v0.23.0)

@Stipple.reactive mutable struct APP <: Stipple.ReactiveModel end

also fails:
ERROR: LoadError: UndefVarError: @with_kw not defined
in expression starting at Mixers.jl:79

Very interesting - looks like import Stipple messes some things up. using Stipple works as expected.
julia> import Stipple

julia> @Stipple.reactive mutable struct FooBar
       x::Int
       end
ERROR: LoadError: UndefVarError: @with_kw not defined
in expression starting at Mixers.jl:79

julia> Stipple.@reactive mutable struct FooBar
       x::Int
       end
ERROR: LoadError: UndefVarError: @with_kw not defined
in expression starting at Mixers.jl:79

julia> @reactive mutable struct FooBar
       x::Int
       end
ERROR: LoadError: UndefVarError: @reactive not defined
in expression starting at REPL[6]:1

julia> using Stipple

julia> @reactive mutable struct FooBar
       x::Int
       end
FooBar

julia> @Stipple.reactive mutable struct FooBar
       x::Int
       end
FooBar

I know that using works. The problem is that import does not work.

The reason is that the macro evaluates in the current name space, where the macro is not available.
Could you live with
import Stipple: @with_kw, @reactors, R, ChannelName

Stipple.@reactive mutable struct Example <: Stipple.ReactiveModel
    x::Stipple.Reactive{Int} = 1
end

That would work out of the box with the current version. We can easily reduce it to
import Stipple: @with_kw, @reactors

with minor corrections. I was not successful in removing the necessity of importing @with_kw in a quick fix @reactors

If found a fix in Mixers.jl that would make importing unnecessary. I'll supply a PR.

Frankly, there's little value in importing Stipple.
Making meaningful data apps requires using a pretty large API, and importing these by hand is counter productive. For instance, using Stipple automatically brings into scope all the Genie.Renderers.Html methods that are needed for building the pages.
@montyvesselinov What's your use case for going with import?

Anyhow, I filed a PR, let's see what the authors say.

I could understand, that some people don't want to populate their name space too heavily. Particularly, Genie.Html has some definitions like a, p, div, which easily collide.
But I agree with @essenciary that many imports are required to write nice code, although this maybe even more relevant to StippleUI than Stipple itself.

although this maybe even more relevant to StippleUI than Stipple itself

Yes good point. Also, the error is not nice, up to the users if they want to import.

a, p, div

Yes, div is problematic because of Base.div and because it didn't make sense to "specialise" a division method with one that does HTML generation. Maybe we could stop exporting it in Genie 5 and export htmldiv or hdiv or something. Or "specialise" Base.div? Specialising Base.div would be what the users expect, even if it's forced.
As for a and p - why are these problematic? Haven't had any clashes so they must be usually available.

If you do scientific computing there is some non-negligeable probability to have variables named a or p ...

A new version of Mixers.jl has been released with my proposed PR.
I updated Stipple accordingly.
With the latest master branch you now can do
julia> import Stipple

julia> Stipple.@reactive mutable struct Example <: Stipple.ReactiveModel
           x::Stipple.R{Int} = 1
       end
Example

Thank you!

Frankly, there's little value in importing Stipple.
Making meaningful data apps requires using a pretty large API, and importing these by hand is counter productive. For instance, using Stipple automatically brings into scope all the Genie.Renderers.Html methods that are needed for building the pages. Not to mention all the Stipple specific methods for adding reactivity. So while there is value in import not breaking, I can't see it as a useful feature.
@montyvesselinov What's your use case for going with import?

when you are dealing with many packages and writing big codes, the coding is much more readable and understandable when the "namespaces" (packages) are explicitly labeled.

I agree, but that's not necessarily tied to import. You can be using and still use the Fully.Qualified.Name for explicit namespacing.
But anyway, now that @hhaensel has sorted out the issue, import should also work.

"import" works! Thank you!


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 1.13.162 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


I can't run project in device after install firebase via Swift Packages

[REQUIRED] Step 1: Describe your environment

iOS version: 14.1
Xcode version: 12.1 (12A7403)
Firebase SDK version: 7.0.0
Installation method: Swift Package Manager` (select one)
Firebase Component: Analytics

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
I can't run project in device after install firebase via Swift Packages

Sorry about the issue.
This is a duplicate of #6626. See the discussion there and the ongoing discussion at #6472


Unpin OCCT when netgen compiles with OCCT 7.8.0

Revert commit https://github.com/fem-on-colab/fem-on-colab/commit/d77eca70487145bcb72ea9145a925440cabe81bf when https://github.com/NGSolve/netgen/issues/170 and https://forum.ngsolve.org/t/compatibility-with-occt-7-8-0/2604 are fixed

Done in 478bbc17c29000b8fddf853d41b3eaed4410fff6 and 884136973dd2d1443420198489f8dc7a56bc4bcc


Feedback for “Next.js SSR”

FYI, you don't have any more credits. :wink:

I'm contacting the support team to solve the problem 🙏

I've fixed it.


SLF4J dependency issue

When creating a new project in AndroidStudio 3.3, and checking the AndroidX box during the new project wizard, creating a default empty project then adding KotlinTest as instructed:
android {
    testOptions {
        unitTests.all {
            useJUnitPlatform()
        }
    }
}

dependencies {
    testImplementation 'io.kotlintest:kotlintest-runner-junit5:3.2.1'
}

Then add a simple test using KotlinTest, the test executes no problem but gives this warning:
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.

I'm new to Android and gradle, so don't really understand what's going on here. Is it some kind of internal dependency error, or should end-users also add a SLF4J implementation (maybe a testImplementation line??) for it to work without warnings?

@Kerooker can you help with this as you're our android man :)

I'm not sure if I can help with this.
I used to have this kind of warn even without KotlinTest, another lib would introduce it.
I usually just ignore it :P
But, the issue here is that Android uses LogCat by default, and I don't think SLF4J integrates with it. For this, you would have to add some implementation compatible with SLF4J for your unit tests.
https://stackoverflow.com/questions/7421612/slf4j-failed-to-load-class-org-slf4j-impl-staticloggerbinder
Currently, there isn't much we can do for you.  SLF4J is a logging facade employed by some dependeny that needs to bind to an actual logging implementation, otherwise it warns you.
You can solve this by attaching the sf4j-simple implementation in your tests:
testCompile group: 'org.slf4j', name: 'slf4j-simple', version: '1.7.26'

@Xerus2000 if that's the fix, then it should be added to the Readme docs.
It's quite confusing to see that warning in every test launch.


IHttpClientFactory returns the wrong type, making it impossible to implement a custom IHttpConnection

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Strawberry Shake
Describe the bug
Presumably, the goal of exposing a DI version of IHttpConnection is to allow the end-user to plug the connection with a different implementation. In my case, System.Net.HttpClient is totally incompatible with my deployment. This should be the perfect use case for IHttpConnection, so I can plug it with a compatible implementation.
Unfortunately, even if a different connection is provided, Strawberry Shake still attempts to create its own HTTP client:
AddTransient<StrawberryShake.Transport.Http.IHttpConnection, MyHttpConnection>()

Regardless of if I use Singleton, Scoped, or Transient... the MyHttpConnection is never used. Strawberry Shake appears to load up the IHttpClientFactory instance and create a new client for each request instead.
This should be easy to fix on my side, except the IHttpClientFactory is incompatible with IHttpConnection. The Factory is obligated to return a StrawberryShake.Transport.Http.HttpConnection connection, instead of a IHttpConnection:
HttpClient CreateClient(string name);

Thus, the StrawberryShake.Transport.Http library is strongly coupled to the system's implementation of System.Net.Http.HttpClient... which is in turn coupled to System.Threading, meaning that the library cannot be used at all in environments where this is unavailable (i.e., Unity WebGL).
All it would take to fix this is to change the return type to IHttpConnection. This is already what the code generators do, coercing the HttpClient into the interface type, so it appears the current implementation is unnecessary.
Steps to reproduce
Try to use a IHttpConnection that is not derived from `
Relevant log output
No response
Additional Context?
I was able to create a very hacky solution by rewriting the generated code from StrawberryShake, but it's quite annoying since I'll need to do it every time the code is generated:
global::Microsoft.Extensions.DependencyInjection.ServiceCollectionServiceExtensions.AddSingleton<global::StrawberryShake.Transport.Http.IHttpConnection>(services, sp =>
            {
                return new MyHttpConnection();
            });

Version
13.6.1

We initially designed strawberry shake as a client for blazor and nothing else :)
At some point we will do a redesign and abstract in more... at the moment our focus is on other things.

But you can go ahead and do a PR if you like.

@michaelstaib are you aware of anyone working on a Unity-compatible version, aside from https://github.com/ChilliCream/graphql-platform/issues/4066? I have successfully implemented an IHttpConnection which leverages the Unity HTTP client under the hood. However, 90% of the code is copy-pasta from the internal and sealed classes in the Transport.Http package. With a few small tweaks to Transport.Http, I could submit a PR that also allowed me to implement the Unity HTTP layer "correctly." Thus I should then be able to publish a minimal StrawberryShake.Transport.Unity package, with your approval, so that other game devs can benefit.

@zaneclaes did you happen to find a solution for it? I see that your PR is still in draft.
Also, what are the symptoms you had in WebGL? I currently get this message:
Unable to open DLL! Dynamic linking is not supported in WebAssembly builds due to limitations to performance and code size. Please statically link in the needed libraries.
But I'm not sure if what you described is the actual cause of my runtime error.

@Vingator full support for Unity is MUCH trickier than just this piece, but yes, I have it working in our project, including WebGL. Your error is a runtime problem, and unrelated to this thread. Specifically, you need to statically link all the necessary HotChocolate/StrawberryShake runtime DLLs compiled for the netstandard2.x framework (and their dependencies, of course). I would recommend making sure your solution works in the editor first, and then moving on to solving the run-time linking errors on the various build-targets.

@zaneclaes thanks for the quick answer! I have the app working on the editor and even on iOS/Android, but I get this exception on WebGL at the point I'm querying data, that's why I supposed it was the same issue you were facing


Can't target Stitches component within :has() pseudo-selector

Bug report
Describe the bug
Following this page in the docs: Target a Stitches component
I'm trying to make something like the Input Group component from Chakra UI by using the has pseudo selector, but the styles aren't being applied.
To Reproduce
const inputElement = css({
  display: 'flex',
  alignItems: 'center',
  justifyContent: 'center',
  position: 'absolute',
  top: '0',
  zIndex: 2,
})

export const leftInputElement = css(inputElement, { insetStart: 0 })
export const rightInputElement = css(inputElement, { insetEnd: 0 })

const StyledInputGroup = styled(DEFAULT_TAG, {
  width: '100%',
  display: 'flex',
  position: 'relative',
  [`&:has(.${leftInputElement})`]: {
    paddingLeft: '$4',
  },
  [`&:has(.${rightInputElement})`]: {
    paddingRight: '$4',
  },
})

Expected behaviour
If InputGroup contains an element with leftInputElement class, the correct padding should be applied
The has pseudo selector doesn't have very good support, so apologies if this is due to that.

Did you already solved this issue? How?


Android emulation support

I know this is not the correct place but I will ask anyway. Recently Mumu player pro released an android emulator for apple silicon. Currently there is only two android emulator one is android studio AVD (very poor performance) and Mumu player pro (excellent performance but paid). Is there any way to run android on apple silicon like Qemu or something? please guide.

If you know this is not the correct place, please don't ask. It has nothing to do with this project, so I do not know, and I cannot help you.


feat: compatible to vue optional api and classed component UI framework

:tada: This PR is included in version 2.12.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[$100] Migrate FormWrapper from withOnyx to useOnyx

Coming from https://expensify.slack.com/archives/C01GTK53T8Q/p1725973460005309?thread_ts=1725905735.105989&cid=C01GTK53T8Q
Migrate src/components/Form/FormWrapper.tsx to use useOnyx instead of withOnyx.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021834283108471964411
Upwork Job ID:<PHONE_NUMBER>471964411
Last Price Increase: 2024-09-12
Automatic offers:

HezekielT | Contributor | 103988904

Issue OwnerCurrent Issue Owner: @abdulrahuman5196

Proposal
Please re-state the problem that we are trying to solve in this issue.
Migration of withOnyx to useOnyx
What is the root cause of that problem?
Migration
What changes do you think we should make in order to solve the problem?
https://github.com/Expensify/App/blob/7748eff72fc43ecaf3db6159928d8d581a63ab99/src/components/Form/FormWrapper.tsx#L192-L198
We shall convert it to useOnyx hook.
const [formState] = useOnyx(formID)

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
We need to migrate the FormWrapper component from using withOnyx to useOnyx to reduce TypeScript compilation time and memory
What is the root cause of that problem?
The withOnyx Higher Order Component (HOC) is a major contributor to high memory usage during TypeScript compilation, and it has been deprecated in favor of useOnyx.
What changes do you think we should make in order to solve the problem?

Initially we need to import useOnyx from react-native-onyx

Update the component's prop types  to remove FormWrapperOnyxProps.
https://github.com/Expensify/App/blob/0c618acc8b5a0b77eb27dfa049a18e3135c51c52/src/components/Form/FormWrapper.tsx#L26-L28

Modify how the formState is accessed within the component, replacing props.formState with the state returned by useOnyx.
https://github.com/Expensify/App/blob/0c618acc8b5a0b77eb27dfa049a18e3135c51c52/src/components/Form/FormWrapper.tsx#L62-L73

const [formState] = useOnyx<Form>(`form_${formID}`);

Replace the withOnyx HOC with the useOnyx hook, to be added inside the FormWrapper function body.
https://github.com/Expensify/App/blob/0c618acc8b5a0b77eb27dfa049a18e3135c51c52/src/components/Form/FormWrapper.tsx#L192-L198

Adjust the export statement to directly export the FormWrapper component without the withOnyx wrapper.

export default FormWrapper;

What alternative solutions did you explore? (Optional)
N/A

Updated Proposal

I've been assigning out these migration issues on a first-come-first-serve basis while trying to avoid duplicates assignees. So I'm going to assign this to @HezekielT

If @HezekielT is not available, I am ready to take upon this. I will raise a PR in not more than 2 hours

PR was merged, just awaiting deploy then payment. @NikkiWines choosing you at random to be here just in case the PR is reverted an internal engineer help is needed. Hope you don't mind!

@HezekielT it looks like it is still waiting on you to accept the proposal — can you complete that ASAP within Upwork? Thanks!
@abdulrahuman5196 — any update on the checklist? Thanks!

@HezekielT — thanks, you have been paid $100 via Upwork!
@abdulrahuman5196 — any update on the checklist?

Back from OoO (thanks @anmurali for holding down the fort!)
@abdulrahuman5196 can you please complete the checklist before EOD? We cannot move forward with payment until it is completed. Thank you!

Dropped a bump in Slack here

Hi, @CortneyOfstad I don't think we need a checklist here since its a code migration. It doesn't have any PRs to point or new test cases. Kindly let me know if anything else is required.

Thank you for confirming @abdulrahuman5196!

Payment Summary
@abdulrahuman5196 — to be paid $100 within NewDot
@HezekielT — paid $100 within Upwork

$100 approved for @abdulrahuman5196


Documentation

Hi
The documentation for this bundle doesn't follow the stable version which it should.
Apparently there are new development done on master that shouldn't be released until version 6, which is quite a big leap from current stable version.
New development shouldn't be done on master branch.
Suggestion: either release current master to new version 5.* or rollback master to latest stable version and move the new development to it's own branch and when ready to release it merge it to master.
It's not user friendly to have a documentation that isn't at the moment true.

I just come back from holiday, so, sorry for the late reply... I agree that it's not user friendly and I need to talk about it with the Symfony guys as it's them which manage the doc build :)

The online doc is now in sync with the code but I let the issue open as I need to get in touch with the Symfony team in order to fix it definitively...

v6 is a bit hard to use without doc

@karael The current online doc is the 6.x one.


Weekday use different Timezone

I have created a stack in ap-southeast-2 region, default timezone is Australia/Melbourne.
I have instances launched in us-east-2 region would like to use the aws-instance-scheduler stack launched in ap-southeast-2.
I created a period
scheduler-cli update-period --stack instance-scheduler-stack --name development-running-hours --weekdays mon-fri --begintime 06:00 --endtime 22:00
and a schedule
scheduler-cli update-schedule --stack instance-scheduler-stack  --name development-schedule --periods development-running-hours —timezone US/Central
Even though the instance is running on US/Central time 6 am to 10 pm, it is running on Australia/Melbourne's Monday to Friday. Hence, the instance did not start on US/Central's Friday at 6 am, it is equivalent to Melbourne's Saturday.

@fionah821 I tried to replicate the issue.

@fionah821 I tried to replicate the issue, used the following steps,

Created a stack in ap-southeast-2 with default timezone as Australia/Melbourne
Create a period with begin time 18:00 to end time 22:00 and weekday set to sun.
Create a schedule with name development-running-hours  and timezone as US/Central,
Created an instance in us-east-2, tagged it with the schedule name development-running-hours,  the instance started as expected. (local time in Melbourne was Monday 11 AM),

Can you update the environment variable Trace to True and provide us the logs for the lambda function, at the time it is expected to start the instance?


Implement custom rate limiter

Supports daily, weekly and monthly quotas. Quotas are measured in API usage time.
Provide stubs to add token-based quotas.
Related to https://github.com/ooni/backend/issues/371

I see this branch is still in draft state. I also don't see any new commits since my comments, @FedericoCeratto did you maybe forget to push?


Tool tips do not match display text for tests

Description
When working with a [Theory] test in xUnit, the tool tip shown for the test(s) in Test Explorer do not match the text printed for the name. Here is an example from Microsoft/perfview#222.

Steps to reproduce

Add a [Theory] test with [InlineData]
namespace Microsoft.Diagnostics.Tracing.Analysis.Example
{
  public class MyTestClass
  {
    [Theory]
    [InlineData("Name")]
    public void MyTheory(string value)
    {
    }
  }
}

Build the project and wait for tests to show in Test Explorer

Locate the new test

Make the Test Explorer window wider in hopes of seeing the full name of the test

Since the Test Explorer window responded in the previous step by placing the output to the right of the test names instead of below, again make the test window wider so you can actually see the full name of the test

Expected behavior

Test explorer shows the test as the following:

Microsoft.Diagnostics.Tracing.Analysis.Example.MyTestClass.MyTheory(value: "Name")

The tool tip for the test is the same as the display name

Actual behavior

The display name for the test shows correctly.
The tool tip shows a meaningless number instead of the test arguments.

Environment
Visual Studio 2017 (15.2), Windows 10, XUnit 2.2.0

This is being tracked external to this repo, here:
https://developercommunity.visualstudio.com/content/problem/145928/tool-tips-do-not-match-display-text-for-tests.html

@snkota There's also the linked xunit/xunit#1454 and https://developercommunity.visualstudio.com/content/problem/113694/the-tooltip-of-a-test-in-the-test-explorer-should.html


Improve sorting further

Allow sorting by name, perhaps with directories too. Also maybe auto-sort on page load.
Make sorting spans appear clickable through CSS

Resolved in 8f732db65971383223ba3073c22cef1b7a323605
Name sorting includes directories, except "Parent Folder" div.
Auto-sorting was decided against


4masterts Feedback

[x] Use CacliteMaps or nothing. Calcite-web is only for web pages
[x] Would prefer not to use any calcite since it should be framework independent
[x] Separate boilerplate code into its own repo. Have other repos use TypeScript, vanilla JS, frameworks, grunt, etc. Don't force users to use TS.

@kellyhutchins @alaframboise @nicksenger
I think for 4x, this repo will only be the Boilerplate Class, which will be written in TS.
We will create other repos that import this class and use it with and without Typescript and we can create even more repos that use it with things like calcite maps or other frameworks.
This way, I can write and maintain the boilerplate class separately in TypeScript but it can be consumed using TS or whatever.
I've created the branch for it here:
https://github.com/Esri/application-boilerplate-js/tree/4masterts-class-only
Thoughts?
I think we can have at least one other repo which will have all the other common app stuff like displaying errors, loading spinners, dojo config setup, etc. This could be written using TS or without. I'm up for either. Or we have two repos, one for each.
We could have more repos with examples only, that users can use. This way, we keep the examples and boilerplate separated so that people dont use the examples expecting that they won't change.
-Matt

@driskull I think this is a great idea. I think it's the only way to truly separate the base from the apps. I think we should start from ground zero for the 4.x stuff. It would be great if we had something like this:
app-template-base
app-template-map
app-template-calcite-maps
app-template-group

I was thinking it would be more like this:

configurable-app-js This is the base application class. (owned by @driskull )
configurable-app-boilerplate-js This is the simple application with basic file structure. (owned by @kellyhutchins )
configurable-app-example-group ... (owned by ???)
configurable-app-example-calcite-maps ... (owned by @alaframboise )
... any other examples?

I think it all sounds good but I think we should also have a Calcite Web example too. Its useful and people are already working with it.

@driskull  I hate naming things its so hard. :)
I think ApplicationHelper works.

@kellyhutchins it is, soo hard.

Isn't it all portal stuff? What about ApplicationBase or ApplicationConfig or ArcGISConfig?

ApplicationBase sounds ok to me. I think ApplicationConfig would be confusing because of the other config files we have.

ConfigurableAppBase or AppPortalBase

ApplicationBase works. I'll go with that.
Ill create the following repo on ArcGIS Github which we can transfer to Esri once ready.

application-base-js

I think we should keep this one for the boilerplate since people already know about this and we have 4x on here already. The name of the repo still works.

application-boilerplate-js

Once I get the repo set up here on the 4masterts branch, you guys can create repos on ArcGIS for

configurable-app-example-webscene
configurable-app-example-group
configurable-app-example-calcite-maps

@driskull, perhaps combine the examples as folders in a single repo? One-stop shopping for examples and uses less of the ArcGIS account quota.

sure. I'll create and manage the base repo. I'll setup the boilerplate repo and @kellyhutchins will manage it.
The other example repos I'll leave up to @alaframboise and whoever else to manage/maintain.

@kellyhutchins I'm all for a single configurable-app-examples repo as @MikeTschudi suggests. We just need to figure out the structure so each can have it's own configs and if there will be shared code. e.g.
\applicationbase\
\map\
\scene\
\group\
\shared\

Its probably best not to share the code so people can just copy the folder of the example they like and tweak it.

ya, i was just thinking that some helper modules would be totally shareable e.g. viewManager

I like @MikeTschudi  idea of combining it all in one repo. That way users can quickly find the examples and pick the one they want. I also think we should keep the code separate in the folders so its easy to copy only the code you want without having to also pull shared code from various places.

We'll already have code sharing--the boilerplate--so a shared folder wouldn't be an outlier.
Perhaps have subfolder placeholders in each example:  Each subfolder would have a text file with a name like insert_boilerplate_here.md (or another README.md?); inside the file would be the link to the repo or folder containing what should be inserted, e.g., the boilerplate's dist folder for one, the share folder for another.
Updating one's app to use an updated boilerplate would be as simple as replacing everything in the boilerplate subfolder. Boilerplate and shared code are maintained in one location and used in multiple.
Should the shared folder be part of the boilerplate? From this viewpoint, the boilerplate provides common elements useful for creating apps, and one picks what one needs from it.  E.g., it'll supply routines for displaying webmaps, webscenes, and groups, but a typical app won't use all three, so the boilerplate is already acting like a common library.

@kellyhutchins, are you suggesting keeping the boilerplate with the examples and the shared code in one repo? If the individual parts can be updated separately--especially the boilerplate and/or shared--this would simplify the documentation and use.
But keeping the boilerplate in a separate repo seemed valuable because it emphasized that it can evolve separately from one's app, like the JSAPI evolves separately. The crisper the separation between boilerplate and app, the easier it is to maintain.  It also might make it easier to source the boilerplate in TypeScript while distributing it in JavaScript for use in TypeScript or JavaScript apps.

No I was suggesting all the examples in one repo and the boilerplate in another.

I have the two repos setup here:
https://github.com/ArcGIS/application-base-js
https://github.com/ArcGIS/application-boilerplate-js
Once we get these approved and the Readme's updated, we can move them to the Esri account on github.
This repo we can rename to "application-boilerplate-3x-js" and mark it as deprecated.
I will work on the application-base-js readme. Can one of you tackle the application-boilerplate-js readme and repo? It should have everything it needs including NPM, Grunt.
Also, we need the examples repo created.

@driskull why not just app-template-base and app-template-examples? The '.js' suffix is only handy when there are examples in many different languages.

Incase there are ever different languages. We've always done that for templates and other things.

@driskull your call, but IMHO application just seems too generic... ;)


ci: add dependabot package updating

partly so that we have PRs to merge to resolve the stuck issues faster
this config set up is pretty good for application projects in my experience

I think it's a good idea in general to automate dependency updates. Considering that dependabot ignores the interval setting when it comes to security updates ^1, I wouldn't run this daily.

partly so that we have PRs to merge to resolve the stuck issues faster

Actually we haven't automated deployments to the server, so merging a PR doesn't fix the stuck issues :sweat_smile:


Add commit-confirm support [on-hold]

I need to think about/test these workflows a bit for various use cases.

On hold pending merge of PR #46 and then I need to rebase and fix the tests (as tests depending on #46)

Feel free to ping me when this is ready for review.

@dbarrosop Okay, this should be good to go now.

Nice! +1


license notice

For topical reasons. I just saw Intel using the Apache license. It would be nice to rename the file COPYING to NOTICE as it is provided in the Apache license. Also to do justice to all licenses in the COPYING file.

Hi @Gallimathias
Let me look into it. Do you have any link on the subject? In different open-source projects I saw different practices, be it COPYING, Attributions file, or something else.
Of course we will to do our absolute best to ensure proper attributions and compliance.

You can see that NOTICE has the advantage that you have to copy it and pass it on. However, the COPYING file is absolutely voluntary and will not be considered.

Thank you, we are looking into the issue

Approved, thank you for raising it

No Problem ;)


Building for iOS: "Shader Is Not Supported On This GPU (none of subshaders/fallbacks are suitable)"

Hello,
First, very cool project! I wanted to try building this for iOS, but I'm running into an issue. What I've done:

Removed NetworkCommunication Code (as referenced in another issue if we are not building for Windows Store)
Stripped Hololens Gesture Control code
Hardcoded IP - manually called connect function.

The project compiles fine, but I see a warning under the GSG Billboard shader object, saying: "Shader Is Not Supported On This GPU (none of subshaders/fallbacks are suitable)". Does anyone know the reason for this? I'm not quite sure why the iOS GPU wouldn't be able to properly render this shader.
Thanks for any information you can provide, and thanks again for the cool project!

Thank you @MarekKowalski! I will try this and report back. I think I still have an error to fix, I was getting a black screen on iOS after the Unity logo appears and the logs say "SocketException:Connection Refused". This is strange to me because on the PC where I am running the server I see that the port connection is established and looking at Wireshark logs I can see the point cloud stream is being sent to my phone's IP over port 48002.
I thought even with the bad shader it should still render, albeit just a solid pink color? Maybe I'm wrong, will let you know as soon as I try with the shader you provided. Thanks again!

Okay @MarekKowalski, I got it rendering on the iPhone! Since I removed the gesture control code, I had to rotate the camera Y axis 180 degrees to see it. That shader is still an issue though, it just renders depth, it doesn't render color at all.
Also one other thing, the background is black. Would you know would I go about making the background transparent and having the background be my phone's camera feed for some sort of AR experience? Thanks again for your help and project!

How did You manage to autoconnect at start, it always freezes my editor when I try to get it like this

@nickcastel50, congratulations on your progress and sorry I did not answer, for some reason I missed your post completely :( Unfortunately I know little a bout shaders as well, so I won't be able to help you there.
@TheBricktop , have you made sure the live view window is open and showing the point cloud (on the same device as Unity)? If you did and it does not work, please check your firewall settings.
Marek

How did You manage to autoconnect at start, it always freezes my editor when I try to get it like this

@TheBricktop: You can just just modify the KeyboardInput.cs file in @MarekKowalski's project. Just call keyboardDone.invoke(YOUR_IP_ADDRESS); in the start() function. It works fine for testing.
Something like this:
void Start ()
    {
        if (keyboardDone != null)
            keyboardDone.Invoke(YOUR_IP_ADDRESS);
    }

void Update ()
    {

    }


Add freethreading_compatible directive to set Py_mod_gil slot

Closes #6239.
The reason this is still a draft is that the test should only run under the free-threaded build. Is that something we can do easily? I saw the mechanism that includes limited api tests, but that seems more complex.

Thanks for the quick response on this @da-woods! I've addressed the feedback.

I'll leave this open for a  little bit so people can bikeshed names. But I think it's basically good

It'd be great to see this get merged soon as it's a prerequisite for building free-threading compatible wheels for many of the projects that depend on Cython.

@lysnikolaou and I had a brief chat about this. I think we'd both like to get this merged with the current behaviour in the fairly near future. (From my point of view that means "probably sometime after EuroPython")
My one suggestion was to explicitly document the option as "experimental" - I think we do want to give ourselves freedom to evolve his we treat free-threading for at least as long as it's experimental in CPython.

I'm fine with merging this, but yes, please document it as experimental. It's not obvious that there will be changes later, but it's going to take a while until we understand the user experience well enough to give good answers to it.
One comment, there is no need to declare the directive type since that's handled automatically for a bool default value.

I tried to compile _pcg64.pyx in numpy with this directive inserted at the top of the file and hit this compilation error when building numpy:
numpy/random/_pcg64.cpython-313t-darwin.so.p/numpy/random/_pcg64.pyx.c:9310:3: error: expected '}'
  {0, NULL}
  ^
numpy/random/_pcg64.cpython-313t-darwin.so.p/numpy/random/_pcg64.pyx.c:9304:51: note: to match this '{'
static PyModuleDef_Slot __pyx_moduledef_slots[] = {
                                                  ^
1 error generated.

Haven't looked further at why this is happening yet.

Hmm, that ended up being because I had a typo in the directive name, frethreading_compatible=True. Not sure why the miscompilation happened with the typo given the code change in this PR.

Thanks a lot everyone for the feedback. Pushed a new commit that:

Fixes the bug that @ngoldbaum discovered (syntax error in single-phase init)
Adds tests for this under single-phase init
Mentions in the documentation that this is still experimental
Removes the directive type declaration like @scoder suggested

Thanks

The new test is failing in the freethreading CPython build:
Errors from shard 6:
======================================================================
FAIL: runTest (__main__.EndToEndTest.runTest)
[6] End-to-end freethreading_compatible
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/cython/cython/runtests.py", line 2040, in runTest
    self.assertEqual(0, res, "non-zero exit status, last output was:\n%r\n-- stdout:%s\n-- stderr:%s\n" % (
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        ' '.join(command), out[-1], err[-1]))
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: 0 != 1 : non-zero exit status, last output was:
'/home/runner/venv-3.13/bin/python -c import default; default.test()'
-- stdout:
-- stderr:Traceback (most recent call last):
  File "<string>", line 1, in <module>
    import default; default.test()
                    ~~~~~~~~~~~~^^
  File "default.py", line 7, in default.test
    assert sys._is_gil_enabled()
AssertionError

https://github.com/cython/cython/actions/runs/9903272103/job/27358534710

default.py should be assert not sys._is_gil_enabled() instead I think. So it's the test that's wrong rather than the behaviour.

Hmm, that's weird. The test succeeds locally for me.

default.py should be assert not sys._is_gil_enabled() instead I think. So it's the test that's wrong rather than the behaviour.

I don't think that's correct. By default, the directive is off, which means that the GIL should be enabled after importing the module.

Oh, it's because the CI job sets PYTHON_GIL=0 so that GIL is not enabled, even if a non-compatible modules is imported.


Bug in encodeData() with tuples

v0.23.1
ERC725.encodeData([
    {
        keyName: 'LSP4CreatorsMap:<address>',
        dynamicKeyParts: address,
        value: ['0xb3c4928f', 0]
    }
], LSP4DigitalAssetSchema)

encodes value to 0xb3c4928f00000000000000000000000000000000000000000000000000000000
but should encode to 0xb3c4928f00000000000000000000000000000000
because LSP4CreatorsMap's valueType is (bytes4,uint128).
the bug is here:
https://github.com/ERC725Alliance/erc725.js/blob/8a783368f4f3deb2f9810a86cf5424d8d95d2627/src/lib/utils.ts#L211
it treats everything as bytes* and hence substrings "uint128" to "28"...

Thanks for reporting @m4r-v1n
This seems to have been fixed with #427 . I am double checking the behaviour but that looks good to me.
Also would be good to improve the Typescript typing here.
With this ✅

Over this ❌


Add link to section in section title

With a hashtag or something that shows on hover

Can't figure out how to do it currently


Web - Chat - When hit (CTRL+K) after opening global action button,  Global action button and search modal opens

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
Click on the "Global action" button
Press on CTRL + K

Expected Result:
The first modal  should be closed when the second is opened
Actual Result:
Both Global action button and search modal opens
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web

Version Number: v1.1.85-4
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

https://user-images.githubusercontent.com/43996225/179831074-abed4adb-d791-4e0e-9237-d6b7ef9288ae.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
View all open jobs on GitHub

Not a huge deal. Setting as monthly!

@Gonals One of the tester experienced a similar issue. When open the New room through the FAB menu and press the spacebar on the keyboard, then the FAB menu will additionally open again.
https://user-images.githubusercontent.com/93399543/183112315-1524c471-8f5f-4786-be08-de92fe5b39cc.mp4


CodeLens Stopped Working for Cells in Python Files

Version: 1.47.2
Commit: 17299e413d5590b14ab0340ea477cdd86ff13daf
Date: 2020-07-17T10:49:51.211Z
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.7.8-200.fc32.x86_64
Steps to Reproduce:

Install Codium via Fedora's Software or gitlab.com_paulcarroty_vscodium_repo repository.
Ensure that you have Python plugins installed and CodeLens enabled using the following settings:

    "editor.codeLens": true,
    "python.dataScience.codeLenses": "python.datascience.runcell,  python.datascience.runallcellsabove, python.datascience.debugcell",

Create a simple Python test file with a couple of cells starting with '# %%'

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

We don't support codium, sorry.

But my understanding is that VSCodium is the same exact code as VSCode, excluding telemetry, no? Is this problem not showing on VSCode on Fedora as well?


NullReferenceException in DeferredRenderer

This issue was originally reported in Wasabi wallet repository https://github.com/zkSNACKs/WalletWasabi/issues/1147
Unhandled Exception: System.NullReferenceException: Object reference not set to an instance of an object.
   at Avalonia.Rendering.DeferredRenderer.Render(Boolean forceComposite) in D:\a\1\s\src\Avalonia.Visuals\Rendering\DeferredRenderer.cs:line 266
   at Avalonia.Controls.TopLevel.HandlePaint(Rect rect) in D:\a\1\s\src\Avalonia.Controls\TopLevel.cs:line 269
   at Avalonia.X11.X11Window.DoPaint() in D:\a\1\s\src\Avalonia.X11\X11Window.cs:line 611
   at Avalonia.Threading.JobRunner.RunJobs(Nullable`1 priority) in D:\a\1\s\src\Avalonia.Base\Threading\JobRunner.cs:line 40
   at Avalonia.X11.X11PlatformThreading.HandleX11(CancellationToken cancellationToken) in D:\a\1\s\src\Avalonia.X11\X11PlatformThreading.cs:line 167
   at Avalonia.X11.X11PlatformThreading.RunLoop(CancellationToken cancellationToken) in D:\a\1\s\src\Avalonia.X11\X11PlatformThreading.cs:line 240
   at Avalonia.Threading.Dispatcher.MainLoop(CancellationToken cancellationToken) in D:\a\1\s\src\Avalonia.Base\Threading\Dispatcher.cs:line 64
   at Avalonia.Application.Run(Window mainWindow) in D:\a\1\s\src\Avalonia.Controls\Application.cs:line 237

https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Visuals/Rendering/DeferredRenderer.cs#L266
It seems for some reason the scene.Item is null.

While it can be inferred from the Exception, I think it's worth explicitly noting that, this is Linux related.


OPSEXP-2066 Get rid of outdated AGS amps in tests

Ref: OPSEXP-2066
After https://github.com/Alfresco/alfresco-ansible-deployment/pull/571 we realized that we were using very old AMPs since ages and in the meantime AGS packaging has changed and we need proper support for that.

Can't we remove MOLECULE_IT_EXTRA_VARS: tests/test-extra-vars.yml from other vars files, as this file is any how empty.

I was thinking that we could make the variable optional (there are some tests that are using some other custom extra vars) but I kept it in case we need to toggle something on all the tests - but I did that just because it was easier


Unable to search: 'Notch Delta'

Breaking case:  Able to search for "Notch" and "Delta" but app breaks with "Notch Delta"

This is expected behaviour. The search looks for the intersection of all the words you type in, in this case in the name. If the dataset of one word does not intersect with the dataset of the other word, you get no results.

The UI indicates nothing is happening so I had to open the console.
Not getting a result for 'Notch Delta' sounds very scary to me.

Yes, the UI not responding is a known bug documented here: https://github.com/PathwayCommons/pathways-search/issues/55.
As for "notch delta" not returning, unfortunately that seems to be a consequence of looking only in name, which is unlikely to contain the participant names unless the pathway is named for it.
Contrast the following...
Enhance disabled
http://localhost:8080/#/search?enhance=false&gt=3&lt=200&q=notch delta&type=Pathway
Enhance enabled
http://localhost:8080/#/search?enhance=true&gt=3&lt=200&q=notch delta&type=Pathway
You will notice that if you search through the enhance disabled search results, delta is not in the display name anywhere. Not certain what the right solution is here since the enhanced search works really well for looking up pathway names in general.


PtP review on

comments

Fix the auto margin on the images
Add classes to image tags
remove scroll-bar from background

Thanks for the feedback @Aidan-bn changes have been incorporated


Incredibly slow

Sadly, at least on my system this is way to slow. If I edit the last lines of a file containing ~1000 files (and 130 begins), I will get a huge lag.

Although I don't have this problem even with similar sized files, I can imagine this being a problem on slower systems. However, there is not much I could do about it, because we have to do some regular expression matching in order to determine what kind of fold level we have. Your only other option is to disable the plugin and use manual folds (i..e. %{{{ / %}}} pairs).


Ajax Load in Select List

This is a feature request.
In Select List, allow Ajax data to be pulled up  with a very long list. Initially only n items are pulled up but later as we scroll more, more data should be pulled up.
Also, during search, the data should be refreshed.

Personally I think this should be a custom built component to prevent feature bloat. Here's a quick example I've spun up using construct-ui and mithril-infinite example.

Yes. That works great... Thanks. Though I still think it could be a feature of the  component. This is a regular requirement and not that easy to implement. Construct-UI is ultimately a collection of various components to make development faster and consistent. Just an opinion.

Personally, I'd vote against this. Construct is lean and does not get in the way. It ports over a lot of the same logic mithril developers embrace (ie: less is more) but more importantly the use cases where you need infinite scrolling features are minimal at best. When you're using a SPA framework like Mithril this type of feature is easy to apply and the whenever you would find yourself needing more complex functionality then mithril-infinite suffices.

Closing as this is functionality beyond the scope of the component.


Update dependencies to support Antlr and add Kibana Query Language grammar

Description
Update dependencies in all environments to allow for the introduction of Antlr to the project, and add a grammar for Kibana Query Language.
This involves

pulling the antlr4 jar
using java 11 (earliest version of java which the antlr4 jar will run on)
using a version of gcc/g++ which supports c++17 (antlr generates c++ code which uses c++17 constructs)

The bulk of this diff is made up of ExternalAntlr4Cpp.cmake and FindANTLR.cmake which come from the Antlr project. These pull in relevant Antlr c++ code and provide a target which generates c++ code from an antlr grammar (.g4) by invoking the antlr4 jar.
Validation performed

Manually built clp in all linux docker containers

Actions for building clp on linux use the prebuilt hosted docker images -- not the ones being modified by this PR

Verified that MacOS action to build clp runs successfully

Build failures are due to limitations in our current GitHub workflows.


Adress bar to small

So compared to default bar on Windows 10 this looks too small.
Default one

And Yours

Even when uninstalling your theme it kept the small one for some reason.

Yeah, you're right! LOL!


Expiration: Add Requester/Requested_by fields to Proposal

For proposal expiration: Include information about
requester:
- job title
- name
- email

I think you are proposing adding this information so that an approver will be able to see information on someone who is asking for re-auth (expiring permissions)...
Don't we already have this information available for new requests?  I don't see why we would need this only for Expiration?  As I understand the current workflow, when my permissions are about to expire I get an alert (email and an alert icon in the UI) so I go to the proper role page and press the "request Extension" button (UI is TBD)
Then the role owner and my manager each get a request to approve.... they can see my details by going to the People page (or on the email that gets generated, right?)  Let's chat...

This seems like a task in support of functionality we already have.  Closing.


Helping the user bring the right things based on the weather

If it's raining, bring an umbrella
If it's sunny, bring suncream
If it's cold, bring a coat

Hey @wrussell1999, can I work on this issue if it's open?