refactor: change access types to BindingAccess

Currently, there are 5 different variants of Expression into which references to bound identifiers within variables or constraints are parsed - Elem, VectorAccess, MatrixAccess, Rand, and TraceBindingAccess. These are all handled in essentially the same way within the IR, so consolidating these into a single type allows us to simplify handling in the IR, especially of variables.
This PR does the following. Because of the nature of the change, it's hard to split it into a smaller change without errors.

introduce BindingAccess
replace Elem, VectorAccess, and MatrixAccess with BindingAccess
update codegen handling for new Value types

Future items for discussion:

Expression::Rand should probably be similarly replaced. This would be fairly straightforward.
Ideally, we would consolidate TraceBindingAccess into BindingAccess as well. This would improve the code clarity significantly, since all trace bindings for the "current" row are already parsed as BindingAccess, so TraceBindingAccess only represents references to trace bindings in the "next" row or being accessed as a Range. This means that the handling within the IR of accessing trace bindings is split into 2 places. The complication with this is that we access ranges of the trace, so AccessType would need to change, and I'm not sure of the best way to adjust this.

Thank you both for the review! I have addressed the smaller points. I agree with the larger points, and appreciate the suggestions for addressing the consolidation of Rand and TraceBindingAccess into BindingAccess. As these adjustments will result in significant changes, I'm going to leave them for future refactoring PRs. I'm planning the following series of PRs:

simple renaming (e.g. BindingAccess to SymbolAccess and SymbolType to SymbolBinding)
replace Rand
replace TraceBindingAccess
revisit the variable handling for improving clarity again (will probably be required as part of replacing TraceBindingAccess)


100 ensure no rating is done before a meal's time

Why?

It was possible for people ratings to be done before meal time

How?

Ensure it's impossible to rate until it's time for a meal or after

Coverage increased (+0.07%) to 97.484% when pulling 334d1388373f8f5f518ce247a4237a37ef3f310f on 100__ensure_no_rating_before_meal_time into 337a9b6a4332a12d1f556b87433b97c1c2f5a356 on master.

@andela-ydaniju this PR should be against staging and not master.


Update mvc/security/cors-policy.rst from beta4 to beta8

Updating the article to use beta8 instead of beta4.
Please review. @MikeWasson

Hi @robbert229, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Looks good! 17dcf7025862f5c011e6e52b08c90144364d4699


Added dropdown list for activity groups

Hi,
I have added activity groups to your integration with drop down lists, instead of having activities in switches buttons.
I had to copy/paste your wrapper library files from pyUnfoldedCircleRemote into the component to make it work (and I removed the external lib from the manifest)
The library works as expected at this state. I let you tell me if it is okay for you

Also I have moved the loading part to the coordinator instead of the switch entity because loaded data is useful for all entities and should be done before (this is what the coordinator is aimed for).
I have renamed the device to match user's one.

Hi Albaintor,
Thanks for the contributions and interest in the project. I've finally been able to sit down and take a closer look at things. Looks like you and I were on the same page in adding the reconfigure support. I have that working in my local branch. The activity groups are a new thought though that I haven't considered. I like the idea but a couple of things to consider:

The devs chose not to group activities by group in the UI. This could be temporary, but they did call it out specifically and didn't say it was coming in the future.
I do like the 'Power Off' option! It is confusing when no activities are running though. It does sort of imply they are
Would a better option be to provide a service that powers off all activity activities and optionally takes a group as input to only power those down?
Do the select options provide a way to show which activities in the group are running? The selected one makes sense. But what if you have more than one active?
Users that currently have activities defined as switches will have those entities no longer provided by the integration. I'm pretty sure they would also have to update any automations they had defined as well.

Unlike the harmony integration which recently switched to using a select entity for its activities, I kept using switches due to the fact that multiple activities on the Remote Two can be active at once. And unless I'm mistaken, a select entity has no way to convey multiple 'on' states at the same time.

A different topic, but I see you're investigating the web socket integration. I did briefly too but never got very far.
The remote communicates to HA through a web socket so it definitely feels like the right answer. However, in my testing the WS connection would only stay active for a very short period of time before disconnecting. We could ping it to keep it active, but is that any better than polling for changes periodically? It seems like it would be at least a little as we'd only need a ping rather than a torrent of get requests.
Then there was pushing changes. I admittedly didn't look very hard but there wasn't an obvious way to do it and so I kept using the REST api. Anyway, I'm curious what you find. Could be some improvement there. Local Push!

Hi,
actually if you include an activity in an activity group, you can’t have 2 activities running in the same group this is the main advantage of this as it will switch off the entities of the old activity before going to the new one. With customization (keep entities active if they are in both…). So we are in the same situation as harmony activities and so is the select appropriate for this. However if an activity is in no groups , it can be enabled apart and the switch is appropriate. This is the principle of the pull request I posted.
on the other hand I am working on websocket : it is nearly working, a little more time is needed. But I can see improvements in the number of requests. The thing to deal with is sleeping state of the remote that can be VERY long.

And indeed after the update users will have to deleted unavailable entities or recreate the integration

I plan to commit an update during the weekend

I have committed the changes : now the battery info + luminance + activities + software updates are updated through the websocket.
The luminance is disabled by default as it consumes a lot of requests : if the user enables it, I enable the event to be received in the websockets.
The problem now is the other sensors that won't be updated anymore (CPU, memory...): there are not available in the websocket which is normal (they change all the time). One must think to a solution to get them if user wants them (with polling necessarily)

Hi again, I have found the solution : mixed polling if any entities require it, and push with websockets for the rest.
This is all committed and working. I let you review it

You're right :)

It works well if you assume only a single media player entity is operating at a time. When you have more than one, they either overwrite the same media player entity in HA, or you create a new entity for each remote media player. And both of those options aren't good. Exposing the data through an array of extra state attributes may be worthwhile though.
Also, since all of my entities configured in the remote are also entities in home assistant, all of this data is duplicative. I can just pull this media info directly into Home Assistant from the nvidia shield or apple tv that the remote started playback on.
It is pretty cool though!

On the other hand, my remote's battery plummeted 25% in a couple of hours after creating the WS. It could be unrelated, but I'm now curious if this the intermittent polling wasn't gentler on the battery...

