Crash when starting app

When I load the app the gem crashes the app. Any ideas?
/Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/activesupport-3.2.13/lib/active_support/dependencies.rb:251:in `require': /Users/elton/.rvm/gems/ruby-1.9.3-p392/bundler/gems/color_sort-440ea407f190/lib/color_sort/ordering.rb:7: syntax error, unexpected tLABEL, expecting ')' (SyntaxError)
    def initialize(start: nil, finish: nil)
                         ^
/Users/elton/.rvm/gems/ruby-1.9.3-p392/bundler/gems/color_sort-440ea407f190/lib/color_sort/ordering.rb:7: Can't assign to nil
    def initialize(start: nil, finish: nil)
                              ^
/Users/elton/.rvm/gems/ruby-1.9.3-p392/bundler/gems/color_sort-440ea407f190/lib/color_sort/ordering.rb:54: syntax error, unexpected keyword_end, expecting $end
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/activesupport-3.2.13/lib/active_support/dependencies.rb:251:in `block in require'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/activesupport-3.2.13/lib/active_support/dependencies.rb:236:in `load_dependency'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/activesupport-3.2.13/lib/active_support/dependencies.rb:251:in `require'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/bundler/gems/color_sort-440ea407f190/lib/color_sort.rb:2:in `<top (required)>'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/bundler-1.3.5/lib/bundler/runtime.rb:72:in `require'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/bundler-1.3.5/lib/bundler/runtime.rb:72:in `block (2 levels) in require'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/bundler-1.3.5/lib/bundler/runtime.rb:70:in `each'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/bundler-1.3.5/lib/bundler/runtime.rb:70:in `block in require'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/bundler-1.3.5/lib/bundler/runtime.rb:59:in `each'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/bundler-1.3.5/lib/bundler/runtime.rb:59:in `require'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/bundler-1.3.5/lib/bundler.rb:132:in `require'
	from /Users/elton/WorkSpace/fameandpartners/website/config/application.rb:7:in `<top (required)>'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/railties-3.2.13/lib/rails/commands.rb:53:in `require'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/railties-3.2.13/lib/rails/commands.rb:53:in `block in <top (required)>'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/railties-3.2.13/lib/rails/commands.rb:50:in `tap'
	from /Users/elton/.rvm/gems/ruby-1.9.3-p392/gems/railties-3.2.13/lib/rails/commands.rb:50:in `<top (required)>'
	from script/rails:6:in `require'
	from script/rails:6:in `<main>'

It's a ruby 2.0 feature.


Revert "Continuously publish Cargo builds"

This reverts commit 7799014dc7330445f62e2f925573ca15dd8e8e03.
Fixes https://github.com/rust-lang/cargo/issues/3269.
The reason doc.crates.io doesn't display correctly is that travis-cargo writes its own index.html that is just a meta-redirect to the api docs, overwriting cargo's own docs.
r? @alexcrichton

I've fixed the symptom temporarily by pushing index.html to the live site.

@bors: r+
ugh

:pushpin: Commit b008422 has been approved by alexcrichton

:hourglass: Testing commit b008422 with merge a8510bb...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing a8510bb2ef4c5623972b14700bf0b64d628ba42b to master...


[11.x] Adds Illuminate\Database\Eloquent\Builder::joinModel() method

Join a model's table to the query while respecting its global scopes and constraints.

This method enhances the standard join functionality by allowing you to join

Eloquent model tables dynamically, automatically applying the model's global scopes

and other query constraints. It's particularly useful for complex queries that need

to maintain model-specific conditions, such as soft deletes.

@example

// Join the 'users' table, respecting the User model's global scopes like deleted_at

User::joinModel(Post::class, 'posts.user_id', '=', 'users.id');

joinModel Method Allow you to join models and allow global scopes like soft delete (where deleted at is null) for each models dynamically

I don't really see the usage case for this feature, you can also just use join()
https://laravel.com/docs/11.x/queries#joins

User::join('posts','users.id','=','posts.user_id')
->whereNull('posts.deleted_at')// you may forget that condition
->whereOtherGlobalScopeInPosts() //
->paginate();
User::joinModel(Posts::class, 'users.id', '=', 'posts.user_id')->paginate(); // just it
@flavio-schoute

@kerolesmonsef I believe you should be able to just add joinModel as a macro to the Eloquent Builder class if you need this.

@kerolesmonsef I believe you should be able to just add joinModel as a macro to the Eloquent Builder class if you need this.

I think that is a better solution

Why not adding this method to the Illuminate\Database\Eloquent\Builder instead of the Illuminate\Database\Query\Builder?
As it is in the title?

there is a simpler way
ModelA::query()->joinSub(ModelB::query(), ...)
I used it and it was ok

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!


Prop type for style does not allow empty/missing style prop

Hey there,
The current release of lottie-react-native (2.5.8) has an error in the prop types declaration.
Right now, style is declared as a ViewStyleExceptBorderPropType, which looks like:

const ViewStyleExceptBorderPropType = (props, propName, componentName, ...rest) => {
  const flattened = StyleSheet.flatten(props[propName]);
  const usesBorder = Object.keys(flattened).some(key => key.startsWith('border'));
  if (usesBorder) {
    return Error(
      `${componentName} does not allow any border related style properties to be specified. ` +
        "Border styles for this component will behave differently across platforms. If you'd " +
        'like to render a border around this component, wrap it with a View.',
    );
  }
  return ViewPropTypes.style(props, propName, componentName, ...rest);
};

This does not allow you to omit the style prop.    <LottieView source={foo} /> shouldn't throw a prop types warning but it does.   <LottieView source={foo} source={{}} /> fixes the warning.
I believe ViewStyleExceptBorderPropType should look something like this:
const ViewStyleExceptBorderPropType = (props, propName, componentName, ...rest) => {
  if(props[propName]) {
      const flattened = StyleSheet.flatten(props[propName]);
      const usesBorder = Object.keys(flattened).some(key => key.startsWith('border'));
      if (usesBorder) {
          return Error(
              `${componentName} does not allow any border related style properties to be specified. ` +
              "Border styles for this component will behave differently across platforms. If you'd " +
              'like to render a border around this component, wrap it with a View.',
          );
      }
  }
  return ViewPropTypes.style(props, propName, componentName, ...rest);
};

Thanks!

I just submitted a fix for this: https://github.com/react-community/lottie-react-native/commit/ec60a087ccd819fc7901d8d8355407f5b1f6c543
It should be available on master and in 2.5.10 (to be released)


Add PEP420 namespace support.

Should address issue #2918.

The test can be disabled for Py2 by adding it to the VER_DEP_MODULES exclusions in runtests.py, section (3,3).

Thanks


no-input-rename rule should allow aliasing dashed properties

Specifically, something like this should be allowed:
@Input('aria-label') ariaLabel: string;
In above, we can't have a variable called aria-label. TS code wouldn't compile, and JS engine would complain invalid left hand side assignment.

Regex to whitelist names makes sense.

Whitelist only for aria-* family then? Btw, I found this reference.


RDKit clogP

May I ask you all:
How to use rdkit to test clogP of molecules ?
Code implementation

Please at least look at the documentation or do a google search before posting questions.


No feedback found.

Fit click-through rate prediction model | Failed | 0001/01/01 03:41 |               | No feedback found.
Search click-through rate prediction model | Failed | 0001/01/01 03:41 |       | No feedback found.
Search collaborative filtering model | Failed | 0001/01/01 03:41 |                   | No feedback found.
docker-compose.yml
rec_db:
image: postgres:14.5
environment:
- POSTGRES_USER=user
- POSTGRES_DB=recm
- POSTGRES_PASSWORD=password
- POSTGRES_PORT=5432
volumes:
- ./data/rec_db:/var/lib/recm/data/pgdata
ports:
- "5479:5432"
redis:
image: redis
expose:
- "6379"
ports:
- "6379:6379"
worker:
image: zhenghaoz/gorse-worker
restart: unless-stopped
ports:

8089:8089
command: >
--master-host master --master-port 8086
--http-host <IP_ADDRESS> --http-port 8089
--log-path /var/log/gorse/worker.log
--cache-path /var/lib/gorse/worker_cache.data
volumes:
gorse_log:/var/log/gorse
worker_data:/var/lib/gorse
depends_on:
master
links:
master

server:
image: zhenghaoz/gorse-server
restart: unless-stopped
ports:

8087:8087
command: >
--master-host master --master-port 8086
--http-host <IP_ADDRESS> --http-port 8087
--log-path /var/log/gorse/server.log
--cache-path /var/lib/gorse/server_cache.data
volumes:
gorse_log:/var/log/gorse
server_data:/var/lib/gorse
depends_on:
master
links:
master

master:
image: zhenghaoz/gorse-master
restart: unless-stopped
ports:

8086:8086
8088:8088
environment:
GORSE_CACHE_STORE: redis://redis:6379
GORSE_DATA_STORE: postgres://user:password@rec_db:5432/recm?sslmode=disable
command: >
-c /etc/gorse/config.toml
--log-path /var/log/gorse/master.log
--cache-path /var/lib/gorse/master_cache.data
volumes:
master_data:/var/lib/gorse
./config.toml.template:/etc/gorse/config.toml
gorse_log:/var/log/gorse
depends_on:
redis
rec_db
links:
redis
rec_db

volumes:
worker_data:
server_data:
master_data:
gorse_log:

Same issue here, thanks for your feedback

I manage to solve my issue by adding positive and negative (read feedback without corresponding positive feedback) feedback.
The problem and the solution is better described here
Hope this helps.

I manage to solve my issue by adding positive and negative (read feedback without corresponding positive feedback) feedback.
The problem and the solution is better described here
Hope this helps.

I also add 'like' positive and 'read' negative 2 rows into table, but the problem is still there. can you give me a help?


Pressing "Connect bank account" navigates to personal information step - reported by @Puneet-here

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 bank account > add info > company step > personal information > save
Dismiss the modal by pressing cross
Now go to workspace and visit different pages where "connect bank account" button is present
Click on "connect bank account" button
Try this few times

Expected Result:
It should take you to continue with the setup page
Actual Result:
Sometimes the pressing the connect bank account takes you to the step where you left and sometimes it takes you to continue with the setup page
It's random, sometimes it navigates you to the start over page and sometimes in the middle of some step, It doesn't matter if you are closing the bank page by going back or by pressing the dismiss.
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.2.3-0
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: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/192429603-aaaf6559-e063-442c-a1c1-24457b6de871.mov
https://user-images.githubusercontent.com/43995119/192429617-381e32b4-ea21-4f54-b2b0-6c42d0bd0195.mov
Expensify/Expensify Issue URL:
Issue reported by: @Puneet-here
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1662749815463499
View all open jobs on GitHub

Not able to reproduce in Web. Discussing further here.

Discussed in the above thread and I'm still not able to reproduce. However, the contributor who reported this is still able to reproduce, and we're both using production web on Mac.
Re-adding the triage label to see if a secondary reviewer is able to reproduce!

Let me jump in and try to reproduce.

Thanks!

Pending reproduction.

Same.

Yes, I was able to reproduce this issue. These are my reproduction steps, which I'm going to add to the issue description above for clarity.

Create a workspace (or use an existing one without a bank account setup)
Click either Issue Cards or Reimburse expenses or Connect bank account
Enter the information up until personal information and save
Exit the workspace modal (X in the top right)
Go back to the same workspace
Tap Reimburse expenses and you'll see the option to continue your setup (or start over)
Tap Connect bank account and you'll see the option to continue your setup (or start over)
Tap Issue Cards and you see Connect bank account instead of the option to continue setup (or start over)

@joekaufmanexpensify are you sure you're unable to reproduce using these steps?

Just went through these steps again and I am able to follow what you listed above through step 8. However, when you click Connect bank account in step 8, does it bring you directly to the Personal information step?
For me, I go Issue cards > Connect bank account > and then I have both options of Continue with setup and Start over, which seems correct to me. Clicking Continue with setup then brings me to the Personal information, which also seems like expected behavior.
So I'm specifically unable to reproduce what's shown in the video above where clicking Issue cards > Connect bank account goes directly to the Personal information step.

Ah gotcha! Thanks @joekaufmanexpensify we are making progress in trying to figure this one out. So yes, I agree with this part specifically.

For me, I go Issue cards > Connect bank account > and then I have both options of Continue with setup and Start over, which seems correct to me. Clicking Continue with setup then brings me to the Personal information, which also seems like expected behavior

That said, I think we still have a bug on our hands, as steps 6-8 should all take you directly to Continue with setup. Right now, only steps 6-7 do. Do you agree with that? If so, we can update the OP.

Hm, should Reimburse expenses and Issue cards go directly to Continue with setup?
 For Reimburse expenses, it's accessing the mileage rate, etc. And for Issue cards, it's viewing benefits about the Expensify Card.
I agree that step 7 in your list is expected though, as that's a more direct way to access the Connect bank account button tucked within Reimburse expenses and Issue cards.

And for Issue cards, it's viewing benefits about the Expensify Card.

Oh interesting, I hadn't considered this. I was thinking that if you had started your bank account setup and didn't finish it, that you didn't need to be sold on the card. That said, I can see your point that it's kind of debatable which is right, and that means it's not truly a bug either.
Ok so I think I'm with you then. There isn't really a bug for this flow. I'm good with closing.

Whoops, mean to leave the closing to you, though also curious for Applause and @Puneet-here to chime in and see if they are still able to reproduce on 1.2.10-0.

Sounds good, thanks!

Hi, I think this is a bug because the behaviour is not consistent even when using the same button to navigate. Example: Sometimes "connect bank account" button at issue cards page navigates to the step where you left and sometimes it takes you to the page with option to start over or continue with the setup.
https://user-images.githubusercontent.com/101883770/194142134-8a9a9eb6-4926-43ec-8bbd-6a0ae0f1356c.mov
cc: @JmillsExpensify @joekaufmanexpensify

Woah ok yeah, thanks @Puneet-here for the second video. It's clear now to me as well. @joekaufmanexpensify who is taking the first test run?

Let's also re-open this in the meantime.

I tried it on latest staging at web. It's hard to repro because sometimes you will get navigated to some middle step and sometimes to the start over option page. You have to keep trying to repro this.

I'll give it another try now!

I still can't reproduce. I tried a different combination of options like 10 times and still getting directed to the page with Continue with setup and Start over every time I click  Connect bank account, which is expected. I haven't been able to reproduce jumping directly from Connect bank account > Personal information.

@Puneet-here Is there a certain number of times you had to click Connect bank account to reproduce?

Sorry for not commenting this earlier, I was refreshing the page everytime before clicking the button, so it took me 4-5 tries.

Even with refreshing the page each time, I still can't reproduce on my end. I see Continue with setup every time I click Connect bank account. I think we'd need to be able to successfully reproduce the bug to proceed.
@JmillsExpensify you want to give it another try?

I think I have found the steps to repro this more easliy.

Decrease the screen size so that you don't see the LHN
Go to workspace > issue cards > connect bank account
Check what page is opening > dismiss the page and refresh and repeat the steps again

https://user-images.githubusercontent.com/101883770/194645484-56fcc4d9-a056-49c5-b7a9-7d320362e840.mov

Not overdue, going to try to reproduce this morning.

Ok cool, that sounds good!

Check what page is opening > dismiss the page and refresh and repeat the steps again

Regarding this part though, I struggle a bit whether we should spend time on this bug given this specific reproduction step. The mainline case is that you don't dismiss the page, refresh and then repeat the steps again. This is hard for us to reproduce, so I also don't think it's a priority. I think I'd still be in favor of closing.

Agreed. I don't think a user is going to complete these steps, given it involves several potential refreshes. Further, following the new reproduction steps above, I still can't reproduce on my end.
I'm going to close for now!


version bump on jemalloc

Hi,
Jemalloc 3.3.1 is quite old(March 6, 2013). According to the changelog, there're number of bug fixes and optimizations merged since then. Some of them may reduce the allocation overhead. Is it possible to do an upgrade on the jemalloc code?
I did a simple test with jemalloc-4.4.1 here, looks like it's working.
thanks, -yuan

Thank you for your advice, I have upgraded jemalloc to 4.1.0 on dev branch.

@ideawu, 可否考虑升级jemalloc到4.2.1


[DO NOT MERGE] Testing Needs-Review Labelling

Summary

Release Notes Entry

Testing putting my own comment on here...

Helping with some testing!


Fix light/display implementation

Included the necessary quorum types into the Python function names. I decided to set this up as a pull request because I was looking for anyone to potentially review/test before getting it pushed.
Also, I accidentally sent this PR to the professor's official release version of the project, so I had to scrap that PR and send it here instead.

Must resolve issue with a couple missing files


Replaced triangle net source with nuget

@phliberato

Good point @phliberato, there are also some unknowns on our end with packaging so let's leave this for now.
I'll ask @teocomi what he thinks of us publishing the triangle net on our nugets

Good point @phliberato, there are also some unknowns on our end with packaging so let's leave this for now. I'll ask @teocomi what he thinks of us publishing the triangle net on our nugets

We already have several nugets, so would be happy to do so for this one too :)

Reopened this now that we have our own NuGet published!


Chat input listening

Add some flow to handle input listening in chats

can be done through bot.update.flow actions.
(since 7.4.0)


Update package.json

There is a problem with old version of puppeteer on macbook with M1 silicon computer. Please update puppeteer dependency to 10.1.0.
It will work fine that way.

@ugljanin

npm ERR! cipm can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with npm install before continuing.

@vladikoff I don't have write access to this repo. I tried uploading both package.json and updated package-lock.json
Happy new year
remote: Permission to gruntjs/grunt-contrib-qunit.git denied to ugljanin.
fatal: unable to access 'https://github.com/gruntjs/grunt-contrib-qunit/': The requested URL returned error: 403


TypeError: unsupported operand type(s) for +: 'NoneType' and 'str', hash(self.name + self.channel.name)

Traceback (most recent call last):
  File "/home/arch/.local/lib/python3.10/site-packages/twitchio/websocket.py", line 315, in _process_data
    return await self._code(parsed, parsed["code"])
  File "/home/arch/.local/lib/python3.10/site-packages/twitchio/websocket.py", line 359, in _code
    self._cache_add(parsed)
  File "/home/arch/.local/lib/python3.10/site-packages/twitchio/websocket.py", line 488, in _cache_add
    self._cache[channel].discard(user)
  File "/home/arch/.local/lib/python3.10/site-packages/twitchio/chatter.py", line 56, in __hash__
    return hash(self.name + self.channel.name)
TypeError: unsupported operand type(s) for +: 'NoneType' and 'str'

When I fetch the same users as the initial channels
UAT = "some token"
user = "user"
class Bot(commands.Bot):
    def __init__(self):
        super().__init__(
            token=UAT,
            prefix="!",
            initial_channels=[user]
        )
    async def __ainit__(self):
        users = await self.fetch_users([user])
bot = Bot()
bot.loop.run_until_complete(bot.__ainit__())
bot.run()

I will add that I and other people on my project experience the exact same issue on our project, for the same reason.

Which version did you start getting this error on?

It was 2.4.0 and I just got the same error from master.

Okay... I changed some code.
UAT = "some token"
user = "user"
class Bot(commands.Bot):
    def __init__(self):
        super().__init__(
            token=UAT,
            prefix="!",
            initial_channels=[user]
        )
    async def event_ready(self):
        self.loop.create_task(self.__ainit__())
    async def __ainit__(self):
        users = await self.fetch_users([user])
bot = Bot()
bot.run()