About battery drain : have you enabled light luminance sensor ? if you disable it you have to restart integration (I didn't write code to dynamically restart the websocket when configuration changes) --> I wonder if this should not be more clever to get it through polling then (because with push you get about one request per second as luminance changes very often)
Otherwise to get (and verify) the benefits of the websocket over polling, you should also disable the other entities (cpu, memory...) except the dropdowns, the activity switches and the battery sensors (and restart the integration).
In that case there is no polling anymore.
I will do the same test to check after battery drain
About current media playing, I had the same thought. I see 2 possibilitiés :

Take the first media playing active and keep that one as long as the media is active
Create one media player entity per activity group : even in that case you may have severals and will have to pick one

This should not drain battery as the media playing is sent once when media starts (unless you play music it will happen more frequently)
Last thing to tryout : reduce the ping/pong delay of the websocket. I asked dev team on discord to get the timeout delay of the websocket. I could try to measure it on my own though

Hi,
I have fixed several things :

Websocket reconnection after remote reboot or sleeping
Increased the ping/pong delay of websocket
Reduced the unnecessary requests
Fixed timeout during the initialization of the integration : many requests sent to the remote to load API data but at the end HA will cancel loading the last entities, especially for the remote entity
Added a polling sensor to be aware if the integration does polling or not. Polling will be disabled if entities that use polling are all disabled
Added attributes to the remote entity (I think you wondered how to display more data)
Improved discovery settings based on mac address : this is the only key which is provided by zeroconf and that can be check against in user flow configuration. The benefit will be not to have already added devices to be discovered again
Renamed the remote device to the one the user named : useful when you have more than one RC2 (which is my case)

Battery consumption seems right : I set up my remote to sleep after 30 minutes (max setting) and battery drains a lot with this setting without this integration. So it seems to be okay on that part :

Hi again,
I have added media player entities to the integration (one per activity group)


format SQL

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Good job. If you want, you can add me on WeChat LL332595980 and invite you to join the shenyu contributor group.


[Question]: AssertionError: The model /workspace/model/llm/Qwen/Qwen2-7B-Instruct you specified is not supported.

Describe the bug
AssertionError: The model /workspace/model/llm/Qwen/Qwen2-7B-Instruct you specified is not supported. You are welcome to add it and open a PR
Steps to reproduce
No response
Expected Behavior
No response
Logs
No response
Additional Information
No response

Hi @LIUKAI0815,
Thanks for your feedback.
If you're using a local model, you need to provide the model name from Hugging Face. You can follow the guidelines from this link.
local_path = "xxx"
model = AutoModelForCausalLM.from_pretrained(local_path)

# Need to use the actual model name in the HF repo
model_name = "Qwen/Qwen2-7B-Instruct"
minference_patch = MInference("minference", model_name)
model = minference_patch(model)

thank you


Pypi Package

Would really appreciate being able to install this from pip. I think a resource like this should be available on pypi anyway.
For developers finding this a problem, you can install from the repository:
pip install git+git://github.com/Pushwoosh/pushwoosh-python-lib.git

It's done. Enjoy! Here is the link: https://pypi.python.org/pypi/pypushwoosh


Fix typo in vec example proof

I believe it should be i<n not i<m so it's i<n=> i < m+1 => (i - 1) < m

Thank you!


Introduce goreleaser

Moving to the original repo: https://github.com/omissis/go-jsonschema/issues/60


Fix integration tests. Latest ubuntu (xenial) images do not have iproute2 installed by default.

It seems that current latest ubuntu image doesn't have iproute2 package installed. So integration tests are failing.
vagrant@vagrant-ubuntu-trusty-64:~$ docker run -t -i ubuntu:latest
root@688812c500c8:/# ip token
bash: ip: command not found
root@688812c500c8:/# exit
vagrant@vagrant-ubuntu-trusty-64:~$ docker run -t -i ubuntu:trusty
root@847c8801a90c:/# ip token
token :: dev eth0
root@847c8801a90c:/# exit

One way is just to stick with trusty image (as in proposed patch).
Also integration test suite doesn't actually catches  printed errors, but i didn't dig into that.

This looks good. Can you also please update the ubuntu reference to ubuntu:trusty in docs/guide.rst? Thanks for the fix!

👍

Thanks!


is there a possibility that nylo supports web platforms?

Hi @teoprayoga, Nylo can be used to build Flutter websites if that's what you are asking.
Hope that helps 👍

how to build nylo web release?
I run fvm flutter build web --release. it says missing index.html

Hi @teoprayoga,
Yeah, you'll need to include an index.html file.


Pluggable backend features: define API

As a first step in the direction of switching to compas.plugin-based backend features, we should first define the API that we want to have from a user perspective.

Notes of the meeting today:

Each backend feature is a pluggable
Change planner_backend argument in RosClient to accept a type directly, instead of just a string mapped to a type
Add to documentation: how to write your own custom planner/solver
Consider moving whole planning scene state to our side (as a list of updates) and then flush it to the backend right before any calculation (if it changed)
Use client.planner as main mechanism to switch backend feature sets
Use client.planner.inverse_kinematics = my_inverse_kinematics as main mechanism to switch individual backend features
PlannerInterface becomes a pluggable, and then, e.g. MoveItPlanner is automatically discovered
We could try to integrate Tesseract, or instead of that, try to do the same with multiple ExecutorInterface implementations (MoveItExecutor {follow/execute}, CompasRRCExecutor, etc)

I believe this is closed, and we will create follow-ups to start implementing these changes.


Add item description

As a User
I need to add a description to an item on a wishlist
So that I can know my thoughts about the item for later
Assumptions:

There is no description currently with the item.
API Call: PUT /wishlists/{id}/{item-id}/description

Acceptance Criteria:
When I type a description and click "add a description",
I should see the description below the item.

The same as update an item. Should we still need it?

I think it's very similar but still a bit different. I noticed that the description of an item can be null, so adding an item description would be when there is no description initially for the item, and updating an item would be for updating the item description.


Fix bold formatting for table header.

Fixed a typo in the Markdown for making the table header bold.

@steji113,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

entity-framework/efcore-and-ef6/features.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Already fixed: https://github.com/aspnet/EntityFramework.Docs/pull/456

@ErikEJ Oops, didn't see, thanks!


Apple Login Error: "lat invalid"

There is an issue with the login using the Apple Sign-In from the Trakt iOS login screen.
The login always fails with the following error message:
Could not authenticate you from Apple because "lat invalid
This error happens using third party apps and also Trakt app.
Our app was rejected by Apple due to this issue. Please, could you check it?
Please advice.

It is also failing the login with Google with the following error:
There was an error during the transport or processing of this request. Error code = 103, Path = /v3/signin/_/AccountsSignInUi/data/batchexecute
Hope this helps.

I pushed a potential fix for Apple sign in. I'm not sure what that Google error means, that isn't an error or path on the Trakt side.

Thank you for your work, but it seems that something is still not working as expected with the Apple Sign In. When I logging using the Apple Sign In flow, the app opens the profile screen of the Trakt website. It doesn't redirect to the app itself.
It happens with every 3rd party app using the Trakt API and also with the Trakt official app.
Please advice,

Good catch, sorry about that! I just pushed another fix and verified with the Trakt app that Apple sign in redirects correctly.

@javikr did your app get approved with the Apple sign in fix?

@rectifyer yes, but now they are rejecting us due to the cookie prompt that appears on the login flow site. As this is an issue that is eligible to be resolved on the next update, they have proceeded with it, but I'm pretty sure that they are going to reject it again :(
Here is the App Review Team message:

Guideline 5.1.2 - Legal - Privacy - Data Use and Sharing
We noticed your app accesses web content you own where you collect cookies. Cookies may be used to track users, but you do not use App Tracking Transparency to request the user's permission before collecting data used to track.
Starting with iOS 14.5, apps on the App Store need to receive the user’s permission through the AppTrackingTransparency framework before collecting data used to track them.
Next Steps
If you do not collect cookies for tracking purposes on iOS, remove the cookie prompts or revise them to clarify you do not track users.
Otherwise, follow these steps to resolve this issue:

If you haven't already, update your app privacy information in App Store Connect to disclose that you track users. You must have the Account Holder or Admin role to update app privacy information.
Implement App Tracking Transparency.
Request permission using App Tracking Transparency before collecting data used to track the user. When you resubmit, indicate in the Review Notes where the permission request is located.
If the user does not allow tracking, do not collect cookies for tracking purposes.

You may also choose to remove the tracking functionality from your app, including tracking that occurs when accessing web content.

I have reply their message explaining that we have no control over third party cookies, but with no success... They are still rejecting it.
Hope this helps.

Hmm, that's an interesting one. It's true we use cookies, but not to track anyone or sell their data (like Apple might assume). As with any standard web sign in form, we use cookies to establish a logged in session. I'll look into this more, but I'm not sure we can simply not use cookies and have the OAuth flow function as it needs to.

Thank you. I will keep you informed if they rejected us again ;)

Just to inform that Apple approved the version without any change in our code... I'm pretty sure it depends on the person who is reviewing the app. Thank you for your help ;)


Issue #3 VnV Plan: Section 4.1.1

Section 4.1.1	Input test models; models (general). The units for certain physical constants aren't explicitly in this document. I am sure you state them somewhere but it would still be nice whenever you use units to put them there. For instance, for temperature having Kelvin (i assume) would be beneficial for readability.

Yes when I look back to it I found it confusing as well because I copy paste the structure from table 1 and forgot to change the unit. It is fixed in 60a803b


[HW] Vishal Ojha

Name: Vishal Ojha
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/vishal-ojha-62b148216/
Attach the homework step I screenshots below:

Hey Vishal Ojha, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/LX6g6UxBS5ms99ohDOAUMw


UnboundLocalError: local variable 'new_value' referenced before assignment

piexif-1.1.2
from PIL import Image
import piexif

im = Image.open("/tmp/test.jpg")

gps_dict = {
    piexif.GPSIFD.GPSLongitude: 1.1,
    piexif.GPSIFD.GPSLatitude: 2.1,
    piexif.GPSIFD.GPSAltitude: 3.1,
}
exif_dict = {'GPS': gps_dict}
exif_bytes = piexif.dump(exif_dict)
im.save("/tmp/test.exif.jpg", "jpeg", exif=exif_bytes)

errors as
---------------------------------------------------------------------------
UnboundLocalError                         Traceback (most recent call last)
<ipython-input-116-b699857b7bb8> in <module>
      8 }
      9 exif_dict = {'GPS': gps_dict}
---> 10 exif_bytes = piexif.dump(exif_dict)
     11 im.save("/tmp/s/45_08034_lvl02-oblique-left.5.exif.jpg", "jpeg", exif=exif_bytes)

~/.local/lib/python3.7/site-packages/piexif/_dump.py in dump(exif_dict_original)
     72         exif_length = 0
     73     if gps_is:
---> 74         gps_set = _dict_to_bytes(gps_ifd, "GPS", zeroth_length + exif_length)
     75         gps_bytes = b"".join(gps_set)
     76         gps_length = len(gps_bytes)

~/.local/lib/python3.7/site-packages/piexif/_dump.py in _dict_to_bytes(ifd_dict, ifd, ifd_offset)
    335             length_str, value_str, four_bytes_over = _value_to_bytes(raw_value,
    336                                                                      value_type,
--> 337                                                                      offset)
    338         except ValueError:
    339             raise ValueError(

~/.local/lib/python3.7/site-packages/piexif/_dump.py in _value_to_bytes(raw_value, value_type, offset)
    245                                 struct.pack(">L", den))
    246         value_str = struct.pack(">I", offset)
--> 247         four_bytes_over = new_value
    248     elif value_type == TYPES.SRational:
    249         if isinstance(raw_value[0], numbers.Integral):

UnboundLocalError: local variable 'new_value' referenced before assignment

I get my code is broken, but the error it triggered is one (probably unrelated) in piexif code :)

You can't use floats. The numbers for these fields are treated as Rationals, and need to either be integral or a (numerator, denominator) tuple.
Error is cryptic, I agree, because the if-elif chain doesn't have a else that raises a ValueError.

Ah, that makes sense. Thanks!

I have the same error with some gps coordinates like these:
{1: 'N', 2: ((44, 1), (30, 1), (2997, 100)), 3: 'E', 4: ((11, 1), (21, 1), (5991, 100)), 5: 0, 6: (324663, 12176), 12: 'K', 13: (3083, 4231), 16: 'T', 17: (406646, 1481), 23: 'T', 24: (406646, 1481), 31: '24/1'}
The problem is with '24/1' value.
These coordinates are automatically created by a digital camera, so i can't handle them.
I think that the better thing should be fixing the code to handle an else statement that doesn't break anything..the problem is that i don't know what should be a safe default value for new_value.
An empty string should be ok?

Use latlons in following formats
((degress, 1), (minutes, 1), (seconds, 10000))


Vercel Deployment Error : Failed to compile