And I don't know why it looks like working...
Doc says run():

A blocking function that starts the asyncio event loop, connects to the twitch IRC server, and cleans up when done.

I understood that bot.run() only connects to the event loop and the Twitch IRC server, so I thought my code would be okay because my code runs the event loop separately and fetch_users requests to the Twitch API server, not the Twitch IRC server.
Is there something wrong with my understanding?
Why doesn't it work if the same users are in the list of initial_channels?
commands.Bot __init__ do something more? Like affecting to fetch_users API request?

We've (read: @chillymosh ) been attempting to trace this issue, and so far we've determined that using loop.run_until_complete before bot.run causes nick to get set to None, thus causing this error.

Is there something wrong with my understanding?
nope, thats how its supposed to work in theory.
Why doesn't it work if the same users are in the list of initial_channels?
it actually doesnt have anything to do with that, as mentioned above, it seems to be usage of run_until_complete.

This is entirely a bug, we'll have it fixed for the next release (hopefully)

@preeded I believe I have resolved this in f111e676cbd2988ca8d99ba84b0b8be118354ae1 which you can test by installing TwitchIO direct from master

Thank you very much for your help! I just tested it on the master, and it worked well!


Creating a memory distributed database fails because of the automatic backup process

OrientDB Version: 3.0.37
Java Version: Java8
OS: Windows
Expected behavior
Database is created on all the nodes
Actual behavior
The memory database starts creation on first node by fails on the other node because the backup process can't create the backup files.
Steps to reproduce
Start 3 nodes
On node1, create a new memory database
Look the 3 server logs files
On one of the 2 remaining nodes, the process keeps waiting for the other node, which is blocked creating de backup files.
The automatic backup should be deactivated in the config/automatic-backup.json file "enabled":"false" but this has no effect

Hi @qualitesys
Yes, it's a known limitation, memory databases cannot be used in a cluster because the backup (and replication) procedure relies on the physical files, that of course do not exist in a memory db.
I think we can fix this problem in the near future, so I'm flagging it as an enhancement
Thanks
Luigi


Support for distributed/asynchronous optimization?

Both chocolate and optuna support asynchronous distributed optimization (e.g. via a shared SQLite database).
This is very handy when one has access to lots of machines sharing a same file system.
Each machine can ask for a new trial and tell the result (ie write to the on-disk database) in parallel.
Is this feature available in nevergrad out of the box?
Or does one need to build this oneself?

I just wanted to add that apart from using the ask and tell methods directly which give you a total freedom as @teytaud mentioned, you can also provide an executor like the ones in concurrent.futures to the optimize method. I for instance use a custom made executor to distribute the computation on several machines.

The optimizer is on one machine, and distributes the function evaluation on many machines; we do not use a sql database. This is fully asynchronous. The infrastructure is admittedly different; rather than having the optimizer run on each machine and share information through a file, in Nevergrad the optimizer runs on a master and the jobs (i.e. function evaluations) are launched (distributed...) via a job scheduler (e.g. Slurm).

Hi, note that this is now quite easy to do with Tune: https://github.com/ray-project/ray/blob/master/python/ray/tune/examples/nevergrad_example.py


Ensure Referral Handler UI and backend Service can be deployed

New approach. Attempt to remove the UI layer to see if the service will deploy. @sravandigitalhie

Update from @sravandigitalhie: Successfully got rid of the UI. When you test locally using Postman, it works. Still need to deploy on Azure and test using Postman again.


[Storage] Add test cases to support Revoke UserDelegationKeys

This is to add test case for a new feature: Revoke UserDelegationKeys
Upgrade the SDK version to 10.1.0, but we might won't release it with 10.1.0, but release it in a latest release with 11.0.0

@blueww So are you planning to release 10.1.0 or not?

We might not release, since we might will release a new feature on 2019-04-01 apiversion first.
But I would like to be prepared for the release, since I am not sure which feature will be release first.

We might not release, since we might will release a new feature on 2019-04-01 apiversion first.
But I would like to be prepared for the release, since I am not sure which feature will be release first.

Thanks for the information. I shall go ahead and merge this but please be cognizant of the versioning story depending on what API version you are working on.


Setting source and target levels Java 21

Description
Addressing items from global opensearch campaign of switching to jdk 21.
Issues Resolved
https://github.com/opensearch-project/neural-search/issues/775
Check List

[X] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

closing as it's duplicate of https://github.com/opensearch-project/neural-search/pull/838


You have a section on how to run it, but I don't know Go programming. I've installed it onto my computer, but I have no idea how to run the script itself.

Yeah, I'm also having a problem with this the section that tells you how to use is a bit out there doesn't help in the slightest.

You don't need to know Go language to use it (I don't know it either).
After you download it, move the .exe file into it's own folder, then run it once. It will generate the files and then close itself to give you a chance to edit the settings.
Open config.ini, fill in the [auth] section with your discord account (if you want to set up a dedicated discord-downloader that runs 24/7, it might be better to make a 2nd discord account and use that info instead if your main account)
The next section, [general] settings: skip edits, if someone edits their comments with new pics/links, if you want to ignore that or not. (personally I set it to false to get the edits too) - download tistory sites, if you dont know what this is, just leave it false - max download retries: if a download fails, it will keep trying this amount of times (personally I lowered it to 1 or 2 because 5 retries takes a long time) - download timeout: how long of trying to download before it gives up - send notices: might become spammy and get banned from servers if you set this to true to let the channels know you're downloading.
[channels] section, the account must be in the server for it to download from channels. (Recommended you do this from the browser version of Discord to see URLs) Lets say you join a server about sharing Cat pics, one of the channels is for fluffy kittens. You click on the #channel and in the URL it will say like discordapp.com/channels/SERVERID/CHANNELID - just copy/paste the CHANNELID into the file where it says "channelid1 =" and then after replace the "C:\full\path\1" with wherever on your system you want to save the pics, like for example "C:\Users\JohnDoe\Downloads\FluffyKittens", whatever you want. (Personally I also organize by server and then channel.)
If you're setting up a dedicated downloader, you should add a new section to the bottom of the file:
[interactive channels]
<your channel id> = <some valid path>

This part gets a bit messy and maybe confusing for new users. With the browser-based discord still open, you'll have to find a way to send a message to the account, maybe open up the discord client or use a 2nd computer, either of which logged into your main account. Send a message to the 2nd account just to start the chat. On the 2nd account in the browser, in the URL for that private message it should be like discordapp.com/channels/@me/CHANNELID - Copy/paste that CHANNELID into the  section (no brackets). The  doesnt matter, I don't think it does anything, just a path to wherever on your computer that's valid.
Save the config.ini. Run the .exe. Now with your main account, you can send the command "history" by itself to the 2nd account in a private message, then it will ask which CHANNELID do you want to download the entire history of. The only valid CHANNELIDs are the ones you set in the config.ini in the [channels] section. Just send the CHANNELID by itself and it will start downloading.
Unfortunately, if you're like me and set up 50+ channels, you'll have to do those last steps for every single channel that you set up. I wish there was a way to just dump a list of channels and have it go through each one so it can work on it's own and I won't have to be babysitting it...
Hope this helps. Sorry Seklfreak for contributing to necro issue. (March 10th)

@DreamPhreak Wow, thank you so much for this extensive guide!

@DreamPhreak Thanks for clearing that up much appreciated

Would it work with WINE on linux?

Maybe? I would recommend just building a native linux executable yourself.

I have no idea on how to make that. Also would the config.ini paths still work with linux paths?

Linux paths should work

Does the secondary option work with downloading images off the second account? Sent help in a DM and never get a popup telling me for an id.

i don't have any .exe in the file i downloaded

Ok I did everything the instructions had me do but the bot doesnt send any message for channel ID or anything. I have a pic of my settings bellow. What do I do!!?? Also, am I supposed to close the cmd after it has logged into the account? I'm just so confused

Don't understand for download history

It doesnt work for me... Here:
discord-image-downloader-go version 1.29.3
Client is now connected as Makein. Press CTRL-C to exit.
[auth]
token = ******************

[general]
skip edits                           = true
download tistory sites               = false
max download retries                 = 5
download timeout                     = 60
send notices to interactive channels = false

[channels]
channelid1 =<PHONE_NUMBER>00938498 E:\3\db
437254129400938498 = E:\3\db
437254129400938498 E:\3\db

[flickr]
api key = your flickr api key

[twitter]
consumer key        = your consumer key
consumer secret     = your consumer secret
access token        = your access token
access token secret = your access token secret

[interactive channels]
<your channel id> = <some valid path>
channelid4 =<PHONE_NUMBER>00938498 E:\3\db

Has this become obselete? I got it working last year but nothing I do now seems to work.

What is the history command and how do I run it?

Hello friends! Can I ask for some help? I've done and followed everything but my selfbot account doesn't respond to the history command? here is the config. I do not know if I have done anything wrong, but I've been at it for about an hour and a half now.
[auth]
token = *****************************************
[general]
skip edits                           = true
download tistory sites               = false
max download retries                 = 5
download timeout                     = 60
send notices to interactive channels = false
[channels]
482761204868317199 = C:\Users\Wanna\Pictures\LOONA\LOML2
channelid2 = C:\full\path\2
channelid3 = C:\full\path\3
[flickr]
api key = your flickr api key
[twitter]
consumer key        = your consumer key
consumer secret     = your consumer secret
access token        = your access token
access token secret = your access token secret
[interactive channel]
590510712711807007 = C:\Users\Wanna\Pictures\LOONA\LOML2

Hello friends! Can I ask for some help? I've done and followed everything but my selfbot account doesn't respond to the history command? here is the config. I do not know if I have done anything wrong, but I've been at it for about an hour and a half now.
[auth]
token = *****************************************
[general]
skip edits = true
download tistory sites = false
max download retries = 5
download timeout = 60
send notices to interactive channels = false
[channels]
482761204868317199 = C:\Users\Wanna\Pictures\LOONA\LOML2
channelid2 = C:\full\path\2
channelid3 = C:\full\path\3
[flickr]
api key = your flickr api key
[twitter]
consumer key = your consumer key
consumer secret = your consumer secret
access token = your access token
access token secret = your access token secret
[interactive channel]
590510712711807007 = C:\Users\Wanna\Pictures\LOONA\LOML2

don't close the cmd exe. try sending history while the exe is still open.


[BUG] Auth will affect other datasource except tidb

Describe the bug

When i use auth, i find that other datasource(except tidb) such as hive will be affected.
for example,show databases can't show hive's database and select xxx from hivetable will fail with no authorization
What do you expect

auth will not affect hive and other datasource

Spark and TiSpark version info

spark: 3.0.x, 3.1.x
tispark version: 5.0

It's because auth doesn't distinguish which type the datasource is.
So if you select hive table with spark SQL, auth module would look up privilege info in TiDB, which would cause Exception.

I think we should use another tag instead of bug.
@zhangyangyu PTAL

@zhangyangyu can you help to change the lable to type/feature-request


Mixing arabic and English  issue

When mixing Arabic and English in the same text the text displays incorrectly

The same issue happens when mixing Arabic text with numbers.

The recommended workflow at this time is to translate (or maybe transliterate) all English words into Arabic. Mixing just doesn't work well.

I have fixed this issue and also implemented the autowrap support. Previously, if the text cannot fit in one line and wrap happens the lines appear reversed, I have implemented a fix, please review it.

Fixed in #9


Question: why not use sprites?

Hey @bendiksolheim ! Great work you are doing on the emojiarea! I was just wondering why you don't use png sprites instead of separate png files? this looks like it results in a LOT of requests and slows down page loading.
cheers!
Wolfgang

okay, I just saw you forked from a version which already converted the sprites into individual images. the original version uses sprites! might be worthy of considering.

Hello @wstoettinger !
That is a valid question. I have to admin that this repository is more or less dead, I have no intention of continuing with it. It was more of a test to see if it was possible at all to render emojis in a textarea in a predictable manner. Sadly, the project I was going to use it in decided not to implement this, so I stopped developing on it.

Hey @bendiksolheim! Thanks for the info! I found your repo because it was the fork that seemed to have the most recent development of the original repo. I then decided to develop one from scratch inspired by OneSignal/emoji-picker and diy/jquery-emojiarea because nothing I could find offered the flexibility and functionality I was looking for while still maintaining some light weight (as far as this is possible with emoji libraries).
In case you need something similar in the future or you are curious, feel free to check out my repo here: https://github.com/wstoettinger/jquery.emojiarea.js
I'm still working on the browser support, especially when it comes to copy, paste, undo and redo actions (As usual IE is a pain in the ass here!).

Glad you got things sorted :)  I’ll definitely keep your respository in mind!
Closing this, as you seem to have found a solution.


Can't submit form.

I am trying to get a form to submit in VBS, but the form thinks i'm not entering anything and just refreshes the page.
selenium.FindElementById("j_username").SendKeys strUserName
selenium.FindElementById("j_password").SendKeys strPassword
selenium.FindElementByName("continue").Click

Page: https://connect.secure.wellsfargo.com/auth/login/present
Any clue what I need to do?   I also tried:
selenium.FindElementById("j_username").Attributes("value") = strUserName
selenium.FindElementById("j_password").Attributes("value") = strPassword
selenium.FindElementByName("continue").Click

and still no luck.  I prefer stetting value if possible, but I have had no success in this.
Update: Ok, so after further research, I now know it appears to be a Google Chrome (chromedriver.exe) issue.   I am able to run this code in Microsoft Edge, and it appears to work fine.

I tried in my computer both method are working.  Driver enters the values in the form as well as "Submit" button is also "Sign On" button also clicking.
I used this code:
driver.Start "chrome"
driver.Get "https://connect.secure.wellsfargo.com/auth/login/present"
a = "xxx"
b = "yyy"
driver.FindElementById("j_username").SendKeys a
driver.wait(1500)
driver.FindElementById("j_password").SendKeys b
driver.wait(1500)
driver.FindElementByName("continue").Click
driver.Wait (15000)
I used dummy values so page doesn't redirect to next page instead refreshes the same page. You try this with your real values.
If you still facing issue tell you mail or any other source I send my chromedriver.exe file, you try updating it in seleniumbasic folder and try the same code.
Regards,
Balaji

I just tried the code you provided and same issue.
If it was successful (even with your dummy values), it would state "We do
not recognize your username and/or password. Please try again or visit",
however, just the page refreshes and no error and fields end up blank.  I
tried updating to the latest chromedriver.exe from
https://sites.google.com/a/chromium.org/chromedriver/downloads.  I am
running 2.23.  Chrome Version: 52.0.2743.116.
On Wed, Aug 31, 2016 at 7:32 PM, balajisekar<EMAIL_ADDRESS>wrote:

I tried in my computer both method are working. Driver enters the values
in the form as well as "Submit" button is also "Sign On" button also
clicking.
I used this code:
driver.Start "chrome"
driver.Get "https://connect.secure.wellsfargo.com/auth/login/present"
a = "xxx"
b = "yyy"
driver.FindElementById("j_username").SendKeys a
driver.wait(1500)
driver.FindElementById("j_password").SendKeys b
driver.wait(1500)
driver.FindElementByName("continue").Click
driver.Wait (15000)
I used dummy values so page doesn't redirect to next page instead
refreshes the same page. You try this with your real values.
If you still facing issue tell you mail or any other source I send my
chromedriver.exe file, you try updating it in seleniumbasic folder and try
the same code.
Regards,
Balaji
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/florentbr/SeleniumBasic/issues/124#issuecomment-243959275,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUzhKdi63b-F8OpB3w54P4y0jzGeB9sks5qljk1gaJpZM4JyO2Q
.

The problem is because of driver not code.
After clicking the "Sign On" button in the driver, take the URL and paste it somewhere.
Do the same in normal chrome browser and take the URL and paste it somewhere, you might see the some difference say "LOB=CON" like this.
I would like to know did you tried with your correct login values still you face the same problem.

Yes.  Same problem with my actual credentials and/or fake ones.
If I manually edit the username after it populates through the code, on the
page, it works.   It just doesn't see the codes input for the username it
seems like even though I see it populated.
On Aug 31, 2016 8:39 PM, "balajisekar"<EMAIL_ADDRESS>wrote:

The problem is because of driver not code.
After clicking the "Sign On" button in the driver, take the URL and paste
it somewhere.
Do the same in normal chrome browser and take the URL and paste it
somewhere, you might see the some difference say "LOB=CON" like this.
I would like to know did you tried with your correct login values still
you face the same problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/florentbr/SeleniumBasic/issues/124#issuecomment-243967566,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUzhGsrl5e5g_O9TupSqBuQjZn4ncHCks5qlkjygaJpZM4JyO2Q
.

I just wiped my computer and did a fresh install of Google Chrome (32-bit
version) and SeleniumBasic to verify it wasn't something weird with my
setup.   Updated Google Chrome driver to the latest manually.   And now
when I run the same code I was running before, I am getting "Automation
Error"
Any clue what would cause that?
The error happens at 2nd line (Driver.start "chrome")
On Wed, Aug 31, 2016 at 8:39 PM, balajisekar<EMAIL_ADDRESS>wrote:

The problem is because of driver not code.
After clicking the "Sign On" button in the driver, take the URL and paste
it somewhere.
Do the same in normal chrome browser and take the URL and paste it
somewhere, you might see the some difference say "LOB=CON" like this.
I would like to know did you tried with your correct login values still
you face the same problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/florentbr/SeleniumBasic/issues/124#issuecomment-243967566,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUzhGsrl5e5g_O9TupSqBuQjZn4ncHCks5qlkjygaJpZM4JyO2Q
.

Well, as shown above, I wiped the machine, installed latest version of SeleniumBasic, latest version of GoogleChromeDriver, and still same issue.

Hey meluvalli,
The issue is with website not in computer. I think some sites have scripts that will disable auto logging in their website to avoid scamming, Phishing, etc.

I would be good with that answer if it didn't work with Microsoft Edge.
However seeing as the code works on Microsoft Edge, can't be there site.
On Sep 1, 2016 3:04 PM, "balajisekar"<EMAIL_ADDRESS>wrote:

Hey meluvalli,
The issue is with website not in computer. I think some sites have scripts
that will disable auto logging in their website to avoid scamming,
Phishing, etc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/florentbr/SeleniumBasic/issues/124#issuecomment-244227045,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUzhGBdmPn8UXUL5nEMmJ0aXcUjj3Y-ks5ql0wBgaJpZM4JyO2Q
.

I don't know how it worked in edge.

Has to be issue with chromedriver would be my guess.
On Sep 1, 2016 3:39 PM, "balajisekar"<EMAIL_ADDRESS>wrote:

I don't know how it worked in edge.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/florentbr/SeleniumBasic/issues/124#issuecomment-244234027,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUzhDICC2xor3FYna54v7uXWtCWtIvfks5ql1QFgaJpZM4JyO2Q
.

Did you tried it in firefox.

I have not tried in FireFox or IE.   Just Chrome and Edge.   I can later today.


Raspberry PI 4 is not starting when using the image

I have tried to use this image in several raspberry 4 devices and it seems that is not booting as expected so I can not see my device in my relative fleet.
How to reproduce

Go in that page and create a new fleet by clicking in the button
Add a device in the newly created fleet (choose raspberry pi 4 as the device of preference)
Use etcher to boot the SD card with the image that I have downloaded
Put the image in the raspberry pi 4

Expected behavior
To see the device in my fleet after some seconds and be able to use Node Red when I enable the public URL.
Actual behavior
Once it is powered on, both green and red light of the raspberry pi stays on. I don't see the device in my fleets or I see anything when I enter the HDMI. I have tried the same process to different devices.
Notes
Looking at the code I think that the base image should be referencing the version 4 but I have not tried to change that yet, neither I am sure that this is the cause of the problem.

Hey there,
Sorry it's taken so long for us to get back to you. I'd point out how busy everyone is, but that's not a decent excuse. We are actively thinking through how to better support our blocks, or archive them if we can't support all the things we've ever made.
In this case, the issue wasn't about using a Pi3 image on a Pi4. A Pi4 can run base 32-bit images just fine.
The issue was that it was running a really old version of debian, and the packages were not being found any more. I've updated the base images and the packages to reference Python3, and tested it all on a Pi4 and a Pi3 for completeness.
All seems fine now.


Fix Ruby syntax errors in "filters_post"

Hi, we were encountering syntax errors in Ruby which cause the pipeline to crash and restart in a loop, if this section is present in your ingestor config. Seems to be due to some missing brackets. Thanks!

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

Thanks @Rylon !


Add module for reading and writing AMF packets

Also fixes two bugs in AMF0:

Reading AMF3 tagged stuff should be a Value, not an Object.
Writing AMF3 tagged objects should prefix it with the AMF3 type

All the clippy failures are pre-existing!


refactor light-oauth2 service to ensure integration with light-portal

The following has been changed.

Add copyright headers
Update GetHandler to filter by host instead of serviceId and return a map with services and total instead of only one array for UI pagination.
Update PostHandler and PutHandler to get ownerId from the JWT token and return the updated service to the caller.
Update openapi.yaml in test folder to adjust the get query parameter with host, change the service schema and update the scope to portal.r and portal.w

 refactor light-oauth2 service to ensure integration with light-portal light-oauth2 master


Wrong Query Generation for Skip() and Take

From @chbach on November 18, 2015 20:36
Hi, I recently switched from SQLite to Postgres on EF7 and noticed the following bug. When using Skip() and Take() (LIMIT and OFFSET), some relational data would be missing in the resulting collection of objects.
I have two models, NewsEntry and Tag that are related in a many-to-many relation via the table Tagging. When I query all NewsEntries with Db.NewsEntries.Include(e => e.Taggings), everything works fine and the Tagging-relation is present and I can iterate over them. However, as soon as I start doing something like Db.NewsEntries.Include(e => e.Taggings).OrderByDescending(e => e.Date).Take(12) some of the Tagging-relations are missing.
I analyzed the generated queries and noticed the following thing. This is the generated query for the relation:
SELECT "t"."NewsEntryId", "t"."TagId"
FROM "Tagging" AS "t"
INNER JOIN (
    SELECT DISTINCT "e"."Date", "e"."Id"
    FROM "NewsEntry" AS "e"
    LIMIT 12
) AS "e" ON "t"."NewsEntryId" = "e"."Id"
ORDER BY "e"."Date" DESC, "e"."Id"

What seems odd to me is that the ORDER BY statement is at the very end and sorts the overall result, but should rather sort the NewsEntries, since we want to get the first 12 elements after sorting:
SELECT "t"."NewsEntryId", "t"."TagId"
FROM "Tagging" AS "t"
INNER JOIN (
    SELECT DISTINCT "e"."Date", "e"."Id"
    FROM "NewsEntry" AS "e"
    ORDER BY "e"."Date" DESC, "e"."Id"
    LIMIT 12
) AS "e" ON "t"."NewsEntryId" = "e"."Id"

You can compare the query with the query the SQLite adapter produces here: https://github.com/aspnet/EntityFramework/issues/3758
Can you please have a look into that?
Christoph
EDIT:
Moving the sorting to the front like Db.NewsEntries.OrderByDescending(e => e.Date).Include(e => e.Taggings) actually generates the right query. However there are still Taggings missing! I have no idea how that happens.
Copied from original issue: npgsql/npgsql#872

From @SimonTouchtech on November 20, 2015 16:6
Seems to be the same as aspnet/EntityFramework#3732?

From @chbach on December 6, 2015 19:26
In fact, I didn't observe this behaviour with SQLite, it only appeared after I switched from SQLite to PostgreSQL. Apart from the wrong query generation, there are still relations missing in the final collection, so something with Include() in combination with Skip() and Take() and the population of relations must be wrong.

@chbach, as @SimonTouchtech suggests this seems to be the same as https://github.com/aspnet/EntityFramework/issues/3732, which is a core EF7 and not an Npgsql problem. Can you please confirm and close this issue if so?

Hi, i have this error in 1.0.1.
Then i use include - Order by not working

If i have select before order - it working
           var entities = this.dbSet.Include(x => x.User).Select(x => new GameRewardLogsGridBackendModel             {                 Id = x.Id,                 DateReward = x.DateReward,                 Place = x.Place,                 Reward = x.Reward,                 RewardId = x.RewardId,                 GroupingId = x.GroupingId,                 Description = x.Description,                 UserId = x.UserId,                 UserName = x.User.Name,                 DispositionId = x.DispositionId             }).OrderByDescending(u => u.DateReward)                 .Skip(skip)                 .Take(take);
and not working (ordering by Include(x => x.User))
           var entities = this.dbSet.Include(x => x.User).OrderByDescending(u => u.DateReward)                 .Skip(skip)                 .Take(take).Select(x => new GameRewardLogsGridBackendModel             {                 Id = x.Id,                 DateReward = x.DateReward,                 Place = x.Place,                 Reward = x.Reward,                 RewardId = x.RewardId,                 GroupingId = x.GroupingId,                 Description = x.Description,                 UserId = x.UserId,                 UserName = x.User.Name,                 DispositionId = x.DispositionId             });

Hi,
I have the same problem. I reported it on EF github but I wonder if it's not related the provider.
https://github.com/aspnet/EntityFramework/issues/6492
this code :
context.Produits.Skip(skip).Take(take).Include(c => c.ProduitCategories).ThenInclude(c => c.Categorie).ThenInclude(c => c.Famille).ToList()
seems to produced :
SELECT "p0"."Produit_ProduitId", "p0"."Categorie_CategorieId", "c"."CategorieId", "c"."Famille_FamilleId", "c"."OrdreAffichage", "c"."Titre", "f"."FamilleId", "f"."OrdreAffichage", "f"."Titre"
FROM "ProduitCategorie" AS "p0"
INNER JOIN "Categories" AS "c" ON "p0"."Categorie_CategorieId" = "c"."CategorieId"
INNER JOIN "Familles" AS "f" ON "c"."Famille_FamilleId" = "f"."FamilleId"
WHERE EXISTS (
    SELECT 1
    FROM "Produits" AS "p"
    WHERE "p0"."Produit_ProduitId" = "p"."ProduitId"
    LIMIT 10 OFFSET 10)
ORDER BY "p0"."Produit_ProduitId"

Now with a rapid analyze of this sql code, I understand why it doesn't work...
The SELECT in the EXISTS clause only return 1 row... when an OFFSET occured (more than 0) obviously the result is false.
So, I'm not a expert on SQL performance, but a query like this do the job :
SELECT "p0"."Produit_ProduitId", "p0"."Categorie_CategorieId", "c"."CategorieId", "c"."Famille_FamilleId", "c"."OrdreAffichage", "c"."Titre", "f"."FamilleId", "f"."OrdreAffichage", "f"."Titre"
FROM "ProduitCategorie" AS "p0"
INNER JOIN "Categories" AS "c" ON "p0"."Categorie_CategorieId" = "c"."CategorieId"
INNER JOIN "Familles" AS "f" ON "c"."Famille_FamilleId" = "f"."FamilleId"
WHERE EXISTS (
    SELECT 1
    FROM (SELECT "p1"."ProduitId" FROM "Produits" as "p1" LIMIT 10 OFFSET 10) AS "p"
    WHERE "p0"."Produit_ProduitId" = "p"."ProduitId"
    )
ORDER BY "p0"."Produit_ProduitId"

Is this problem is related to EF core or to the PGSQL provider ?

I don't have time to dive into this right now, but there's little chance this is an Npgsql provider issue - it's most likely an EFCore issue. One easy way to help out would be to simply test this on SqlClient with LocalDB - it's really trivial to get a small model together and run this query (see the EFCore getting started). This will at least tell us exactly where the problem is.

Closing for each, likely to have been fixed in EF Core a long time ago.


Should 'using' suppress exceptions from @@dispose when already throwing an exception

This was brought up by @waldemarhorwat in the July TC39 meeting. Java's try-with-resources block will suppress exceptions raised when disposing resources if already propagating an exception from the body.
However, there are two points that cause me concern about adopting this behavior:

This behavior would differ significantly from a regular try..finally statement.
Java has a way to retrieve these suppressed exceptions by calling the Throwable.getSuppressed() method on the exception thrown. ECMAScript does not have a reliable way to provide this as it is possible to throw a non-Error value.

Should we consider following Java's lead here and perform the same suppression?

My intuition here is to match try/finally and not suppress the error, but this is not a strongly held opinion.

I think we should not suppress the exception.

Addressed by the addition of AggregateError in #26.


chore: Update testcases to reflect operation doc codegen changes.kt

This PR is to resolve downstream CI issue in smithy-swift repo for PR #578.
Issue #

N/A
Description of changes

Changes the string matched in test case to reflect code generation change that adds documentation to operations.
New/existing dependencies impact assessment, if applicable

N/A
Conventional Commits

N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/test

/test

/test

/test

/test


gpg verify instructions don't work

Getting started doc says:
Download the latest version of fcct and the detached signature from the releases page. Verify it with gpg:
gpg --verify <fcct binary> <detached sig>

However:
$ gpg --verify fcct-x86_64-unknown-linux-gnu fcct-x86_64-unknown-linux-gnu.asc
gpg: no valid OpenPGP data found.
gpg: the signature could not be verified.
Please remember that the signature file (.sig or .asc)
should be the first file given on the command line.