Description
Hi ! I tried to deploy a second instance of LinkTree on Vercel. I deployed one last week without any issues and I did the same way for the second one, because when I tried to update the profiles data, it suddently stop updating and each deployment shows errors.
This is the log I got from the deployment screen on vercel
[15:15:09.585] Cloning github.com/MYACCOUNTPATH (I hide it for security purposes)
[15:15:10.245] Cloning completed: 659.522ms
[15:15:10.731] No Build Cache available
[15:15:10.768] Running "vercel build"
[15:15:11.262] Vercel CLI 28.4.6
[15:15:11.555] Installing dependencies...
[15:15:24.622] npm WARN deprecated<EMAIL_ADDRESS>Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility
[15:15:24.939] npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
[15:15:25.821] npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
[15:15:31.407] npm WARN deprecated<EMAIL_ADDRESS>This SVGO version is no longer supported. Upgrade to v2.x.x.
[15:15:32.779] npm WARN deprecated<EMAIL_ADDRESS>This package is now published under @cucumber/gherkin
[15:15:34.630] npm WARN deprecated<EMAIL_ADDRESS>This package is now published under @cucumber/cucumber-expressions
[15:15:40.299] npm WARN deprecated<EMAIL_ADDRESS>This package is now published under @cucumber/cucumber-expressions
[15:15:40.617] npm WARN deprecated<EMAIL_ADDRESS>Cucumber is publishing new releases under @cucumber/cucumber
[15:15:44.182] npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js.
[15:16:03.262]
[15:16:03.262] ><EMAIL_ADDRESS>prepare
[15:16:03.262] > husky install
[15:16:03.262]
[15:16:03.328] husky - Git hooks installed
[15:16:03.357]
[15:16:03.357] added 1826 packages in 51s
[15:16:03.357]
[15:16:03.357] 235 packages are looking for funding
[15:16:03.357]   run npm fund for details
[15:16:04.731] Creating an optimized production build...
[15:16:29.559] Failed to compile.
[15:16:29.559]
[15:16:29.559] [eslint] Failed to load config "react-app" to extend from.
[15:16:29.559] Referenced from: /vercel/path0/package.json
[15:16:29.560]
[15:16:29.560]
[15:16:29.622] Error: Command "react-scripts build" exited with 1
Screenshots
No response
Additional information
No response

Could this be due to the github action/vercel outage that happened earlier today?

No I got this issue since last week with my first deployment. (I was able to make changes and update to files and suddently after I modified a name in a profile file, every deployments failed with the same error log.)
I fork a new repository in my github account from this repository (without any change) (EddyJouade Community) start a second deployment and still have the exact same error log.

Did you follow the instructions for single user mode? https://github.com/EddieHubCommunity/LinkFree#single-user-mode

Did you follow the instructions for single user mode? https://github.com/EddieHubCommunity/LinkFree#single-user-mode

No since it's not a single user usage

Not sure I understand what you are trying to do here @solucompme ?

[15:15:09.585] Cloning github.com/MYACCOUNTPATH (I hide it for security purposes)

Also why is this more secure? Does this mean you have more than 1 GitHub account?

Closing as no activity for 4 weeks