So it seems that those two files should be swapped:
$ gpg --verify fcct-x86_64-unknown-linux-gnu.asc fcct-x86_64-unknown-linux-gnu
gpg: Signature made Thu 11 Jul 2019 12:46:56 AM CEST
gpg:                using RSA key 9CEB8FE6B4F1E9E752F61C82CDDE268EBB729EC7
gpg: Can't check signature: No public key

That works better, but complains about no public key. I'm not very fluent with gpg, so I might be doing something wrong.

#11 fixed the ordering of the signature file in the readme, so you are correct about needing to flip the two files.
To resolve that public key, try running  gpg2 --receive-keys  9CEB8FE6B4F1E9E752F61C82CDDE268EBB729EC7

also make sure you have imported the CoreOS application public signing key from here
http://coreos.com/security/

OK, thanks. It might be a good idea to link this in the doc, because a lot of people will have no idea about these prerequisite steps.


fix(curriculum): update video ids

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

This updates video ids for:

Working with Higher Order Functions and callbacks
still to update:

[ ] 673362a34edda41dedf87623
[ ] 673362b3f763ae1e38e17df7
[ ] 673362be2f70c21e65bc5459
[ ] 673362d7f94d551edb532d24
[ ] 673362e43d57b51f1ad2d466

Understanding core JavaScript Fundamentals
still to update:

[ ] 6732c6ab21031b60d2b0c999
[ ] 6732c6ba2ea42b610b9f9ce1
[ ] 6732c6c72d3738614e1230a2
[ ] 6732c6e281c14a61c4858361
[ ] 6732c69d82814160951b1aa7

Understanding Computer, internet and tooling basics
still to update:

[x] 672ab82c1a9bbd0e3aabc39d
[x] 672ab83c4297910eade53c2e
[ ] 672ab849aa1ef70eefd29364
[ ] 672ab8573f32480f192aaae1

Debugging techniques
still to update:

[x] 6733becf4b0c353553b9bfa4
[x]  6733befb703ca6361da3755b

Working with maps and sets

It looks like we have a video now for HTML entities
https://www.youtube.com/watch?v=mVdIq0H_B4E

It seems it's been already updated https://github.com/freeCodeCamp/freeCodeCamp/blob/397c4a5372a34e69dd8278a21d6fe041d597a01e/curriculum/challenges/english/25-front-end-development/lecture-html-fundamentals/67083868d5fdcb17bf8c14bd.md?plain=1#L5

Some of the links in the computer basics module were updated so the ids will need to updated here

Some of the links in the computer basics module were updated so the ids will need to updated here

I found only one among those updated in this PR @jdwilkin4 . Do you mean that some of those already added to main needs update?

It looks like you added the updated one already 👍


Remove price calculation

Closes #1758

@Uxio0 this shouln´t be included on the next release until 30th November but could be in staging until that day.


Research report for Turing

2-3 page report including:

The Turing research project or projects you have been working on since joining the Institute and any progress to date. Please also include the details of collaborations with other Turing researchers, connections/collaborations with Turing partners and any external collaborations.
The research you plan to undertake over the next 12-24 months, outlining how your research aligns to the Institute’s strategic priorities.
Publications that include acknowledgement of the Turing support.

Deadline 22nd August

Done! 🎉


Fix Simple Types Encoding In cb0r_write

There seems to be a wrong encoding there...

Oh, closed as duplicate of #2


Osc, small optimization

Directly cast the packet into a message or a bundle instead of using tostring and reparsing (also saves quite some gc)

I'm not sure how to merge this pull request without fixing the crash on change OSC port... :(


Yatin: Watch +91<PHONE_NUMBER>

Yatin requests the watch of the watch_number +91<PHONE_NUMBER>. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by tripleee in Charcoal HQ

Approved by tripleee in Charcoal HQ


Error Install usql via Go on Arch Linux

I'm trying to install usql via go using the command: go get -u -tags all github.com/xo/usql but I'm getting the following error:
# github.com/alexbrainman/odbc/api
go/src/github.com/alexbrainman/odbc/api/api_unix.go:12:11: fatal error: sql.h:
No such file or directory
 // #include <sql.h>
           ^~~~~~~
compilation terminated.
# pkg-config --cflags oci8
Package oci8 was not found in the pkg-config search path.
Perhaps you should add the directory containing `oci8.pc'
to the PKG_CONFIG_PATH environment variable
Package 'oci8', required by 'virtual:world', not found
pkg-config: exit status 1

My Go version is go1.10.3 linux/amd64 and I'm using Arch Linux

You're building with the -tag 'all' -- this includes Oracle and ODBC support, both of which require third-party C dependencies. For ODBC, at least, most Linux distributions have a package named unixodbc or unixodbc-dev that you can easily install:
# on debian/ubuntu systems
$ sudo apt install unixodbc unixodbc-dev

# on arch systems
$ sudo pacman -S unixodbc

# additionally on macOS
$ brew install unixodbc

Building Oracle is a bit more tough, as it requires downloading Oracle's Instantclient SDK and installing it / configuring it to work properly with pkg-config, which I will not cover here.
I would suggest that instead of building with -tag all that you instead build with -tag most, which is the same as all but excludes the ODBC and Oracle drivers. Most users are unlikely to need ODBC or Oracle drivers -- and likely if you do, you probably already have them installed / configured.


AVX2 warning?

When I run the example I get:
This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
Should I be concerned? The version of libtensorflow that I downloaded was as instructed on https://www.tensorflow.org/install/lang_c:
Windows CPU only | https://storage.googleapis.com/tensorflow/libtensorflow/libtensorflow-cpu-windows-x86_64-2.5.0.zip

Hi @logixworx
No, you shouldn't be concerned. The provided binaries are compiled for generic architectures and therefore they don't use all optimizations that your cpu offers. Unless you are worried on performance you shouldn't be worried. If you want to use all your cpu operations then you will need to compile the tensorflow library yourself.


KaliLinux-NICE-DCV and assignStaticIP issue

Hello.
I tried to deploy the latest KaliLinux-NICE-DCV.yaml template and go the issue with assignStaticIP:
$ wget https://raw.githubusercontent.com/aws-samples/amazon-ec2-nice-dcv-samples/ce2c1345349d979fe9156b009f00e24f22201230/cfn/KaliLinux-NICE-DCV.yaml

$ rain deploy --yes KaliLinux-NICE-DCV.yaml KaliLinux-NICE-DCV --params assignStaticIP=yes,ec2KeyPair=wordpress-test,vpcID=vpc-026739ec964c7e79e,subnetID=subnet-096568d329a921083,allowWebServerPorts=HTTP-and-HTTPS

error creating changeset: Parameter 'assignStaticIP' must be one of AllowedValues

Thanks for looking at it...

Hello!
Default value for assignStaticIP is Yes

rain deploy --yes ...  --params assignStaticIP=yes ...

CloudFormation is case-sensitive,  use  Yes instead of yes
rain deploy --yes KaliLinux-NICE-DCV.yaml KaliLinux-NICE-DCV --params assignStaticIP=Yes,ec2KeyPair=wordpress-test,vpcID=vpc-026739ec964c7e79e,subnetID=subnet-096568d329a921083,allowWebServerPorts=HTTP-and-HTTPS

Thank you for your quick reaction.
It seems line yes -> Yes didn't help and rain without the assignStaticIP doesn't seems to work as well.
It seems like it is a problem of rain.
Does any of these commands work for you?
$ rain --version
Rain v1.19.0 darwin/arm64

$ rain deploy --yes KaliLinux-NICE-DCV.yaml KaliLinux-NICE-DCV --params assignStaticIP=Yes,ec2KeyPair=wordpress-test,vpcID=vpc-026739ec964c7e79e,subnetID=subnet-096568d329a921083,allowWebServerPorts=HTTP-and-HTTPS

$ rain deploy --yes KaliLinux-NICE-DCV.yaml KaliLinux-NICE-DCV --params ec2KeyPair=wordpress-test,vpcID=vpc-026739ec964c7e79e,subnetID=subnet-096568d329a921083,allowWebServerPorts=HTTP-and-HTTPS

None of these works for me, but I'm able to deploy your CloudFormation template with aws cloudformation deploy:
$ aws cloudformation deploy --capabilities CAPABILITY_IAM --parameter-overrides ec2KeyPair=wordpress-test vpcID=vpc-0aff7a314408ad32b subnetID=subnet-03431286e23f54b8e allowWebServerPorts=HTTP-and-HTTPS --stack-name KaliLinux-NICE-DCV --template-file KaliLinux-NICE-DCV.yaml --s3-bucket my-bucket

Waiting for changeset to be created..
Waiting for stack create/update to complete
Successfully created/updated stack - KaliLinux-NICE-DCV

If you agree I would like to close this ticket and open a new one with rain.
Thank you...

Does not work for me either.
% rain --version
Rain v1.19.0 darwin/arm64
% rain deploy --yes KaliLinux-NICE-DCV.yaml KaliLinux-NICE-DCV --params ec2KeyPair=MyKeyPair,vpcID=vpc-632fde01,subnetID=subnet-8d559fc1,allowWebServerPorts=HTTP-and-HTTPS,assignStaticIP=Yes
error creating changeset: Parameter 'assignStaticIP' must be one of AllowedValues

Think I know why.
In my template, you will notice that Yes and No are enclosed in double quotes.
  assignStaticIP:
    Type: String
    Description: Associate static public IPv4 address ( https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/elastic-ip-addresses-eip.html )
    AllowedValues:
      - "Yes"
      - "No"
    Default: "Yes"

  allowWebServerPorts:
    Type: String
    Description: Allow HTTP and/or HTTPS inbound
    AllowedValues:
      - "No"
      - HTTP-and-HTTPS
      - HTTP
      - HTTPS

If we change the "Yes" and "No" to Yes and No and upload to CloudFormation console, the options will appear as true and false.
Tried enclosing Yes in quotes and double quotes. Still does not work
% rain deploy --yes KaliLinux-NICE-DCV.yaml KaliLinux-NICE-DCV --params ec2KeyPair=MyKeyPair,vpcID=vpc-632fde01,subnetID=subnet-8d559fc1,allowWebServerPorts=HTTP-and-HTTPS,assignStaticIP="Yes"
error creating changeset: Parameter 'assignStaticIP' must be one of AllowedValues

Guess rain is interpreting Yes as a true/false boolean instead of Yes/No string
I agree. We can close this ticket
Thanks for the support


Moving Pipe on ITransformation to Buffers Experimental

Pipe added in https://github.com/dotnet/corefxlab/pull/1574
cc @KrzysztofCwalina

looks good


Remove an extra new line in the Conclusion list

It's not consistent with the next Appendix list, and the extra line is causing the list items to have <p> tags inside.

The extra <p> tags don't seem to affect real browsers' rendering, but they pose extra challenges to my toy browser: margin collapsing of parent-child (<li> and <p>) and then siblings (<li> and <li>); and <li> not being InlineLayout 😃

Thank you!


[mysql] Snapshot split the chunks asynchronously

The snapshot splitting process is now blocked. We need to optimize this process to make it asynchronous splitting.

Implemented in #931


[KOGITO-3791] Accumulate in a query used for automatic generation of REST endpoint of Rule Units

Many thanks for submitting your Pull Request :heart:!
Related
https://github.com/kiegroup/drools/pull/3541
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Your code is properly formatted according to this configuration
[ ] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

How to retest this PR or trigger a specific build:

Pull Request
Please add comment: Jenkins retest this

Quarkus LTS checks
Please add comment: Jenkins run LTS

Native checks
Please add comment: Jenkins run native

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

jenkins retest this

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.


correct the sequence number

correct the sequence number
1 -> 2 -> 3
[ ] Configuration Infrastructure
[X] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[-] Does not have any changes that may affect Istio users.

Hi @wangchenglong01. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@googlebot I signed it!

Both are valid md syntax: https://www.markdownguide.org/basic-syntax/#ordered-lists

Both are valid md syntax: https://www.markdownguide.org/basic-syntax/#ordered-lists

Yes, you are right. I will close it.


set x and y element to 0 then it displayed below each other

If you have a feature request, please try to implement it before requesting it. This is free software and the author is busy with other projects.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Is it possible to add grid box dynamically without giving x and y position if we will give x and y position to all element then it is added below to next element but my request is if we provide x and y axes 0 in all elements then grid should be displayed side by side rather than below each other.

Have you thought about rendering the grid items in a loop? Then based on your index you can use the modulo operator to calculate your x position. E.g. assuming you have grid items with the width of 3, then x = (index % 3).
If your items have variable width then you can always use the bin packing algorithm to pack your items in the layout.

Have you thought about rendering the grid items in a loop? Then based on your index you can use the modulo operator to calculate your x position. E.g. assuming you have grid items with the width of 3, then x = (index % 3)*3, so your items will be render besides each other in 3 columns.
If your items have variable width then you can always use the bin packing algorithm to pack your items in the layout.

I don't know about bin packing algorithm can you please explain

modulo

There are plenty of NPM modules that implement bin-packing algorithm, you have to look into it ;)
But the other solution by calculating the x position on the fly works as well.
I am not quite sure what you want to do tho, a bit more information/code sections would be cool.

I would like to have add column dynamically and that column should be added besides the previous column.
For example,
If the width of the column is 3 then there should be 4 columns in the first row and when I add next column then should be added to next column.

In this case it is simple math, you can always calculate the x,y position based on the number of the items. Like I mentioned above x = (index % number_of_columns) * width should work. So when you dynamically add a new item to your layout, which is going to be at the 5th place for instance, then x = (5 % 4) * 3.

@davidmunkacsi93 Thank you for help.


Fix some AddComponentMenu paths

Overview
Three components used identical MixedRealityKeyboard paths, and one was missing the Scripts entry in the path.

/azp run


Plugin Repository

Ability to download, install, enable, disable plugins via core API's. And perhaps a core plugin...

Fixed in dev by @plafosse.


[Discuss] Support custom networks

I create this, because the support for custom networks is not yet really there in ark-ts, but it's needed if we want to support custom networks in ark-mobile (which I'm implementing right now).
Discussion points:
NetworkType
What about the NetworkType. Should we just omit it on custom networks or should we introduce new enum values (which I personally find cleaner). I suggest even introducing two values. Though I'd just always use CustomNet at the moment,.
FindGoodPeer method
I think most of ark-ts will work right away. One point where it breaks is the method:  findGoodPeer.
The big question is: What is the real intention of this method? Because in ark-mobile we have an own findGoodPeer method. This method already gets a list of peers and selects the best peer based on the height and the delay. This method uses the findGoodPeer method of ark-ts only as a fallback and ark-ts then uses the static peer list from the config file to find the best peer.
If this is really the intention of the ark-ts method we can probably just do the changes which you see in this PR (or should be just return null as a peer then?). However - and I know this breaks the api - I strongly recommend renaming this method then to something like findGoodPeerFromConfig. Because the intention of the method is really not clear.
If the intention of the method is not to just find peers based on the static list, but should just find the best peer in general, then we have to rewrite it. We could basically move the logic from ark-mobile to this class and use the static peer list only as a fallback mechanism (very similar to how ark-desktop does it).
Peer (model)
I noticed there is not protocol member defined. Is this intentionally missing (because only http is supported) or should there be an enum for it (i.e. Http and Https)?

Well, this is the simple solution. However, I think we could implement a ConfigManager, something similar to the ark-js v2.
Yes, an enum should look good.

What is the simple solution?  Could you maybe be a little more specific concerning my questions?

You are right about the findGoodPeer method. Will move the logic from ark-mobile.
The problem is that we use the config object for things other than peers. In this library it is used in HDNode but in ark-mobile we also use the same object. So my point is, it should have a config manager to add/get the configuration (instead of a static file) to the specific network.
But, my real intention is to replace ark-ts to ark-js#v2 on ark-mobile. So we don't need to make major changes here, because it will be deprecated soon.

I see thanks.
Then I recommend just merging this stuff here back - or even nothing at all? :D Because the only benefit of this PR in this case would be clearer error messages.


Perform websocket request during onDefinition

When connection.onDefinition is invoked, I'm hoping to send a websocket request and await its response before returning the definition Location, which takes the user to that location in VS Code. The execution of connection.onDefinition completes before the websocket request/response finishes.
I've experimented with using a callback function. First, by having the callback return the desired Location, but that does not trigger that location to open in VS Code. Secondly, I tried having the callback invoke connection.onDefinition with updated parameters, and making sure to return the Location in this case, but that also does not jump the user to the definition location.
Is it possible to achieve this? Or is there simply no way currently to delay the execution of connection.onDefinition or invoke it a subsequent time using a callback function?

if anyone ever stumbles upon this, the solution was to return a new Promise() from connection.onDefinition

Yes all request callback can return a Promise which will be awaited before the response is sent back.


type of inequality constraints in scipy.optimize.minimize docstring

In docstring for scipy.optimize.minimize it is said

In general, the optimization problems are of the form:

minimize f(x) subject to

g_i(x) >= 0,  i = 1,...,m
h_j(x)  = 0,  j = 1,...,p

Which in general is not true. The classical form is g_i(x) <= 0,  i = 1,...,m. As I see the issue comes from the fact that the only solver for constrained optimization problem is SLSQP, which uses this form.

It's just a convention. No generality is lost by making this choice of convention because constraints following one form can be trivially converted to the other form. The fact that they can be expressed in the other form (and even often are) doesn't reduce the generality.

@rkern, I totally agree with you. It is the question of how you define the Lagrange function, but if you say general you should follow some convention to be in agreement with other world:). Also  linprog docstring is in accordance with general linear programming problem formulation, and no matter that for simplex method you should transform it to canonical form.

The way that we use the term in the docstring is in accord with my experience with the usage of the term in (English, at least) mathematics. It's claiming that the general class of problems that can be solved with this solver can be expressed in this form (and that this is the form that the solver is expecting). It is not claiming that most people use this convention in other contexts.

There doesn't seem to be a bug here, so I'm going to close this issue.


Modify on the first sentence

-The BitcoinKit library is a Swift implementation of the Bitcoin protocol, supporting both of BCH and BTC. Our mission is improving the mobile ecosystem for Bitcoin developers.
It allows maintaining a wallet and sending/receiving transactions without needing a full blockchain node. It comes with a simple wallet app showing how to use it.
->
The BitcoinKit library is a Swift implementation of the Bitcoin protocol which support both BCH and BTC. Improving the mobile ecosystem for Bitcoin developers is our mission.
BitcoinKit allows maintaining a wallet, sending or receiving transactions without a full blockchain node. Following is a wallet app that demonstrates the way to use it.
updated #207

@GitCash 1000 bits for @kelvin2468


container: remove ulimit nofile parameter

Since Ceph Octopus is python3 only we don't need to specify the max open
files anymore with the container engine.
Signed-off-by: Dimitri Savineau<EMAIL_ADDRESS>

jenkins test centos-non_container-update

jenkins test centos-container-update

jenkins test centos-non_container-purge


add support for send and receive use different protocol

https://github.com/v2ray/v2ray-core/issues/1388

This is a big change. Please write a document about your plan, and review it with dev team. Such PR will not be simply accepted.

Ok, I'll try.

My Plan
In client, using TCP as send protocol and mKCP or QUIC as receive protocol.
We consider the send protocol and the receive protocol is reliable so we not consider internal of protocol.
Client needs to do

Establish a connection using send protocol.
Sending heartbeat packets using receive protocol per 60s for keeping NAT port mapping.
Generate an UUID and send the UUID using the send protocol and the receive protocol respectively to make server link two connections, server need to record the client's receive ip and port.

Then, when client sends request data to server using send protocol, the server will send data to the client's ip and port using the client's receive protocol after gets response data.

What is the benefit to split uplink and downlink? In your proposal, the connections are 1:1 mapped between uplink and downlink. Both connections has some-what obvious fingerprint (UUID and heartbeat).

Goal is solve the uplink UDP QoS.
Uplink data can be encrypted by some encryption protocols so UUID not is an obvious fingerprint I think.


workflow-apply error handling

closes 591
closes #616

Note: the local (sqlite-based) CI still fails, as due to #661.


PyTorch compatible dataloaders

Enable online learning with PyTorch compatible dataloaders.
These should use the same underlying observer pattern of the CLI, with options to write out asynchronously to disk, and more.
Also should provide an interface to stack pre-processing steps in a pipeline before exposing the data to NNs.

API is enough, no longer think this is worthwhile.


Improved code style

New PR 👍

Taken from https://github.com/fastlane/fastlane/pull/4955 basically

👍

Great to see this merged, thanks @mfurtak


rustup for find_fn rename

@bors r+

:pushpin: Commit edac086f1c9b324f97a9e0aab4c2f46b317d64e1 has been approved by RalfJung

:hourglass: Testing commit edac086f1c9b324f97a9e0aab4c2f46b317d64e1 with merge a0ba079b6af0f8c07c33dd8af72a51c997e58967...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing a0ba079b6af0f8c07c33dd8af72a51c997e58967 to master...


break juicy-html on purpose to see if tests fail

Woah this is awesome 😍

Closing as this PR is no longer needed and was not intended to be merged.


Updated to Swift2

Will review today. I'm thinking about keeping this in a swift 2 branch until it's officially released

I agree - it is still in beta phase at the moment so we should keep the branches separate until we get an official Xcode 7 release.

Awesome. Can you open a new PR to the swift-2.0 branch? (https://github.com/ViacomInc/Router/tree/swift-2.0)


Some minor "issues"

While creating erlef/setup-erlang (heavily inspired by this one), I noticed two very small details that you might want to improve:

[ ] test.yml contains npm test (that's "failing", even though not signalling CI properly)
[ ] action.yml contains author: GitHub (which I changed to Erlang Ecosystem Foundation in my file)
[ ] some licenses might be in need of review for .licensed.yml (or maybe not, since your CI runs fine: I wonder what the main differences are, though?)

I'm also not sure how .github ends up inside dist (I don't think it's actually required there), but it does when we do npm run build.

some licenses might be in need of review for .licensed.yml (or maybe not, since your CI runs fine: I wonder what the main differences are, though?)

Could you expand on that?

Try running licensed locally, on the project. If there's no issues presented, then there's no issue with the licenses. I assumed that, since I did that and some issues were presented (for setup-erlang) there might be some here too, which is why the actions state "might be... or maybe not".

setting process.env.PATH might have unintended side-effects (it did for me when I did that twice, but I'm probably missing something here; stuff seems to work without changing it too)

Can you elaborate on this? What are the unintended side-effects and what happened for you?

wget got reported as not being available, for example. Removing it seemed to do the trick. But also, I was doing it before running the bash script, so it's probably correlated.
In your case, for example, you already have echo "$(pwd)/.setup-elixir/elixir/bin" >> $GITHUB_PATH in the bash script. I'm not sure that shouldn't suffice.

wget should in theory still be available since we only prepend to the path.
IIRC process.env.PATH is used to modify the PATH for the current action and $GITHUB_PATH is used to modify it for subsequent actions, we need to do both.

wget should in theory still be available since we only prepend to the path.

Agree. Just not what I saw, unfortunately. And searching around the web I can't see examples for changing the process.env or what are the recommended practices. (I'll resolve this one, since in your case it seems to be a non-issue).

IIRC process.env.PATH is used to modify the PATH for the current action and $GITHUB_PATH is used to modify it for subsequent actions, we need to do both.

My expectation exactly, but not what I observed. If you look at setup-erlang, it's working and you don't have those elements anymore (at the same time I'm not calling commands I just defined, like you do for mix, for example).

@ericmj, I think I found the culprit in my JS (while merging the actions). I was missing :${process.env.PATH}, much the same as if you miss that in bash you render your PATH "useless".

With the merge I'll hopefully have solved all issues except "action.yml contains author: GitHub (which I changed to Erlang Ecosystem Foundation in my file)". I took the liberty to adapt it to EEF. If y'all agree with this choice, this issue is good to be closed.

Closing due to "all actions resolved".


Add GitHub Action to create issues for each TODO comment

Current Behaviour
When a  TODO/BUG/FIXME/HACK comment is committed into the develop branch, nothing happens.
Proposed Changes
When a  TODO/BUG/FIXME/HACK comment is committed into the develop branch, a new GitHub Issue is created to keep track of it.
Fixes  #1930.
Note that merging this PR would create >100 of new Issues, as seen in my fork of the report where I tested this Action.
Additionally, the labels of the new issues might be configured to something more appropriate.
Checks

[x] All commits in this Pull Request are signed.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor Licence Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the ockam-network/contributors repository.

Thanks for your code review! Should be ready for a second round now.

@frazar Thank you for taking time to contribute.
Sorry that we haven't merged this so far.
I fear that turning this on will crate too many new issues and hesitated on the merge ... maybe a better setup would be a single issue that tracks a checklist of TODOs. What do you think?
Thank you and happy new year.

Sorry that we haven't merged this so far.

No worries! I understand this PR has a significant impact on the project, so you should carefully weigh whether or not to merge.
I am not a project maintainer, so I don't really feel entitled to an opinion. But since you asked (thank you!), I am going to express my point of view.
Personally, I do not think that having many open Issues is a problem per se, but I agree that having many non-triaged Issues is not ideal. Indeed, going through all these Issues and applying the correct labels would take precious maintainer time.
However, I believe that many non-triaged Issues are better than no Issues at all. The triaging process is not required to happen right away, and the amount of non-triaged Issues is expected to decrease over time. It is also possible to introduce a "needs-triage" label so that the auto-generated Issues can be filtered out quickly.

maybe a better setup would be a single issue that tracks a checklist of TODOs. What do you think?

I see two drawbacks to this approach:

The GitHub Action used in this PR does not support such functionality, so it would require implementing it.
Having a single Issue page would actually make Issue triaging harder since maintainers would need to manually create the new Issues.

Hey @frazar for the time being, we decided to not create so many issue based on TODOs. Thank you so much for working on this, I regret that we couldn't land it. I'm closing this PR for now. Thank you 🙏


node or cluster stop: make timeout configurable

Currently, node.stop(), and cluster.stop() which uses it, are hard-coded
to allow Scylla 127 seconds (=2^7-1), just a bit over two minutes, to
shutdown before failing. In some tests, see for example
https://github.com/scylladb/scylla/issues/4019, we know that we need more
than that time for the shutdown to complete.
So this patch makes the waiting time configurable, via an optional parameter
"wait_seconds" which defaults to the previously hard-coded 127 seconds and
can now be set to any number of seconds.
Signed-off-by: Nadav Har'El<EMAIL_ADDRESS>

BTW, this needs backport to branch 3.0 (if we have such a branch for ccm)

ping

Oops, sorry, I missed that it was already committed (didn't refresh automatically...)


net/procfs: add entry mapping table

Summary
net/procfs: add entry mapping table
add entry mapping table to simplify new component registration
Signed-off-by: chao an<EMAIL_ADDRESS>Impact
N/A
Testing
nsh> ifconfig
Joins: 0000 Leaves: 0000
Sent       Sched Sent
  Queries: 0000  0000
  Reports:
    Ver 1: ----  0000
    Ver 2: 0000  0000
  Done:    0000  0000
Received:
  Queries:
    Gen:   0000
    MAS:   0000
    MASS:  0000
    Ucast: 0000
    Bad:   0000
  Reports:
    Ver 1: 0000
    Ver 2: 0000
  Done:    0000
eth0	Link encap:Ethernet HWaddr 00:00:00:00:00:00 at DOWN
	inet addr:<IP_ADDRESS> DRaddr:<IP_ADDRESS> Mask:<IP_ADDRESS>
	inet6 addr: ::/0
	inet6 DRaddr: ::/0

             IPv4  IPv6   TCP   UDP  ICMP  ICMPv6
Received     0000  0000  0000  0000  0000  0000
Dropped      0000  0000  0000  0000  0000  0000
  IPv4        VHL: 0000   Frg: 0000
  IPv6        VHL: 0000
  Checksum   0000  ----  0000  0000  ----  ----
  TCP         ACK: 0000   SYN: 0000
              RST: 0000  0000
  Type       0000  0000  ----  ----  0000  0000
Sent         0000  0000  0000  0000  0000  0000
  Rexmit     ----  ----  0000  ----  ----  ----

@anchao
I noticed that the ifconfig now shows nothing with spresense:wifi
Do you know how we should modify gs2200m driver and daemon?

@anchao I noticed that the ifconfig now shows nothing with spresense:wifi Do you know how we should modify gs2200m driver and daemon?

@masayuki2009  I am not sure the side effect of this PRon spresense, let me confirm on device further, I will reply later.

@masayuki2009 I am not sure the side effect of this PR on spresense, let me confirm on device further, I will reply later.

@anchao
spresense:rndis works OK but spresense:wifi does not work with this PR.
I think the issue might relate to usrsock-based driver such as gs2200m.

@anchao
I noticed that /proc/net/wlan0 does not exist now.

@anchao I noticed that /proc/net/wlan0 does not exist now.

Yes, I missed another case that disable CONFIG_NETDEV_IFINDEX, the index of device should begin from 1, I have fixed this issue, please review this PR:
https://github.com/apache/incubator-nuttx/pull/7039