Remove the Chisel global variable (from #1618)

In modern JavaScript, it is usually better to use imports rather than a global variable. For example, to type-check code that uses the global variable, we must feed tsc with an extra file that specifies the type of the variable. Getting rid of the Chisel variable will simplify our code in the fugure.
This is a spin-off from #1618.

@honzasp Looks good to me! Will let others also review this too because I wonder why we didn't get rid of it before.


pt_BR Buy Now button is blank

for pt_BR locale, there is no button text when button size is small and branding is turned off.
Please provide a short description of the issue here, along with:

Fiddling around, I found the same with es_ES

Actually I think @vishakha94 may have fixed this already but the fix didn't make it out yet. Hopefully with tomorrow's Hermes release.

This has been fixed but is not live it. Will close it once after today's push.

@vishakha94 I don't think this got fixed -- I still see width: 60% but I think we need width: auto otherwise the text will cut off if it's greater than 60%.
DW about it though, I'm doing some refactoring to add the credit dual button so I'll slot this in.

Fixed and pushed.


Fixes #169: Compare version using integers

pytorch version 1.10 has been released and this causes the version check
to erroneously fail. This fixes that comparison.
>>> [1, 10, 0] < [1, 7, 1]
False
>>> ["1", "10", "0"] < ["1", "7", "1"]
True
```

Thanks for the PR - I've merged #160 and will close this as duplicate.


Can only connect through Lan

I can't manage to connect to my server even though it is listed in Public Servers browser and shows a value in Ping.
It shows in Lan browser and can connect but when trying through Public Servers browser or Connect to IP I get the
'Couldn't establish network communication with server.'
Set-up:
Ubuntu server VM with Bridged Network Adapter (<IP_ADDRESS>).
Ports 34198/UDP and 27015/TCP forward in router to <IP_ADDRESS>
$ sudo nmap -sUV <IP_ADDRESS> -p 34198
Starting Nmap 7.60 ( https://nmap.org ) at 2020-08-31 14:04 CEST
Nmap scan report for winserver (<IP_ADDRESS>)
Host is up.

PORT      STATE SERVICE VERSION
34198/udp open  unknown
1 service unrecognized despite returning data. If you know the service/version, please submit the following fingerprint at https://nmap.org/cgi-bin/submit.cgi?new-service :
SF-Port34198-UDP:V=7.60%I=7%D=8/31%Time=5F4CE761%P=x86_64-pc-linux-gnu%r(N
SF:TPRequest,9,"\xf2\x06\x80\0\x01\0\x04\xfa\0");

Service detection performed. Please report any incorrect results at https://nmap.org/submit/ .
Nmap done: 1 IP address (1 host up) scanned in 102.50 seconds

Docker-compose file
  factorio:
    image: factoriotools/factorio:stable
    container_name: factorio
    hostname: factorio
    restart: unless-stopped
    network_mode: host
    ports:
      - "34198:34198/udp"
      - "27015:27015/tcp"
    volumes:
      - ${DOCKERDIR}/factorio:/factorio
    environment:
      TZ: ${TZ}
      PORT: 34198

server-settings.json
{
  "name": "redacted",
  "description": "Partida a tope",
  "tags": ["game", "tag"],

  "_comment_max_players": "Maximum number of players allowed, admins can join even a full server. 0 means unlimited.",
  "max_players": 0,

  "_comment_visibility": ["public: Game will be published on the official Factorio matching server",
                          "lan: Game will be broadcast on LAN"],
  "visibility":
  {
    "public": true,
    "lan": true
  },

  "_comment_credentials": "Your factorio.com login credentials. Required for games with visibility public",
  "username": "redacted",
  "password": "",

  "_comment_token": "Authentication token. May be used instead of 'password' above.",
  "token": "redacted",

  "game_password": "redacted",

  "_comment_require_user_verification": "When set to true, the server will only allow clients that have a valid Factorio.com account",
  "require_user_verification": true,

  "_comment_max_upload_in_kilobytes_per_second" : "optional, default value is 0. 0 means unlimited.",
  "max_upload_in_kilobytes_per_second": 0,

  "_comment_max_upload_slots" : "optional, default value is 5. 0 means unlimited.",
  "max_upload_slots": 5,

  "_comment_minimum_latency_in_ticks": "optional one tick is 16ms in default speed, default value is 0. 0 means no minimum.",
  "minimum_latency_in_ticks": 0,

  "_comment_ignore_player_limit_for_returning_players": "Players that played on this map already can join even when the max player limit was reached.",
  "ignore_player_limit_for_returning_players": true,

  "_comment_allow_commands": "possible values are, true, false and admins-only",
  "allow_commands": "admins-only",

  "_comment_autosave_interval": "Autosave interval in minutes",
  "autosave_interval": 10,

  "_comment_autosave_slots": "server autosave slots, it is cycled through when the server autosaves.",
  "autosave_slots": 5,

  "_comment_afk_autokick_interval": "How many minutes until someone is kicked when doing nothing, 0 for never.",
  "afk_autokick_interval": 0,

  "_comment_auto_pause": "Whether should the server be paused when no players are present.",
  "auto_pause": true,

  "only_admins_can_pause_the_game": true,

  "_comment_autosave_only_on_server": "Whether autosaves should be saved only on server or also on all connected clients. Default is true.",
  "autosave_only_on_server": false,

  "_comment_non_blocking_saving": "Highly experimental feature, enable only at your own risk of losing your saves. On UNIX systems, server will fork itself to create an autosave. Autosaving on connected Windows clients will be disabled regardless of autosave_only_on_server option.",
  "non_blocking_saving": false,

"_comment_segment_sizes": "Long network messages are split into segments that are sent over multiple ticks. Their size depends on the number of peers currently connected. Increasing the segment size will increase upload bandwidth requirement for the server and download bandwidth requirement for clients. This setting only affects server outbound messages. Changing these settings can have a negative impact on connection stability for some clients.",
  "minimum_segment_size": 25,
  "minimum_segment_size_peer_count": 20,
  "maximum_segment_size": 100,
  "maximum_segment_size_peer_count": 10
}

Logs
+ FACTORIO_VOL=/factorio
+ LOAD_LATEST_SAVE=true
+ GENERATE_NEW_SAVE=false
+ SAVE_NAME=
+ mkdir -p /factorio
+ mkdir -p /factorio/saves
+ mkdir -p /factorio/config
+ mkdir -p /factorio/mods
+ mkdir -p /factorio/scenarios
+ mkdir -p /factorio/script-output
+ [[ ! -f /factorio/config/rconpw ]]
+ [[ ! -f /factorio/config/server-settings.json ]]
+ [[ ! -f /factorio/config/map-gen-settings.json ]]
+ [[ ! -f /factorio/config/map-settings.json ]]
++ find -L /factorio/saves -iname '*.tmp.zip' -mindepth 1
++ wc -l
+ NRTMPSAVES=0
+ [[ 0 -gt 0 ]]
+ [[ '' == \t\r\u\e ]]
++ id -u
+ [[ 0 = 0 ]]
+ usermod -o -u 845 factorio
usermod: no changes
+ groupmod -o -g 845 factorio
+ chown -R factorio:factorio /factorio
+ SU_EXEC='su-exec factorio'
+ sed -i '/write-data=/c\write-data=\/factorio/' /opt/factorio/config/config.ini
++ wc -l
++ find -L /factorio/saves -iname '*.zip' -mindepth 1
+ NRSAVES=1
+ [[ false != true ]]
+ [[ 1 == 0 ]]
+ [[ false == true ]]
+ FLAGS=(--port "$PORT" --server-settings "$CONFIG/server-settings.json" --server-banlist "$CONFIG/server-banlist.json" --rcon-port "$RCON_PORT" --server-whitelist "$CONFIG/server-whitelist.json" --use-server-whitelist --server-adminlist "$CONFIG/server-adminlist.json" --rcon-password "$(cat "$CONFIG/rconpw")" --server-id /factorio/config/server-id.json)
++ cat /factorio/config/rconpw
+ [[ true == true ]]
+ FLAGS+=(--start-server-load-latest)
+ exec su-exec factorio /opt/factorio/bin/x64/factorio --port 34198 --server-settings /factorio/config/server-settings.json --server-banlist /factorio/config/server-banlist.json --rcon-port 27015 --server-whitelist /factorio/config/server-whitelist.json --use-server-whitelist --server-adminlist /factorio/config/server-adminlist.json --rcon-password aeZ0ohphui7pieT --server-id /factorio/config/server-id.json --start-server-load-latest
   0.000 2020-08-31 12:11:33; Factorio 1.0.0 (build 54889, linux64, headless)
   0.000 Operating system: Linux
   0.000 Program arguments: "/opt/factorio/bin/x64/factorio" "--port" "34198" "--server-settings" "/factorio/config/server-settings.json" "--server-banlist" "/factorio/config/server-banlist.json" "--rcon-port" "27015" "--server-whitelist" "/factorio/config/server-whitelist.json" "--use-server-whitelist" "--server-adminlist" "/factorio/config/server-adminlist.json" "--rcon-password" <private> "--server-id" "/factorio/config/server-id.json" "--start-server-load-latest"
   0.000 Read data path: /opt/factorio/data
   0.000 Write data path: /factorio [76978/100277MB]
   0.000 Binaries path: /opt/factorio/bin
   0.025 System info: [CPU: AMD Ryzen 5 2600X Six-Core Processor, 2 cores, RAM: 3944 MB]
   0.025 Environment: DISPLAY=<unset> WAYLAND_DISPLAY=<unset> DESKTOP_SESSION=<unset> XDG_SESSION_DESKTOP=<unset> XDG_CURRENT_DESKTOP=<unset> __GL_FSAA_MODE=<unset> __GL_LOG_MAX_ANISO=<unset> __GL_SYNC_TO_VBLANK=<unset> __GL_SORT_FBCONFIGS=<unset> __GL_YIELD=<unset>
   0.025 Running in headless mode
   0.033 Loading mod core 0.0.0 (data.lua)
   0.147 Loading mod base 1.0.0 (data.lua)
   0.776 Loading mod base 1.0.0 (data-updates.lua)
   1.010 Checksum for core:<PHONE_NUMBER>
   1.010 Checksum of base:<PHONE_NUMBER>
   1.397 Prototype list checksum:<PHONE_NUMBER>
   1.432 Info PlayerData.cpp:68: Local player-data.json available, timestamp<PHONE_NUMBER>
   1.432 Info PlayerData.cpp:75: Cloud player-data.json unavailable
   1.434 Factorio initialised
   1.435 Info ServerSynchronizer.cpp:28: nextHeartbeatSequenceNumber(0) initialized Synchronizer nextTickClosureTick(0).
   1.435 Info ServerMultiplayerManager.cpp:769: updateTick(4294967295) changing state from(Ready) to(PreparedToHostGame)
   1.435 Info ServerMultiplayerManager.cpp:769: updateTick(4294967295) changing state from(PreparedToHostGame) to(CreatingGame)
   1.435 Loading map /factorio/saves/prepetroleo.zip: 4403365 bytes.
   1.464 Loading level.dat: 9272987 bytes.
   1.470 Info Scenario.cpp:187: Map version 1.0.0-0
   1.835 Loading script.dat: 427 bytes.
   1.838 Checksum for script /factorio/temp/currently-playing/control.lua:<PHONE_NUMBER>
   1.840 Info UDPSocket.cpp:27: Opening socket at (IP ADDR:({<IP_ADDRESS>:34198}))
   1.840 Hosting game at IP ADDR:({<IP_ADDRESS>:34198})
   1.840 Info HttpSharedState.cpp:54: Downloading https://auth.factorio.com/generate-server-padlock-2?api_version=4
   2.590 Info AuthServerConnector.cpp:66: Obtained serverPadlock for serverHash (GpKyA7W3CxjxNH0yXEIl0sR21IDYXxzN) from the auth server.
   2.590 Info ServerMultiplayerManager.cpp:769: updateTick(1347597) changing state from(CreatingGame) to(InGame)
   2.757 Info ServerRouter.cpp:642: Asking pingpong servers (pingpong1.factorio.com:34197, pingpong2.factorio.com:34197, pingpong3.factorio.com:34197, pingpong4.factorio.com:34197) for own address
   2.757 Info UDPSocket.cpp:39: Opening socket for broadcast
   2.758 Info RemoteCommandProcessor.cpp:131: Starting RCON interface at IP ADDR:({<IP_ADDRESS>:27015})
   2.758 Info CommandLineMultiplayer.cpp:277: Maximum segment size = 100; minimum segment size = 25; maximum-segment-size peer count = 10; minimum-segment-size peer count = 20
   2.759 Error InterruptibleStdioStream.cpp:61: Got EOF on stdin; closing
   2.953 Info ServerRouter.cpp:525: Own address is IP ADDR:({correctPublicIP:34198}) (confirmed by pingpong1)
   2.970 Info ServerRouter.cpp:525: Own address is IP ADDR:({correctPublicIP:34198}) (confirmed by pingpong3)
   3.108 Info ServerRouter.cpp:525: Own address is IP ADDR:({correctPublicIP:34198}) (confirmed by pingpong4)
   3.664 Info MatchingServer.cpp:114: Matching server game `8066169` has been created.
   3.664 Info ServerMultiplayerManager.cpp:701: Matching server connection resumed

I once started the Factorio game in the host (W10) of the VM and invited a friend through Steam and could connect, and even through the Public Servers browser (we think).
And tips on where should I look to try to solve this issue?

Did you try nmap on your public IP? Maybe your router does not forward the ports correctly.

@SuperSandro2000 just tried nmap and nc while tethering from my phone so I'm outside of the network and nmap says open|filetered and nc says succeed!
Using online UDP testing websites they say Open or filtered.

filtered means that the packet got dropped by the firewall. It is definitely not open which you want.
Closing this because this is unrelated to the docker image. Feel free to further discuss this.

Did a test with:
server
$ nc -ul 34198

client outside of server network
$ nc -u correctPublicIP 34198
$ factorio test

And received factorio test at the server, so it seems that the port is open.

No clue but I don't believe this is an issue with the docker image but rather with your setup.

I think so to.
Continued to do test in my network and now fails also in windows.
Sorry about the time wasted.
On Tue, 1 Sep 2020 at 18:03, Sandro<EMAIL_ADDRESS>wrote:

No clue but I don't believe this is an issue with the docker image but
rather with your setup.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/factoriotools/factorio-docker/issues/343#issuecomment-684962467,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA5U35CSXULPQRW5TDZDYTSDULLJANCNFSM4QQOUO5A
.
--

Alejandro Cámara López
Interaction Designer
alejandrocamara.info

@thewinger did you get this resolved? I am running into something similar on the helm chart. I believe is has something to do with the way the cni affects traffic leaving the cluster.


Does not allow you to login if "Show the debug toolbar in the control panel" is checked in user preferences

Hey there. Running Craft CMS 4.8.0, plugin version 3.3.6, and I can't login when I have "Show the debug toolbar in the control panel" checked in my User preferences. It asks for the authenticator code, but never approves it. Disabling the plugin, unchecking the box, and re-enabling the plugin works.

@illiterati, the debug toolbar does requests before 2fa authentication, causing the plugin to logout again. If these would be allowed, it would create a possible security vulnerability. You could turn of 2fa during development in the php settings file.

This is probably something that should be mentioned in the Read Me, as I spent an hour debugging it this morning.


Trello-1808: Tie down access to content-stores AWS

The content-store external ALB and external ELB in staging_aws and
production_aws are currently opened to the world. Here we are limiting
access to the content-stores from the corresponding environment in
Carrenza. This is so teh frontends in Carrenza which have not been
migrated yet are allowed to talk with the content-store.
Allowing the GDS Office IP ranges in also.
Solo: @ronocg<EMAIL_ADDRESS>

Do we want to keep ${var.office_ips} in there?

yep the office ip ranges have been re-added


Update to 2018 edition

Thanks for sharing this code, and writing the fantastic blog posts to go along with it. This PR attempts to modernize this crate, but updating it to 2018 edition and correcting all clippy issues.

there is also an 2018 edition branch :wink:

Oops, I missed that. Thanks for the feedback, I'll close this.


Highly confusing delay issue

There is an instruction that states "Go to the Azure Cosmos DB account page, and select Keys. Copy the values to use in the web application you create next." Many sections including the Keys section and the quick start picture above the line I just shared, do
not load until a little (took me like... 3-5 minutes) while after the supposed completion shown by Step 8. There should really be something to indicate that sections are not ready yet in the Azure Portal as well as some sort of acknowledgement of this in the docs.

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

ID: f7ad84a6-f6b6-dd21-e7be-865d922c7ad4
Version Independent ID: 1f5a0315-d7b4-3f8f-3b1d-ae90c807859d
Content: Tutorial: Build a Node.js web app with Azure Cosmos DB JavaScript SDK to manage SQL API data
Content Source: articles/cosmos-db/sql/sql-api-nodejs-application.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @deborahc
Microsoft Alias: dech

@AuWiMo Thank you for the feedback. We are actively investigating and will get back to you soon.

@AuWiMo Thanks for your patience. I tried it by creating a new Cosmos Account and the sections are loaded within seconds. Not sure if it was some other issue. Could you please once again try it and let me know if you still have the same issue?

@AuWiMo Awaiting your reply.

This time it worked successfully. Last time I the problem persisted through a few refreshed over a few minutes, so it definitely wasn't user error

@AuWiMo Thanks for replying back. It could be temporary glitch due to any reason. Please let us know if you face similar issue again and we can analyze it further.
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


In POSIX sh, 'local' is undefined. [SC2039]

@thoughtbot/shell We use local quite a few times in the script but ShellCheck doesn't like it. Thoughts on how to approach it?

The best I got is this: https://stackoverflow.com/questions/18597697/posix-compliant-way-to-scope-variables-to-a-function-in-a-shell-script#answer-18600920
Which points out that local is understood by most OSes.

Cool, thanks. Going to add this config to my vimrc:
let g:syntastic_sh_shellcheck_args="-e SC2039"


hivemq-platform-operator: when service account is not created, Service is also not created

See: https://github.com/hivemq/helm-charts/blob/master/charts/hivemq-platform-operator/templates/service.yml#L1
Probably due to copy-paste?

Thanks a lot @avthart for reporting this. It is indeed a bug here.
Will raise a PR soon to fix this.
Keep you updated.

Fixed by https://github.com/hivemq/helm-charts/pull/361


Housekeeping: use plain enums with IN clauses where possible

In some parts of waltz we needlessly convert enum values to string equivalents eg.
dsl.select().from(Tables.FOO).where(FOO.COL.in(MyEnum.BAA.name());
jOOQ is perfectly happy to allow this instead:
dsl.select().from(Tables.FOO).where(FOO.COL.in(MyEnum.BAA);
As a first pass looking for code that matches the following search str: ENTITY_LIFECYCLE_STATUS.in(  - gives a few candidates where we are mapping over a set of enums to get their string's and using those in the in clause.
`

Here's an example:
List<String> validStatusNames = map(
          options.entityLifecycleStatuses(),
          s -> s.name());

Condition statusCondition = LOGICAL_DATA_ELEMENT.ENTITY_LIFECYCLE_STATUS.in(validStatusNames);

where we could just have written:
Condition statusCondition = LOGICAL_DATA_ELEMENT.ENTITY_LIFECYCLE_STATUS.in(options.entityLifecycleStatuses);

Hi @vijaya-lakshmi-venkatraman
Thanks for the interest in making a contribution!
I've taken a look at your suggestion, however I'm not sure it will work as I don't think jOOQ offers overloaded variants of .eq which can accept enums.
Thanks, /dw

I've assigned the issue to you for tracking purposes.  It doesn't mean we are expecting you to contribute, just that we want to ensure that we don't end up with duplicate work.  Let me know if you'd rather not be listed.


Feat/fix auto threads

This SHOULD be all the rewrites required to make this work. Basically it creates the HELPER_THREAD_CHANNELS config that can be used to check if a channel is a help thread, and AUTO_THREAD_CHANNELS extends that list. That way we can reuse all the code for auto threading and just extend it with the help feature.
According to my tests this should be all the required fixes but I'll need someone to try to break it before it goes live.

I guess the only thing would be it seems newlines get stripped, so maybe adding in a space if there's a newline?
Other than that I just ran everything through prettier to do some formatting. Give me a shout when you have time to fire it up online and I can do some testing for you!


Improve documentation for mappings

Feature request
I'd like to be able to disable the fuzzy finding feature. I've found it's too slow for use in Scala projects. But even if it were performant I'd still like to disable it and stick to using Telescope for fuzzy finding.
In its place I'd like to be able to perform search jumps with the normal vim /.
Bug
As well the above feature request, I also noticed what I assume is a bug. I tried to remap the fuzzy find from ["/"] = "fuzzy_finder" to ["F"] = "fuzzy_finder", but then instead of changing the mapping, the default / is kept in addition to the new F, and both bindings appear in the ? help menu
I'm using branch v2.x

To disable a single built-in mapping, you can map it to the string "none":
    ["/"] = "none"

You can also disable all built-in mappings if you want to explicitly map them all yourself:
    use_default_mappings = false

Yep, that worked. Thanks!
That approach to mapping is a little unusual, so it might be worth noting in the readme?

@cseickel I'm reviewing old issues. Maybe this can be closed? (The title itself is qualified for a standalone issue IMO)


Command line resource (for machine_type) results in parsing error

The error we are seeing is the following:
InputFunctionException in line 7 of /home/kbinder/snakemake-tutorial-data/Snakefile:
Error:
  NameError: name 'n1' is not defined
Wildcards:
  sample=B
Traceback:
  File "<string>", line 1, in <module>

Triggered with cloning the snakemake-tutorial-data, and then running snakemake master via:
$ snakemake --google-lifesciences --default-remote-prefix snakemake-testing-data --use-conda --google-lifesciences-region us-central1 --default-resources machine_type=n1-standard-4

The strategy to get a resource from the command line has a fallback function that tries to evaluate the val parameter:
class DefaultResources:
    def __init__(self, args=None):
        self.args = args

        def fallback(val):
            def callable(wildcards, input, attempt, threads, rulename):
                value = eval(
                    val, {"input": input, "attempt": attempt, "threads": threads}
                )
                return value
            return callable

        self.parsed = dict(_cores=1, _nodes=1)
        if self.args is not None:
            self.parsed.update(parse_resources(args, fallback=fallback))

And this results in a NameError because val turns out to be n1-standard-1 and we obviously can't evaluate that in Python. I'm not sure if it's the case that this fallback shouldn't be triggered to begin with, but for a potential fix we can catch the NameError and just return the value, which (with testing) seems to resolve the issue and honor the command line resource request.
class DefaultResources:
    def __init__(self, args=None):
        self.args = args

        def fallback(val):
            def callable(wildcards, input, attempt, threads, rulename):
                try:
                    value = eval(
                        val, {"input": input, "attempt": attempt, "threads": threads}
                    )
                # Triggers for string arguments like n1-standard-4
                except NameError:
                    return val
                return value

            return callable

        self.parsed = dict(_cores=1, _nodes=1)
        if self.args is not None:
            self.parsed.update(parse_resources(args, fallback=fallback))

I'll open a PR with this fix for discussion soon.

Fixed by PR #559.


Add diagnose code for NPE thrown in ExceptionPlaceHolder

See https://github.com/gradle/gradle-private/issues/1368
There was a weird NPE in https://builds.gradle.org/viewLog.html?buildId=14073871&buildTypeId=Gradle_Check_Quick_Java8_Oracle_Linux_modelGroovy . This PR adds some diagnose code to catch the potential cause.

This is temporary diagnose code. You're right, I'll make it into log.

@marcphilipp If you don't object, I'll merge this one.


Convert all Ark shaders to Rust GPU

This will be a follow up to #4 and try to convert all of our static vertex and pixel shaders in Ark to Rust and compile to SPIRV.

This has been completed. 🙂


WIP: imx8mmsolidrun with eMMC support

Add support for Hummingboard Solid with eMMC. Requires LmP v81.
Boot mode configuration: https://developer.solid-run.com/knowledge-base/hummingboard-pulse-ripple-boot-select/
Device must be connected to PC from the top most USB port with USB type-A to USB type-A OTG cable (e.g. https://www.amazon.com/gp/product/B01KRO8D20).
Ethernet cable must be connected to CON1 -connector (closest to power connector).

Rebased. I also created a separate PR about renaming wigwag imx8mmevk changes as generic mx8mm (see pr #297).
Still to be done: Instead of deleting lmp.cfg and boot.cmd, I will create imx8solidrun versions of those to not break imx8mmevk build.

This PR is now ready for review.

Added do not merge flag until Pulse/Ripple HW compatibility issues with mPCI has been resolved.

Closing this as PR checker cannot access fork. New PR: https://github.com/PelionIoT/meta-pelion-edge/pull/300


MINOR:Incorrect/canonical use of constants in AdminClientConfig and StreamsConfigTest

Could you help to review? @dengziming

Thanks for the catch! LGTM.


Doc  error

Version
5.0.2
Reproduction link
https://echarts.apache.org/zh/option.html#series-lines.markPoint
Steps to reproduce
quote: 'For example, [0, '50%'] means to move upside side position of symbol height. It can be used to make the arrow in the bottom to be at data position when symbol is pin.'
What is expected?
it should be [0, '-50%']
What is actually happening?
It can be used to make the arrow in the bottom to be at data position when symbol is pin.

PR is welcome

https://github.com/apache/echarts-doc/pull/170


[FEATURE] Try to support cross-platform

Anole plans to run on win32 platform!
For this target, we need to do the followings:

use macro to distinguish different platforms, win32 and linux in plan
edit old code of loading cpp module
edit old code of REPL which uses library readline and signal of linux to implement the main loop

update CMakeLists.txt

Fuck windows!


issue with creating qfile for complex scopus queries

When you have complex (and long) scopus queries, the qfile creation mechanism fails. For example, for the search query "test" (with double quotes). Rather than just replacing slashes of the queries (query.replace('/', 'slash') in scopus_search.py how about implementing a solution like the one mentioned here (see the accepted solution): http://stackoverflow.com/questions/7406102/create-sane-safe-filename-from-any-unsafe-string
Then again, when the query is very long, this will still generate a very long file name. Therefore may be it's better to hash the query string and use the hash to create the file.

The hash is probably the best approach. Any chance you want to do it and make a pull request?

also noticed a few more issues, might work on these  as and when i have time :)

PR #40 will solve this issue.

This change has gone live with scopus 1.0 (b7f2bcb9bca3777b2bb7fb6f272865188933b5a9) on Saturday.


onChange fires when value is changed programatically

onChange is called when the Codemirror component receives a new value when rendering.
Test case: https://gist.github.com/insin/9e507894f3858028c7e5
Live version: http://bl.ocks.org/insin/raw/9e507894f3858028c7e5/
Every call to onChange is alert()-ed, Clicking the "Change value" button changes the value in the component's state.
Just got bit by this using Redux, as updating some state which was being passed to Codemirror resulted in another dispatch firing via onChange, resulting in old state being retained. Worked around it myself by having onChange check if it received the same value as was being used for rendering, but is this behaviour something that's possible to control, or it is a consequence of how Codemirror itself is implemented?

The example doesn't really work for me (some errors in the console). However, the change event from CodeMirror instance has a second argument which describes the change. If value comes from  setValue() call, then change.origin == 'setValue', so I think that may be used to differentiate between the changes.

I've been battling with this for a while, and it's messy. I see state stored in three separate places:

The CodeMirror object;
The react-codemirror React component (in the _currentCodemirrorValue property);
And my own component's state.

Synchronizing these three pieces of state is a nightmare. The way the state is sent to and from each object is unpredictable. I sunk a few hours into trying to patch up the state synchronization and haven't figured it out yet. Instead, I think we should (a) admit that having three separate "sources of truth" is a bad code smell, and (b) restructure the flow of data in the react-codemirror component.
The CodeMirror library is going to maintain its own internal state; there's no easy way to get around that. Therefore, the other state, in react-codemirror and the client app, need to go. I think this implies that the react-codemirror component should not expose a value attribute, just a defaultValue.
Am I off base here?

Here's a test case that demonstrates this failure-to-synchronize: https://jsfiddle.net/n1gz4hLn/1/
Watch out, this will crash your tab. When you try to type, you enter an infinite loop.

I took a stab at fixing this by removing the state from the CodeMirror React component. It's not perfect, but remove a lot of the code smell.
#35

The PR reference by this issue has been merged. Should this issue be closed?

Hey all. I was struggling with some related issues and decided to roll another wrapper for this at react-codemirror2. No external deps, state free, and works well for my use case. Please give it a look if you are feeling adventurous. I'd like to actively maintain it moving forward. I'm also looking for suggestions to craft it in the best fashion moving forward, so feel free to open up an issue with any wants/tips/etc you may want. Thanks!


Add gzip/compression support

Adds Gzip support for requests, really useful for saving bandwidth with bulk requests.

that seems fine, if we could get a couple unit tests this could get merged.

Added some tests, I tried to make an error check by getting json.Marshal to return an error based on what I passed in but it handled everything I threw at it.

thanks!


Fortify scan

Hi ,
I have been facing an issue after working with react . My react pages are failing fortify scans as they are complaining of the password as per the screenshot below . The password issue is shown at quite a few places and fortify scan is showing it as high risk issue.

Please suggest me if I need to upgrade the versions of my react libraries

ReactAlertBox is not provided by React. It's something in your code or one of your dependencies.


How to pass loop to test request?

sanic.test_client.get(url, loop=loop) # Do not work

try pass in server_kwargs = { 'loop': MyEventLoop}
are you wanna use your own event loop for testing ? if so, make sure that your event loop is consistent everywhere.
If you're using pytest, you can take a look https://github.com/yunstanford/pytest-sanic for easy testing.

@tonal please reopen this if you're still having issues and need more help.

@r0fls is this still the correct way to pass a loop to my test_client requests? I believe passing a loop to app.run() was removed here https://github.com/channelcat/sanic/pull/335.


Preloader - Running into Compilation Error

I am trying to use the library. I was trying to use your library with the example electron-forge-typescript-webpack as a template.
However, I am running into an issue with the following:
An unhandled rejection has occurred inside Forge:
Error: Compilation errors in the preload: group_0:
  assets by info 2.98 KiB [immutable]
    assets by path *.json 150 bytes
      asset main_window_compiled.783c92c0224cb1109eb5.hot-update.json 44 bytes [emitted] [immutable] [hmr]
      asset preload_compiled.783c92c0224cb1109eb5.hot-update.json 40 bytes [emitted] [immutable] [hmr]
      + 2 assets
    assets by path *.jsc 2.25 KiB
      asset main_window.compiled.783c92c0224cb1109eb5.hot-update.jsc 1.13 KiB [emitted] [immutable] [hmr] (name: main_window.compiled)
      asset preload.compiled.783c92c0224cb1109eb5.hot-update.jsc 1.13 KiB [emitted] [immutable] [hmr] (name: preload.compiled)
    assets by path *.js 598 bytes
      asset main_window.783c92c0224cb1109eb5.hot-update.js 301 bytes [emitted] [immutable] [hmr] (name: main_window)
      asset preload.783c92c0224cb1109eb5.hot-update.js 297 bytes [emitted] [immutable] [hmr] (name: preload)
  asset main_window/preload.js 254 KiB [emitted] (name: main_window)
  asset preload/preload.js 254 KiB [emitted] (name: preload)
  asset main_window.compiled/preload.jsc 227 KiB [emitted] (name: main_window.compiled)
  asset preload.compiled/preload.jsc 227 KiB [emitted] (name: preload.compiled)
  Entrypoint preload 255 KiB = preload/preload.js 254 KiB preload.783c92c0224cb1109eb5.hot-update.js 297 bytes
  Entrypoint main_window 255 KiB = main_window/preload.js 254 KiB main_window.783c92c0224cb1109eb5.hot-update.js 301 bytes
  Entrypoint preload.compiled 228 KiB = preload.compiled/preload.jsc 227 KiB preload.compiled.783c92c0224cb1109eb5.hot-update.jsc 1.13 KiB
  Entrypoint main_window.compiled 228 KiB = main_window.compiled/preload.jsc 227 KiB main_window.compiled.783c92c0224cb1109eb5.hot-update.jsc 1.13 KiB
  runtime modules 106 KiB 50 modules
  cached modules 175 KiB [cached] 32 modules
  javascript modules 312 bytes
    node:assert/strict 39 bytes [built] [1 error]
    node:child_process 39 bytes [built] [1 error]
    node:fs 39 bytes [built] [1 error]
    node:module 39 bytes [built] [1 error]
    node:path 39 bytes [built] [1 error]
    node:vm 39 bytes [built] [1 error]
    node:v8 39 bytes [built] [1 error]
    node:zlib 39 bytes [built] [1 error]

  ERROR in node:assert/strict
  Module build failed: UnhandledSchemeError: Reading from "node:assert/strict" is not handled by plugins (Unhandled scheme).
  Webpack supports "data:" and "file:" URIs by default.
  You may need an additional plugin to handle "node:" URIs.
      at /home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/NormalModule.js:918:25
      at Hook.eval [as callAsync] (eval at create (/home/dallas/Github/mosaic/canvas-desktop/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:6:1)
      at Hook.CALL_ASYNC_DELEGATE [as _callAsync] (/home/dallas/Github/mosaic/canvas-desktop/node_modules/tapable/lib/Hook.js:18:14)
      at Object.processResource (/home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/NormalModule.js:915:8)
      at processResource (/home/dallas/Github/mosaic/canvas-desktop/node_modules/loader-runner/lib/LoaderRunner.js:220:11)
      at iteratePitchingLoaders (/home/dallas/Github/mosaic/canvas-desktop/node_modules/loader-runner/lib/LoaderRunner.js:171:10)
      at runLoaders (/home/dallas/Github/mosaic/canvas-desktop/node_modules/loader-runner/lib/LoaderRunner.js:398:2)
      at NormalModule._doBuild (/home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/NormalModule.js:905:3)
      at NormalModule.build (/home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/NormalModule.js:1079:15)
      at /home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/Compilation.js:1374:12
      at NormalModule.needBuild (/home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/NormalModule.js:1408:26)
      at Compilation._buildModule (/home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/Compilation.js:1355:10)
      at /home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/util/AsyncQueue.js:305:10
      at Hook.eval [as callAsync] (eval at create (/home/dallas/Github/mosaic/canvas-desktop/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:6:1)
      at AsyncQueue._startProcessing (/home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/util/AsyncQueue.js:295:26)
      at AsyncQueue._ensureProcessing (/home/dallas/Github/mosaic/canvas-desktop/node_modules/webpack/lib/util/AsyncQueue.js:282:12)
   @ ./node_modules/bytenode/lib/index.js 3:15-44
   @ ./src/main/preloaders/preload.loader.js 1:0-19

Is this something to do with my preloader? I cant find much on the error.

Hello @fras2560
I imagine it has something to do with this part of the message
You may need an additional plugin to handle "node:" URIs.


[Enhancement]  Reduce unnecessary computation

Fixes #issue
This code changes can reduce unnecessary computation,  make  code better
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.0
[x] 2.5
[x] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 7 / 7 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/statistics/BinaryPredicateStatisticCalculator.java
7
7
100.00%
[]

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4


the code on npm.js   is not up-to-date(Compare with [github]

Node version (or tell us if you're using electron or some other framework):
v12.18.1
ShellJS version (the most recent version/Github branch you see the bug on):
0.8.5
Operating system:
win10
Description of the bug:
the code on npm.js   is not up-to-date(Compare with github)，can you update it?
Example ShellJS command to reproduce the error:
// see [npm](https://www.npmjs.com/package/shelljs?activeTab=explore) mv.js line 115,116
cp('-r', src, thisDest);
rm('-rf', src);
// the correct code on github is
cp({ recursive: true }, src, thisDest);
rm({ recursive: true, force: true }, src);

I haven't pushed an update for a while. There are some blocking issues on tip-of-tree that need to be resolved before the 0.9 update. You can watch https://github.com/shelljs/shelljs/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+milestone%3Av0.9.0 to see progress.


docs: more examples for Orbit?

It may be useful to show what is possible with motion-ui beyond the default setup.
Some old demo that does not yet work perfectly: https://danielruf.github.io/orbit-test/dist/index.html
Not sure if this would be helpful.

Is this helpful in any way and would we have / need more examples?


[WIP] [Text] Shaped buffer pooling

What does the pull request do?
I'm pooling shaped buffer array because it's stores a lot of data. Getting good results.
This is a continue of my previous PR. I will ship all perf PR's as a small well-tested changes that can be easily reviewed.
What is the current behavior?

Method
Mean
Error
StdDev
Median
Gen 0
Gen 1
Allocated

BuildTextLayout
670.2 us
13.05 us
32.27 us
658.5 us
3.9063
-
26 KB

BuildEmojisTextLayout
1,496.4 us
13.38 us
12.52 us
1,492.9 us
3.9063
-
31 KB

BuildManySmallTexts
5,513.6 us
109.66 us
294.61 us
5,402.5 us
429.6875
210.9375
2,656 KB

VirtualizeTextBlocks
5,914.6 us
111.34 us
119.13 us
5,926.3 us
414.0625
54.6875
2,538 KB

Current results is not that impressive as I want them to be. I will work i bit more and hopefully improve this PR today or tomorrow

I wonder why VirtualizeTextBlocks is slower with pooling

Test is long. and error is huge. so I thing CPU is fine. just measurement was a bit incorrect

Method
Mean
Error
StdDev
Median
Gen 0
Gen 1
Allocated

VirtualizeTextBlocks
5,053.0 us
28.44 us
25.21 us
5,045.5 us
414.0625
54.6875
2,538 KB

here is re-test. no changes error is a bit lower -> cpu time is lower


WIP DO NOT MERGE - PERFSCALE-1848 - Add initial DAG for windows containers.

Description
An initial attempt at a DAG to track the max pods-per-node baseline for Windows worker nodes.
Fixes

Dependencies on these merging first (in this order):
https://github.com/cloud-bulldozer/e2e-benchmarking/pull/462
https://github.com/cloud-bulldozer/scale-ci-deploy/pull/200

Codecov Report

Merging #250 (db82835) into master (d9792bb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #250   +/-   ##
=======================================
  Coverage   95.43%   95.43%
=======================================
  Files          22       22
  Lines         964      964
=======================================
  Hits          920      920
  Misses         44       44

Flag
Coverage Δ

gha
95.43% <ø> (ø)

python-3.9
95.43% <ø> (ø)

unit
95.43% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ags/openshift_nightlies/tasks/install/openshift.py
98.36% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Support legacy defaultForNew predicate in ACLs

Support legacy defaultForNew predicate in ACLs
Implements #251
Checklist

[ ] All acceptance criteria are met.
[ ] Relevant documentation, if any, has been written/updated.
[ ] The changelog has been updated, if applicable.
[x] New functions/types have been exported in index.ts, if applicable.
[x] Commits in this PR are minimal and have descriptive commit messages.

I don't think this is quite ready, but wanted to push it up for feedback. I suspect this needs more unit tests, but I'm not totally sure about the way I implemented the existing one - any suggestions for alternate approaches?

 Thanks!


Paren-matching too aggressive sometimes

Sometimes,  when editing code I have encountered issues with the paren-matching being far too aggressive. This happens especially with comments. When I start a comment while having trailing parens those then become comments and then don't get matched so it assumes it needs to close expressions again and the comment ends up being outside the desired block. 😿
I'm adding a screencap that shows an example.

You can actually keep typing after hitting enter after the comment, and it will move the paren to the right spot. It's trying to prevent a broken AST even temporarily. Please try that out and let me know if it works for you.

Not sure what you mean. When I hit enter after a comment I get tabbed out and have to tab back in. This does the trick, but it's a bit annoying.
Also note how this doesn't happen with strings.

Yep that screenshot is what I was referring to. Is the main issue the fact that you lose your indent level? I can definitely see why that would be annoying. I will try to figure out a workaround for that. The general paren matching behavior is inherent in parinfer, though.

I think it's either about losing indent level (I wouldn't have cared so much if that had been the case or maybe handle it like quotes. I'm not sure which would be better although I'm leaning towards quotes.

Closing this because the parinfer behavior is pretty set in stone at this point.


Fixed symfony version constraints

@GrahamCampbell  Thanks for the contribution. I am a little nervous about maintaining a dependencies on either the 2.x or 3x. versions of Symfony, that see, like it is pretty risky. I don;t have a lot of experience with 3.x, so I'd like to get your thoughts.
I am also open for cutting a 2.x branch and requiring Symfony 3.x. Do you think this approach is OK?

@cpliakas Your current version already allows 3.x and 4.x and 5.x and 6.x...
This PR fixes that...

like it is pretty risky

Not really. Symfony 3 is the same as Symfony 2 just with some old methods removed.

Your current version already allows 3.x and 4.x and 5.x and 6.x...

Ah, I see. That's not good.

Not really. Symfony 3 is the same as Symfony 2 just with some old methods removed.

Great, thanks for your perspective.
Given these two points this is good to merge. Thanks again for the contribution.

Given these two points this is good to merge.

Great.
@cpliakas Thanks for this library. The ability to set the private key so easily is so useful to me. :)

Reflected in the 1.5.0 release.

Awesome. :)


The "platformio-ide-terminal:insert-selected-text" is not working.

I installed the package but one of the features that must meter for mi is "insert-selected-text"  is not working.
I use "ctrl-enter" but it is not passing the code to the shell.
I test it on windows 10 and 7.

By default in atom,  ctrl+enter inserts a new line. To make ctrl-enter behave as expected, you could add this to keymap.cson
'atom-workspace atom-text-editor:not([mini])':
  'ctrl-enter': 'platformio-ide-terminal:insert-selected-text
```'

If these are necessary changes, this should be documented. It can't be considered a 'feature' if it doesn't do this by default.

For mac users' reference:
'atom-workspace atom-text-editor:not([mini])':
'cmd-enter': 'platformio-ide-terminal:insert-selected-text'

ctrl-enter to insert-selected-text worked by default when i installed platformio-ide-terminal on my mac.
However, after I later installed the linter-shellcheck package, ctrl-enter stopped working. So i followed these instructions and added the line to keymap.cson, and the ctrl-enter function was restored.  (I prefer ctrl-enter over cmd-enter, in my case).
by the way, when copying the above line, remove the newline between ([mini])': and  'cmd-enter':
Thank you!

i have now also successfully used this (to re-bind ctrl-enter) in the keymap.cson in Atom installed on Ubuntu 16 inside VirtualBox on macOS (whew!):
'atom-workspace atom-text-editor:not([mini])': 'ctrl-enter': 'platformio-ide-terminal:insert-selected-text'

Ubuntu 18.04 LTS Atom solve the bug thanks all the guys above -
change paltform ide key binding
'atom-workspace atom-text-editor:not([mini])':
'ctrl-enter': 'platformio-ide-terminal:insert-selected-text'

thx for feedback @atongsa
Related fix in this project https://github.com/platformio/platformio-atom-ide-terminal/commit/de2cc2efd88e5d14dbbb25bf9cdcd161c36f86a2 and cleanup https://github.com/platformio/platformio-atom-ide-terminal/commit/23343e8b06e3b56224c518d8c584f9af80f1e2d6


Add you page

Adds in the new Scratchinfo you page!
Thank you to everyone, this is the first time I've done something with a databse so this is really fun!

I fixed the merge conflict hopefully 😄

wait i accidentally duplicated so much code aaaahhh

merging at 2:00 if I remember 😀


Is it support for IOS?

I have tried to build it with IOS target, but always not found the MusicKit ?

I have tried to run it on IOS, and both Mac and IPhone, didn't hear MIDI sound output ?

I haven't created an iOS example yet, but the examples do work on a Mac. The example apps transform MIDI, but do not produce sound. If you have a MIDI keyboard, the ChordRecognition example will identify the chords you play.
Creating more intuitive examples with audio output is definitely on the roadmap, but I've been pretty busy lately.


(presumed bug) Lost access to repo from webapp after mistaken actions with collaborators list

Bug Description
Pre-conds : We had a Gogs with 7 users, one organization, one repo inside that organization. Nothing else.
What happened : One user was new and we hadn't set him up as a member of the organization. Wanting to do so, instead I mistakenly added him as a repo collaborator. Shortly after, I rightly added him to the organization, and went back to remove him from being the only person  on the repo's collaborator list. Right after I clicked the little red cross, I got a big 404.
Post-conds : the repo is unaccessible through the webapp. The direct URL yields a 404, and the welcome screen does not show it. However, the repo count is valid. I looked into the DB repository table. The line was there, but all fields seemed reset to default values, though I'm not completely sure about that. I've tried restarting Gogs and all, to no avail. Trying to access the repo over Git through https brings up an authentication failure.
Current mitigation : I tried to create a new repo and copy the old repo's .git folder content. Git was happy enough like that, by Gogs' webapp still showed me a newly created repo, even after restarting the Gogs server. So I deleted the new repo and went on as follows.
I've cloned the old repo directly in git command line, thus making a temporary client repo on the server outside of gogs' dir tree, using the file system as the upstream (no network, pointing to my old .git directory in gogs-repositories). Then I've created a new repo through the webapp, and pushed all branches to it through https://localhost. Gogs was happy with that, though for now I have not been able to restore access to the original repo.
Please see screen capture, showing the discrepancy between repo count and repo list.
http://kibu.fr/entrepot/imgdiv/gogsbug.png
Gogs Version
<IP_ADDRESS>5 Beta
Git Version
1.9.5.msysgit.1
System Type
Windows Server 2008 R2 Standard
Service Pack 1
Xeon CPU ; 2 Go RAM (VM), over 1.0Go free
Sqlite
Error Log
There's stuff in the C:\gogs\log\gogs.log several days BEFORE (we had a days long break), and more starting back ten minutes AFTER the event. But absolutely no trace of the event itself. I've checked for clock discrepancies, and they're not accountable for that gap in the log.
In gogs-nssm.log the event has been captured as such :
[Macaron] Started GET /FakeOrgName/constellation/settings/collaboration for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation/settings/collaboration 200 OK in 66.0038ms
[Macaron] Started GET /api/v1/users/search?q=o for <IP_ADDRESS>
[Macaron] Completed /api/v1/users/search?q=o 200 OK in 4.0003ms
[Macaron] Started GET /api/v1/users/search?q=p for <IP_ADDRESS>
[Macaron] Completed /api/v1/users/search?q=p 200 OK in 4.0002ms
[Macaron] Started GET /api/v1/users/search?q=pa for <IP_ADDRESS>
[Macaron] Completed /api/v1/users/search?q=pa 200 OK in 4.0002ms
[Macaron] Started GET /api/v1/users/search?q=pan for <IP_ADDRESS>
[Macaron] Completed /api/v1/users/search?q=pan 200 OK in 3.0001ms
[Macaron] Started POST /FakeOrgName/constellation/settings/collaboration for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation/settings/collaboration 302 Found in 80.0046ms
[Macaron] Started GET /FakeOrgName/constellation/settings/collaboration for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation/settings/collaboration 200 OK in 67.0038ms
[Macaron] Started GET /admin for <IP_ADDRESS>
[Macaron] Completed /admin 200 OK in 12.0007ms
[Macaron] Started GET /admin/users for <IP_ADDRESS>
[Macaron] Completed /admin/users 200 OK in 8.0004ms
[Macaron] Started GET /admin/users/3 for <IP_ADDRESS>
[Macaron] Completed /admin/users/3 200 OK in 8.0004ms
[Macaron] Started GET /admin/orgs for <IP_ADDRESS>
[Macaron] Completed /admin/orgs 200 OK in 8.0004ms
[Macaron] Started GET /org/FakeOrgName for <IP_ADDRESS>
[Macaron] Completed /org/FakeOrgName 200 OK in 17.001ms
[Macaron] Started GET /org/FakeOrgName/invitations/new for <IP_ADDRESS>
[Macaron] Completed /org/FakeOrgName/invitations/new 200 OK in 6.0003ms
[Macaron] Started GET /api/v1/users/search?q=p for <IP_ADDRESS>
[Macaron] Completed /api/v1/users/search?q=p 200 OK in 4.0002ms
[Macaron] Started GET /api/v1/users/search?q=pa for <IP_ADDRESS>
[Macaron] Completed /api/v1/users/search?q=pa 200 OK in 3.0002ms
[Macaron] Started GET /api/v1/users/search?q=pan for <IP_ADDRESS>
[Macaron] Completed /api/v1/users/search?q=pan 200 OK in 3.0002ms
[Macaron] Started POST /org/FakeOrgName/invitations/new for <IP_ADDRESS>
[Macaron] Completed /org/FakeOrgName/invitations/new 302 Found in 13.0007ms
2015/08/04 15:48:54 [T] New member added(FakeOrgName): pan
[Macaron] Started GET /org/FakeOrgName/members for <IP_ADDRESS>
[Macaron] Completed /org/FakeOrgName/members 200 OK in 18.001ms
[Macaron] Started GET /org/FakeOrgName for <IP_ADDRESS>
[Macaron] Completed /org/FakeOrgName 200 OK in 15.0009ms
[Macaron] Started GET /FakeOrgName/constellation for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation 200 OK in 334.0191ms
[Macaron] Started GET /FakeOrgName/constellation/settings for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation/settings 200 OK in 85.0049ms
[Macaron] Started GET /FakeOrgName/constellation/settings/collaboration for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation/settings/collaboration 200 OK in 68.0038ms
[Macaron] Started GET /FakeOrgName/constellation/settings/collaboration?remove=pan for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation/settings/collaboration?remove=pan 302 Found in 72.0041ms
[Macaron] Started GET /FakeOrgName/constellation/settings/collaboration for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation/settings/collaboration 404 Not Found in 6.0003ms
[Macaron] Started GET /img/404.png for <IP_ADDRESS>
[Macaron] Completed /img/404.png 200 OK in 0
[Macaron] Started GET /FakeOrgName/constellation/settings/collaboration for <IP_ADDRESS>
[Macaron] Completed /FakeOrgName/constellation/settings/collaboration 404 Not Found in 7.0004ms

Other information
Yes. The person I added is my manager. It might actually be a feature.
Best regards.

Thanks your feedback!
v0.6.1 does have this issue but it should be fixed with v0.6.3, try upgrade your Gogs and see if problem remains! :star:

Hello Unknwon. Cheers for that swift answering. :)
I've stopped the Windows service, copied all 0.6.3 files, and restarted. Checked the version number in the webapp : <IP_ADDRESS>2 Beta
The problem is still there : I can't see the old repo in the webapp.
I looked into the DB table to compare values between the new repo and the old (invisible) repo : they are exactly the same (apart of the obvious id, name and time of creation) ; in practice, one is visible, the other one is not.

Hmm... @mdolidon , well, by fixed I mean if you do same operation, owner will not lose repository access anymore, but for the ones already lost access, need some other ways to fix with.

Ah OK.
1- indeed, I replayed the scenario on our real organization, a test repo and a test user. It's gone all fine. Bug fixed apparently.
2- concerning our situation, is there a way for me to fix it by hand ? Do you know what may have become corrupted, at least a hint to where I may look ?

2- concerning our situation, is there a way for me to fix it by hand ? Do you know what may have become corrupted, at least a hint to where I may look ?

In the access table, add a record with correct user_id, repo_id and mode 4, should fix this problem.

Awesome, it's fixed. All access data for the old repo was gone. Thank you :+1:

Glad to hear it! :beer:


Update Databricks cluster options

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.

Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[ ] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

@microsoft-github-policy-service agree company="Microsoft"

From: microsoft-github-policy-service[bot] @.>
Sent: Tuesday, April 16, 2024 10:52 AM
To: Azure/azure-rest-api-specs @.>
Cc: Keshav Sharma @.>; Mention @.>
Subject: Re: [Azure/azure-rest-api-specs] Update Databricks cluster options (PR #28572)
@keshavsharma2024https://github.com/keshavsharma2024 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]
Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"
Contributor License Agreement
Contribution License Agreement
This Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
and conveys certain license rights to Microsoft Corporation and its affiliates (“Microsoft”) for Your
contributions to Microsoft open source projects. This Agreement is effective as of the latest signature
date below.

Definitions.
“Code” means the computer software code, whether in human-readable or machine-executable form,
that is delivered by You to Microsoft under this Agreement.
“Project” means any of the projects owned or managed by Microsoft and offered under a license
approved by the Open Source Initiative (www.opensource.orghttp://www.opensource.org/).
“Submit” is the act of uploading, submitting, transmitting, or distributing code or other content to any
Project, including but not limited to communication on electronic mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf of, the Project for the purpose of
discussing and improving that Project, but excluding communication that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material Submitted by You, including any
associated comments and documentation.
Your Submission. You must agree to the terms of this Agreement before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now or in the future (except as
described in Section 4 below), Submit to any Project.
Originality of Work. You represent that each of Your Submissions is entirely Your original work.
Should You wish to Submit materials that are not Your original work, You may Submit them separately
to the Project if You (a) retain all copyright and license information that was in the materials as You
received them, (b) in the description accompanying Your Submission, include the phrase “Submission
containing materials of a third party:” followed by the names of the third party and any licenses or other
restrictions of which You are aware, and (c) follow any other instructions in the Project’s written
guidelines concerning Submissions.
Your Employer. References to “employer” in this Agreement include Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your employer has intellectual
property rights in Your Submission by contract or applicable law, You must secure permission from Your
employer to make the Submission before signing this Agreement. In that case, the term “You” in this
Agreement will refer to You and the employer collectively. If You change employers in the future and
desire to Submit additional Submissions for the new employer, then You agree to sign a new Agreement
and secure permission from the new employer before Submitting those Submissions.
Licenses.

Copyright License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license in the
Submission to reproduce, prepare derivative works of, publicly display, publicly perform, and distribute
the Submission and such derivative works, and to sublicense any or all of the foregoing rights to third
parties.
Patent License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license under
Your patent claims that are necessarily infringed by the Submission or the combination of the
Submission with the Project to which it was Submitted to make, have made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the Project.
Other Rights Reserved. Each party reserves all rights not expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.

Representations and Warranties. You represent that You are legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your original work (except as You may
have disclosed under Section 3). You represent that You have secured permission from Your employer to
make the Submission in cases where Your Submission is made in the course of Your work for Your
employer or Your employer has intellectual property rights in Your Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You represent and warrant that You
have the necessary authority to bind the listed employer to the obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6, THE SUBMISSION PROVIDED UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO, ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
Notice to Microsoft. You agree to notify Microsoft in writing of any facts or circumstances of which
You later become aware that would make Your representations in this Agreement inaccurate in any
respect.
Information about Submissions. You agree that contributions to Projects and information about
contributions may be maintained indefinitely and disclosed publicly, including Your name and other
information that You submit with Your Submission.
Governing Law/Jurisdiction. This Agreement is governed by the laws of the State of Washington, and
the parties consent to exclusive jurisdiction and venue in the federal courts sitting in King County,
Washington, unless no federal subject matter jurisdiction exists, in which case the parties consent to
exclusive jurisdiction and venue in the Superior Court of King County, Washington. The parties waive all
defenses of lack of personal jurisdiction and forum non-conveniens.
Entire Agreement/Assignment. This Agreement is the entire agreement between the parties, and
supersedes any and all prior agreements, understandings or communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may be assigned by Microsoft.

—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-rest-api-specs/pull/28572#issuecomment-2059643756, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHFEOUQNZX35SLAYR2S2GTDY5VQONAVCNFSM6AAAAABFUT3UBCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJZGY2DGNZVGY.
You are receiving this because you were mentioned.Message ID: @.***>

And pls also update the synapase swagger.

@Ruoxuan @.***> updated the pr. Please review

From: Ruoxuan Wang @.>
Sent: Friday, April 19, 2024 12:20 AM
To: Azure/azure-rest-api-specs @.>
Cc: Keshav Sharma @.>; Mention @.>
Subject: Re: [Azure/azure-rest-api-specs] Update Databricks cluster options (PR #28572)
And pls also update the synapase swagger.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-rest-api-specs/pull/28572#issuecomment-2065919042, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHFEOUVFXR45BBNCFFSBAGTY6DAUNAVCNFSM6AAAAABFUT3UBCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRVHEYTSMBUGI.
You are receiving this because you were mentioned.Message ID: @.***>


No way to install android-sdk

Latest Mac OS X, brew.
Until today, the following worked:
brew install android-sdk
Today, it fails and recommends this:
brew cask install Caskroom/cask/android-sdk  which also doesn't work:
Geoffs-MacBook-Pro:stardust-backend giro$ brew cask install Caskroom/cask/android-sdk
Error: Tap caskroom/cask already tapped.

Follow the instructions here:
  https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/pre_bug_report.md

If this doesn’t fix the problem, please report this bug:
  https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Homebrew/Library/Homebrew/tap.rb:196:in `install'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source/untapped_qualified.rb:9:in `me?'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source.rb:28:in `block in for_query'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source.rb:26:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source.rb:26:in `find'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source.rb:26:in `for_query'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc.rb:56:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:21:in `block in install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:10:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:101:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:144:in `process'
/usr/local/Homebrew/Library/Homebrew/cmd/cask.rb:8:in `cask'
/usr/local/Homebrew/Library/Homebrew/brew.rb:91:in `<main>'

Please advise, thanks.

https://github.com/caskroom/homebrew-cask/pull/31598


major: Removed deprecated props

BREAKING CHANGES
Removed the following deprecated props:

titleIOS
customTitleContainerIOS
onHideAfterConfirm
customDatePickerIOS

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: