Add notification types for when something is taken/put/both into or out of chests.

Instead of just "Accessed", have 4 different types of reports, depending:
"Opened" means opened but didn't do anything.
"Added Items" means opened and put stuff in only.
"Removed Items" means opened and took stuff out only.
"Exchanged Items" means opened and did a mixture of taking and adding stuff.
It doesn't spam the logs or take up any more disk space. Only one of those is still ever logged for each opening of the block, it's just that it's less annoyingly vague.
Opening an unlocked chest and then walking away is not what most people would consider a crime.  Thus, the plugin would be much more useful if it distinguished. Especially when there might be 7 people who opened the chest in between checking the snitch. And then something is missing...

http://imgur.com/LPPOiMK
https://github.com/Civcraft/JukeAlert/commit/cb7fe0e42c75dccaba3060433aea374edbded398
How Juke Alert is set up you can't really set custom messages.

It would just be additional values in the enum for opened/added/removed/exchanged. Even the item ID of what was added/removed could be placed into the material field instead of the container block type, since it's known that a container at location X is being accessed.

So would it be four extra lines per exchange in the log?

Yes, although I missed "Only one of those is still ever logged for each opening of the block". With the current implementation there is no log consolidation so it would create log spam. Probably not worth it with the Exchange notification, but GavJenks can argue for it.

I am thinking I didn't explain clearly enough.
It was poor word choice with "exchanged" -- I did not mean "used item exchange," and you seem to have posted the same screenshot for here and the itemexchange logging suggestion I made. Whichever one is actually implemented no, thanks! But two different concepts. One is logging itemexchanges. The other is an indication that items were both placed in and removed from a chest while it was opened by a player. Perhaps "Swapped" or whatever, I don't know. The idea is to therefore cover all logical possibilities, each with just ONE log entry, so that the spam is not any higher than it is right now:
Did not open a chest -- isn't logged
Opened a chest but did nothing with the contents -- "opened"
Opened a chest and put stuff in it but did not remove stuff -- "removed item(s)"
Opened a chest and removed stuff but didn't put anything new in -- "added item(s)"
Opened a chest and within that one interaction, added things and removed other things -- "swapped item(s)" or whatever terminology.
Every chest access event therefore generates exactly one log entry, just like now, but it tells you whether they actually stole things, which is what people actually care about. The way it stands now, i could open a chest, PUT diamonds in it, and then get pearled for being a thief...


Control the column on upsert

Hi !
I have an entity where I generate the UUID on application side (can't use database generation), where the email is the unique field. I try to do the upsert like this:
const customer = entityManager.create(Customer, request.body);
const managedCustomer = await entityManager.upsert(customer);

However MikroORM generate this SQL:
insert into customers (email, first_name, id) values<EMAIL_ADDRESS>'foo', '71b804dc-7c92-4287-aff8-6e2c536832be') on conflict (id) do update set email = excluded.email, first_name = excluded.first_name returning last_name;

However, I would like MikroORM to check the conflict on the email, not the ID, so generate this query:
insert into customers (email, first_name, id) values<EMAIL_ADDRESS>'foo', '71b804dc-7c92-4287-aff8-6e2c536832be') on conflict (email) do update set email = excluded.email, first_name = excluded.first_name returning last_name;

Is there a way to override the column that MikroORM is using? Maybe somehing like that:
await entityManager.upsert(customer, {conflictColumn: 'email'});

Thanks!

What you can do now is to pass in DTO without the id and let the database generate it.
const customer = await em.upsert(Customer, { email: '...' });

Note that in your code, customer === managedCustomer, you won't get a new entity instance from that call.

I would like to generate uuidv7, which is not yet supported natively (I generate it server side).

I will add 4 new options:

onConflictFields?: (keyof T)[]; (to control the conflict clause)
onConflictAction?: 'ignore' | 'merge'; (used ignore and merge as that is how the QB methods are called)
onConflictMergeFields?: (keyof T)[]; (to control the merge clause)
onConflictExcludeFields?: (keyof T)[]; (to omit fields from the merge clause)

Which should make the upsert queries completely configurable. Already have a WIP that handles the queries correctly, the tricky part will be handling the entity mapping (both to existing entity instances and to the POJO signatures) and reloading the missing data (for mysql that does not have a returning clause).
This should cover also #4325, which is about being able to omit a field from the merge data.

Thanks a lot, this looks like a perfect solution to this problem! Great job!


Unable to Create Generic Constructor of T that extends a concrete Type

Hi. Today I faced an issue when I wanted to create a parent data class for all my data models in dart.
What I am trying to do,
abstract class DataClass {
// throwing an error since abstract classes cannot be instantiated,
// but I need my sub classes to implement this factory constructor
 factory DataClass.fromMap(Map<String, dynamic> map) => throw UnimplementedError();
}

class ModelClass<T extends DataClass> extends DataClass {
final List<T> values;
factory ModelClass.fromMap(Map<String, dynamic> json) => ModelClass(
       // Cannot call a constructor invokation on T
        values: json["results"] == null ? null : List<T>.from(json["values"].map((x) => T.fromMap(x))),
      );
}

I dont know if there is another way of achieving a similar outcome. But implementing something like this would drastically reduce a lot of Api Response duplication, where some parameters are always constant.

@leafpetersen

Note that the ability to invoke static methods and/or constructors of a class based on the value of a type variable was discussed in https://github.com/dart-lang/sdk/issues/10667 and https://github.com/dart-lang/language/issues/356.

Closing in favor of https://github.com/dart-lang/language/issues/356.

Let me just mention the standard workaround for being able to call a constructor of a type variable: Introduce a formal parameter which is a factory function (which would typically be the torn-off constructor, but could also be any other function with the right type), and then call that factory function rather than trying to call a constructor on a type variable:
abstract class DataClass {
  DataClass.fromMap(Map<String, dynamic> map);
}

class ModelClass<T extends DataClass> extends DataClass {
  final List<T>? values;
  ModelClass.fromMap(
      Map<String, dynamic> json, T Function(Map<String, dynamic>) tFactory)
      : values = json["results"] == null ? null : <T>[],
        super.fromMap(json) {
    var values = this.values;
    if (values != null) {
      Iterable<Map<String, dynamic>> jsonValues = json["values"]!;
      for (var x in jsonValues) {
        values.add(tFactory(x));
      }
    }
  }
}


2.1

Priority:

[x] Allow compiling directly into plugin
[x] Fix TitleAPI for 1.11
[x] Allow use on craftbukkit

Hi, can you do a system for the timer for her games ? (more easy) without fix the speed of the timer ?

Mind saying what that would be used for? Or have some examples for what you
want to use that for?
On Wed, Nov 23, 2016 at 12:39 PM, x9nico<EMAIL_ADDRESS>wrote:

Hi, can you do a system for the timer for her games ? (more easy) without
fix the speed of the timer ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ben12345rocks/AdvancedCore/issues/39#issuecomment-262583194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQSIN-BydXYQkyHiPrvuTlD43QkVsL2jks5rBHppgaJpZM4K3xMx
.

For create a minigames

Any code examples?
On Nov 23, 2016 1:08 PM, "x9nico"<EMAIL_ADDRESS>wrote:

For create a minigames
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ben12345rocks/AdvancedCore/issues/39#issuecomment-262590105,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQSINwrUEkDFvN6RiBOPiIcPtj5LFNkQks5rBIENgaJpZM4K3xMx
.

if(Bukkit.getOnlinePlayers().size() >= 2){
task = Bukkit.getScheduler().scheduleSyncRepeatingTask(Bukkit.getPluginManager().getPlugin("FauGamesBabyKillMe"), new Runnable() {
			@Override
			public void run() {
				timer--;

				if(timer == 120 || timer == 60 || timer == 30 || timer == 15 || timer == 10 || timer <= 5 && !(timer == 0)){
					Bukkit.broadcastMessage(ChatUtils.getPrefix() + FaruGamesAPI.getLanguageSystem().returnStringPlayerLanguage(p.getUniqueId(), "§6La partie démarre dans " + timer + " secondes", "§6The game start in "+timer+"seconds"));
				}else if(timer == 1){
					Bukkit.broadcastMessage(ChatUtils.getPrefix() + FaruGamesAPI.getLanguageSystem().returnStringPlayerLanguage(p.getUniqueId(), "§6La partie démarre dans "+timer+" seconde.", "§6The game start in"+timer+"second."));
				}else if(timer == 0){
					Bukkit.broadcastMessage(ChatUtils.getPrefix() + FaruGamesAPI.getLanguageSystem().returnStringPlayerLanguage(p.getUniqueId(), "§6La partie a commencée ! bonne chance !" , "§6The game has started ! good luck !"));
					Bukkit.getScheduler().cancelTask(task);
					GameState.setState(GameState.GAME);
				}

			}
		}, 20, 20);
	}

}

}

So you would like to make that simplified?
On Dec 5, 2016 2:53 PM, "x9nico"<EMAIL_ADDRESS>wrote:

if(Bukkit.getOnlinePlayers().size() >= 2){
task = Bukkit.getScheduler().scheduleSyncRepeatingTask(
Bukkit.getPluginManager().getPlugin("FauGamesBabyKillMe"), new Runnable()
{
  	@Override
  	public void run() {
  		timer--;

  		if(timer == 120 || timer == 60 || timer == 30 || timer == 15 || timer == 10 || timer <= 5 && !(timer == 0)){
  			Bukkit.broadcastMessage(ChatUtils.getPrefix() + FaruGamesAPI.getLanguageSystem().returnStringPlayerLanguage(p.getUniqueId(), "§6La partie démarre dans " + timer + " secondes", "§6The game start in "+timer+"seconds"));
  		}else if(timer == 1){
  			Bukkit.broadcastMessage(ChatUtils.getPrefix() + FaruGamesAPI.getLanguageSystem().returnStringPlayerLanguage(p.getUniqueId(), "§6La partie démarre dans "+timer+" seconde.", "§6The game start in"+timer+"second."));
  		}else if(timer == 0){
  			Bukkit.broadcastMessage(ChatUtils.getPrefix() + FaruGamesAPI.getLanguageSystem().returnStringPlayerLanguage(p.getUniqueId(), "§6La partie a commencée ! bonne chance !" , "§6The game has started ! good luck !"));
  			Bukkit.getScheduler().cancelTask(task);
  			GameState.setState(GameState.GAME);
  		}

  	}
  }, 20, 20);

}
}
}
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Ben12345rocks/AdvancedCore/issues/39#issuecomment-264958481,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQSIN2AnhF9lz79EmKQ3uR3FMjTTExlrks5rFGvEgaJpZM4K3xMx
.

Yes

I'll see what I can do but that won't be my priority.
On Dec 9, 2016 3:32 PM, "x9nico"<EMAIL_ADDRESS>wrote:

Yes
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Ben12345rocks/AdvancedCore/issues/39#issuecomment-266115839,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQSIN7YwgumhT6qXTMgP1CUm8ZBEaW7oks5rGbrigaJpZM4K3xMx
.

ok


StokesSpectralCube access

To access the components of a StokesSpectralCube, you presently need to access a private attribute, for example:
cube_I = cube._stokes_data['I']

That's not good!  We should probably implement at least a public access like:
cube_I = cube.stokes_data['I']

and better still,
cube_I = cube['I']

@Kitchi, what do you think?
A PR would be welcome!

I've got a PR ready with the first suggestion, but I like the second one a lot more. Both @preshanth and I can work on implementing that for the Stokes cubes.

closed by #751


IPMI user test cases - Suggestion

ipmi/test_ipmi_user.robot

Delete All Non Root IPMI User

we are doing this to clean up all users before testing.. however, we could bit better by just letting the test adapt to determine the number of users, N, and then make the change and look for N+1.

Merged


Zigbee2Mqtt not reading state from Feibit 1/2/3/Dimmer gang switch

zigbee2mqtt add-on version (if edge, please report commit hash):
version 1.16.1, commit has 6b32f30
Operating environment (HassOS, Virtual Machine, Device/platform running Home Assistant):
HassOS 5.3, Virtual Machine - false Home Assistant 0.118.4, Raspberry AARCH 64

Description of problem:
Home Assistant ZigbeeMQTT integration does not fetch device state for the following devices.
FB56+ZSC05HG1.0 - Dimmer Switch
FB56+ZSW1GKJ2.5 - 1 Gang Switch
FB56+ZSW1HKJ2.5 - 2 Gang Switch
FB56+ZSW1IKJ2.5 - 3 Gang Switch
FB56-ZCW11HG1.2 - Downlight
FB56-ZCW08KU1.1 - Bulb
FB56-ZCW11HG1.2 - Bulb
Your entire configuration from the frontend (with sensitive fields redacted):
data_path: /share/zigbee2mqtt
external_converters: []
devices: devices.yaml
groups: groups.yaml
homeassistant: true
permit_join: false
mqtt:
  base_topic: zigbee2mqtt
  server: 'mqtt://core-mosquitto'
  user: mqtt
  password: password
serial:
  port: /dev/ttyAMA0
  adapter: deconz
advanced:
  log_level: info
  pan_id: 6754
  channel: 11
  network_key:
    - 1
    - 3
    - 5
    - 7
    - 9
    - 11
    - 13
    - 15
    - 0
    - 2
    - 4
    - 6
    - 8
    - 10
    - 12
    - 13
  availability_blocklist: []
  availability_passlist: []
device_options: {}
blocklist: []
passlist: []
queue: {}
frontend:
  port: 8099
experimental:
  new_api: true
socat:
  enabled: false
  master: 'pty,raw,echo=0,link=/dev/ttyZ2M,mode=777'
  slave: 'tcp-listen:8485,keepalive,nodelay,reuseaddr,keepidle=1,keepintvl=1,keepcnt=5'
  restartdelay: 1
  initialdelay: 1
  options: '-d -d'
  log: false
zigbee_shepherd_devices: true

Your logs from Home Assistant

<EMAIL_ADDRESS>start /zigbee2mqtt-1.16.1
node index.js
/zigbee2mqtt-1.16.1/node_modules/zigbee-herdsman-converters/devices.js:265
exposes.enum('effect', exposes.access.SET, ['blink', 'breathe', 'okay', 'channel_change', 'finish_effect', 'stop_effect']),
^
TypeError: Cannot read property 'SET' of undefined
at Object. (/zigbee2mqtt-1.16.1/node_modules/zigbee-herdsman-converters/devices.js:265:51)
at Module._compile (internal/modules/cjs/loader.js:1137:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
at Module.load (internal/modules/cjs/loader.js:985:32)
at Function.Module._load (internal/modules/cjs/loader.js:878:14)
at Module.require (internal/modules/cjs/loader.js:1025:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object. (/zigbee2mqtt-1.16.1/node_modules/zigbee-herdsman-converters/index.js:3:17)
at Module._compile (internal/modules/cjs/loader.js:1137:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
npm
ERR! code ELIFECYCLE
npm ERR! errno 1
npm
ERR<EMAIL_ADDRESS>start: node index.js
npm ERR!
Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!

Dear,
I have the some issue.
I have opened a issue: check it:
Koenkk/zigbee2mqtt#4461

I will close this as the repo here is just for the add-on. Please continue to discuss this on Koenkk/zigbee2mqtt#4461


Missing files

re... <% Server.Execute("../core/include/application.asp") %> on default.asp
/core/include/application.asp is missing from the download ZIP.

I did a search for <% Server.Execute("../core/include/application.asp") %> and found one file that does contain an incorrect relative path to the file, which is the core/error_docs/400.asp file. But not the default.asp.


[AutoRelease] t2-networkcloud-2023-04-24-00634(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4074
Live test success

issue link:https://github.com/Azure/sdk-release-request/issues/4074

/azp run prepare-pipelines


Add boot parameters transformer

Add a transformer to allow for adding additional boot parameters to the kernel commandline.

@johnsongeorge-w @kamalca is it needed? if yes, feel free to create or reopen it when it is ready for review


Store token at TrackingEventProcessor shutdown if in store-before-batch mode

This fixes the problem, but causes testMergeSegmentWithDifferentProcessingGroupsAndSequencingPolicies() to fail because, as far as I can tell, the token that's stored by the new code here overwrites the merged token that's created by the call to mergeSegment(). So that seems like a legitimate problem. Not sure if the fix is to do something completely different to solve the problem I'm targeting here, or to add logic to detect merges when writing the token at shutdown.

This change turns out to not make much sense; closing the PR.


Allow searching for players with IGN shorter than 3 chars (examples: G, XH)

https://namemc.com/profile/g.2
https://namemc.com/profile/XH.1

How did they get a username less than 3 characters?
https://help.minecraft.net/hc/en-us/articles/360034636712-Minecraft-Usernames

It was possible during alpha iirc, and after with a glitch


Pierce County, WA Parcel Data

TERMS OF USE here: http://yakima.co.pierce.wa.us/GeoDataExpress/PierceCountyGISDataTermsofUse.pdf
@nvkelso can you look over the terms above to make sure the data is okay to use?  I believe it is, just want another set of eyes to check it out before a PR is created.  Thanks!
the address field is: Site_Address; physical location address of the property. Site address may be different than the mailing address. When a site address contains "XXX" it means the house number is unverified.
ArcGIS open data portal site: http://gisdata.piercecowa.opendata.arcgis.com/datasets/0bddcec060764ce8a6b6c07a60af170c_0
Rest: https://services2.arcgis.com/1UvBaQ5y1ubjUPmd/arcgis/rest/services/Tax_Parcels/FeatureServer/0

Terms look fine to me.

Thank you sir!  Although not in these terms, I think every city/ county (in WA ) is concerned with lists of people's names being used for commercial purposes.  When I tell them we want nothing to do with it, there are a lot more easy going...

**as per the county creating the actual address point layer from Linda Gerrell -<PHONE_NUMBER>
no timeline, fighting to get it in 2017 budget - and once it is created, cannot say whether or not it will be open data

Only Site_Address is valid, the City_State and Zipcode fields are for mailing address.  Rather than not skipping house number or using 0, the source adds XXX, eg - XXX KEY PENINSULA HWY N.


Feat: vela auth gen-kubeconfig

Signed-off-by: Somefive<EMAIL_ADDRESS>Description of your changes

vela auth gen-kubeconfig generate kubeconfig for users with given User/Groups or ServiceAccount.
For User/Groups, the command will generate a new private key, create a CertificateSigningRequest, approve itself and generate kubeconfig with the signed X.509 certificate.
For ServiceAccount, the command will find the serviceaccount secret and generate kubeconfig with the token inside the secret.
Example usage:
  # Generate a kubeconfig with provided user
  vela auth gen-kubeconfig --user new-user

  # Generate a kubeconfig with provided user and group
  vela auth gen-kubeconfig --user new-user --group kubevela:developer

  # Generate a kubeconfig with provided user and groups
  vela auth gen-kubeconfig --user new-user --group kubevela:developer --group my-org:my-team

  # Generate a kubeconfig with provided serviceaccount
  vela auth gen-kubeconfig --serviceaccount default -n demo

I have:

[x] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

Other commands that will be developed following this PR.

vela auth create serviceaccount <ServiceAccount> --namespace <Namespace> --cluster <Cluster> Create ServiceAccount.
vela auth create role/clusterrole <Name> --verb <Verb> --resource <Resource> --namespace <Namespace> --cluster <Cluster> Create Role/ClusterRole.
vela auth create rolebinding/clusterrolebinding <Name> --role <RoleName> --clusterRole <ClusterRoleName> --user <User> --group <Group> --serviceaccount <ServiceAccount> --namespace <Namespace> --cluster <Cluster> Create RoleBinding/ClusterRoleBinding.
vela auth show-privileges --user <User> --group <Group> --serviceaccount <ServiceAccount> --namespace <Namespace> --cluster <Cluster>

vela create is not good enough to be occupied in this usage

vela auth？

vela auth？

vela auth kubeconfig is confusing. vela auth create kubeconfig is a bit long. Besides, this also affects other Role/RoleBinding/ClusterRole/ClusterRoleBinding commands.  However, those are all resources and kubeconfig here is a bit different. So vela create might be a bit improper.

@Somefive Can you provide a scenario, of what commands users should use?

@Somefive Can you provide a scenario, of what commands users should use?

For example, if you are the admin, you want to give demo namespace's permission to a user named Alice, and Alice want to access the demo namespace with vela-cli, like create an application or check application running status.
The first step admin needs to do, is to give her a kubeconfig file. This file  cannot be a copy of the admin's kubeconfig, since that will cause Alice to have very large permissions, like deleting other people's namespace. With this PR, admin can run vela create kubeconfig --user=alice first, and give Alice the generated kubeconfig. Then the admin can run vela create rolebinding -cluster-role=edit --user=alice --cluster=local,cluster1,cluster2 --namespace=demo, this will give Alice the privileges to manipulate resources in demo namespace in three clusters.

vela auth？

@wonderflow vela auth create kubeconfig although a bit long but clear enough? I think this one is ok to me.

vela auth？

@wonderflow vela auth create kubeconfig although a bit long but clear enough? I think this one is ok to me.

What about vela auth create --kubeconfig ? There can be other type of authorization

vela auth？

@wonderflow vela auth create kubeconfig although a bit long but clear enough? I think this one is ok to me.

What about vela auth create --kubeconfig ? There can be other type of authorization.
vela auth gen --kubeconfig

How about vela auth gen-kubeconfig? Otherwise we would need to have vela auth gen --kubeconfig --user --group seems a bit weird?

vela auth gen-kubeconfig

works for me

How will the generated kubeconfig be used? By joining as a cluster with limited permissions?

How will the generated kubeconfig be used? By joining as a cluster with limited permissions?

No, the kubeconfig generation here is not related with joining clusters. It works for the control plane. For example, you are the admin of the control plane, you want to give a developer the permission to run applications in demo namespace. This command can create a kubeconfig for that developer, since giving the admin's kubeconfig to that developer will give the developer overlarge permissions.


Intersection does not have a shape

When using a union/intersection type, the resulting object does not have a shape.
export const GetPulseMeterCurrentUsageParams = z
    .object({
        duration: z.number().optional(),
    })
    .and(
        z
            .object({ iotDeviceID: z.string() })
            .or(z.object({ pulseMeterID: z.string() }))
    );

GetPulseMeterCurrentUsageParams.shape // undefined

Hello, I believe this is related to this issue: https://github.com/colinhacks/zod/issues/1768

Is this what you are looking for?
const shared = z.object( {
    duration: z.number().optional(),
} )

const getPulseMeterCurrentUsageParams = z.union( [
    shared.extend( { iotDeviceID: z.string() } ),
    shared.extend( { pulseMeterID: z.string() } ),
] )

getPulseMeterCurrentUsageParams._def.options[ 0 ].shape // works
getPulseMeterCurrentUsageParams._def.options[ 1 ].shape // works

If you found my answer satisfactory, please consider supporting me. Even a small amount is greatly appreciated. Thanks friend! 🙏
https://github.com/sponsors/JacobWeisenburger


Investigate alternative methods for communicating with Storm

Because all communication with Storm currently happens over stdin and stdout, which cannot reliably be read/written in a non-blocking fashion in Python, we should come up with ideas for how else we might communicate with Storm. The simplest method would be to use sockets.
We would need a custom subclass of ShellProcess so that we could override ShellProcess.launch to pass socket-based input and output streams to the serializer class. We would then need to subclass ShellBolt so that we could modify this line in ShellBolt.prepare to use the new subclass.
There are probably other approaches we could take too, so let's hash them out here as they come up.

I think that's a good start. I found that both Spark and Flink have python API which leverages socket instead of stdin and stdout. I don't know why Storm relies on stdin and stdout, but changing ShellProcess to leverage socket is clearly breaking backward compatibility, so Storm may not be able to go with this direction.
I hope that Storm community supports python API officially (because Python is the language data scientists are relatively familiar with), which would not be a part of multilang but a part of streams API in 2.0.0. From the API, we can ignore backward compatiblity and implement from the scratch, so we could concentrate on optimization with Python. I have been thinking about it, but I'm not an expert on Python (was an one of users though) and haven't have time to give it a shot.
Not sure my idea would make a conflict with Streamparse project, so would like to hear opinions. To be honest, I think Storm's approach - defining interfaces and let ecosystem implementing - is not working well. There's no better way than supporting features officially.

@HeartSaVioR thanks so much for your input! My hope was that if we can show this is valuable, we could maybe make a PR for Storm that would allow people to choose whether or not to communicate via stdin/stdout or sockets. That way it wouldn't break backward compatibility.
I'm not aware of the "streams API in 2.0.0" yet. I've fallen a bit behind on the plans for 2.0. Is there anything you can point me to explaining what the plans are there?

@dan-blanchard
For streams API, you can see the document in master branch (since 2.0.0 is not out yet)
https://github.com/apache/storm/blob/master/docs/Stream-API.md
Streams API supports typed, declarative API which other streaming frameworks provide by default. (Storm has been providing core API which is non-typed, compositional.)
We already have Trident API for a long time, but that's based on micro-batch and still non-typed so introducing streams API should help users to compose their business logic much easier.
We didn't document it, but I answered question on what's changed in Storm 2.0.0:
http://mail-archives.apache.org/mod_mbox/storm-user/201712.mbox/<CAF5108j0iGxH7QdZKz81uxCTykQZZfNwpagEV_hYYhT8xmxYUg%40mail.gmail.com>
I would really want to get it really soon, say, by 2018 1Q, because it is actually late 1+year then expected.

@dan-blanchard

My hope was that if we can show this is valuable, we could maybe make a PR for Storm that would allow people to choose whether or not to communicate via stdin/stdout or sockets. That way it wouldn't break backward compatibility.

I guess we can make communication layer in multilang being plug-in and provide both stdin/out and socket, and finally set default to use stdin/out.


automatically log in the user in the background

Hello,
We, for our project, currently have the following workflow:
Person signs into our login service via the app. The app then authenticates with the adfs but currently by redirect and we do somehow have to handle that redirect and make the user basically sign in again. Our service then registers the successfully authenticated user after adfs redirect.
Because this sounds ridiculous, we want to ease this process and let our service do the login for the user.
Is it possible to attach the needed parameters for the login, so that it automatically is filled out and completed or do we have to crawl the website and pretend to be the user?
Does anyone have already experience with that?
Greetings
Chris

Why the app is not delegating the authentication directly on ADFS?
Normal login + Log via ADFS

User credentials should not be sent from app to ADFS. This is not how SAML protocol works.


Machine unloading? Also a client crash.

I have noticed that the multiblock machines (precision assembler, saw for silicon, the silicon vat, rolling machine and lathe) all keep forgetting they're on.  When I queue up a job (using AE2 connected to the input bus of its respective machine via an interface) the machine does nothing most of the time until I right click on the primary block for that respective machine before it becomes active.  I do not recall having this issue at all in previous versions of the mod.  I also have noticed that, while staying in the area, the machine sometimes becomes inactive again, after a period of time, if it has nothing to do and I will have to repeat the steps mentioned above.
I was trying to make some tin plates and noticed the rolling machine was not working.  When I right click on the rolling machine's main block, my client crashed with the attached logfile.
crash-2017-03-07_00.46.06-client.txt
I hope this helps!  Let me know if there's anything else you need.

I think I fixed an issue with AE and AR's machines in last night's release (1.0.13) (issue #435 ) , can you give that build a try and see if it's still broken there?
As for the machine unloading, is there any pattern or way to make them stop working on demand?  Also can you verify if they've stopped working on the server as well.  Something like requesting rods or plates through AE should be enough to test that.

I have tried to duplicate the issue and it appears that the changes have fixed the problem.  I have queued up multiple jobs from the various machines and everything was crafted successfully.  If I have any further issues I'll let you know!

Alright, good to hear!

Not sure what's going on but I'm seeing this happen again.  The machines are unloading.  I have chunk loaders in the area where these are located.  Should I open a new ticket?


"swagger diff" does not work with spec that has a recursive definition

Problem statement
When Swagger doc has a recursive definition, the  swagger diff  command  seem to enter an  infinite loop.
Swagger specification
{
  "swagger": "2.0",
  "info": {
    "description": "",
    "version": "1.0.0",
    "title": "test"
  },
  "host": "test.example.com",
  "basePath": "/",
  "schemes": [
    "https"
  ],
  "paths": {
    "/categories": {
      "get": {
        "tags": [
          "categories"
        ],
        "summary": "get all categories",
        "description": "",
        "produces": [
          "application/xml",
          "application/json"
        ],
        "responses": {
          "200": {
            "description": "successful operation",
            "schema": {
              "$ref": "#/definitions/Category"
            }
          }
        }
      }
    }
  },
  "definitions": {
    "Category": {
      "type": "object",
      "properties": {
        "id": {
          "type": "integer",
          "format": "int64"
        },
        "name": {
          "type": "string"
        },
        "subCategories": {
          "$ref": "#/definitions/Category"
        }
      },
      "xml": {
        "name": "Category"
      }
    }
  }

}

Steps to reproduce

save the spec above to old.json
save the spec above to new.json
run command swagger old.json new.json

expect the result is no change, but the command never completed
Environment
swagger version: v0.29.0
go version: 1.18.1
OS:   Arch Linux

I'v been able to test with previous releases to determine if and when this was fixed.
This issue has been fixed as a side-effect of PR #2832 , released in v0.30.3


PkgEval: julia binary may be missing

The tests here run julia ..., which is fragile: julia might not be on PATH, and the binary may be called julia-debug, so it's better to use Base.julia_cmd(). This causes a PkgEval failure when running in debug mode.
Base.IOError("could not spawn `julia --compile=min scripts/mpihello.jl`: no such file or directory (ENOENT)", -2)
StaticMPI.jl: Test Failed at /home/pkgeval/.julia/packages/StaticMPI/EzNpR/test/runtests.jl:20
  Expression: status isa Base.Process
   Evaluated: -1 isa Base.Process
Stacktrace:
 [1] macro expansion
   @ /opt/julia/share/julia/stdlib/v1.9/Test/src/Test.jl:477 [inlined]
 [2] macro expansion
   @ ~/.julia/packages/StaticMPI/EzNpR/test/runtests.jl:20 [inlined]
 [3] macro expansion
   @ /opt/julia/share/julia/stdlib/v1.9/Test/src/Test.jl:1492 [inlined]
 [4] top-level scope
   @ ~/.julia/packages/StaticMPI/EzNpR/test/runtests.jl:7

Ah ok, will Base.julia_cmd() also be sure not to propagate the --check-bounds=yes flag from the test environment to the specified command?

It looks from the docstring like it will, in which case it may not work here

Then you can peek into the Cmd, or use joinpath(Sys.BINDIR, Base.julia_exename()).

It's also recommended to use --startup-file=no, to avoid leaking anything into the test env.

That sounds workable


Annotations doc

Hello
Is there any documentation for the annotations that you are using ?
(I could find this https://github.com/TelluIoT/ThingML/blob/master/Documentrospection/Annotation.html but it is incomplete)
Thanks

Unfortunately, they are not very well documented, as they are evaluated at run-time when you compile your ThingML model to a target.
As you have already seen, there is the Documentrospection code that reads all the Java-source and tries to find places where we use specific annotations. But as you have noticed, it was last updated in October 2016, so there might be some changes.
If you are compiling ThingML from source, you can run the Documentrospection yourself to generate it again from the current Java source code.
Is there any specific ones that you are missing?

Actually several ones, I thought that it could be better to have this doc especially when it comes to the platform specific ones as usually people will not be experts of the platform. It may avoid to dive into the code trying to figure it out


proposal: x/crypto/argon2: support secret value and associated data

Hello,
It seems though the exported functions of x/crypto/argon2 only permits for customised parameters relating to generation and obviously the salt and password for hashing. Being able to use a secret key would in theory increase security.

CC @FiloSottile

I assume you are referring to the inputs that the draft calls "Secret value K" and "Associated data X". The package API is already a handful as it is, I am not inclined to add support for them, unless there are protocols that require their use.

I just thought it was useful to be able to use the entirety of argon2, but I agree the package API does require many parameters already and it doesn’t seem as though any protocols need to use it.

Adding to proposal minutes.

There doesn't seem to be much interest in this feature. Based on the discussion above, this seems like a likely decline.

No change in consensus, so declined.

Hi :wave: @rsc @FiloSottile
Not sure if my +1 here makes any difference, but I'd love to see the secret (K) value exposed in x/crypto/argon2.
The motivation here is that other argon2 implementations such as Ruby/C, allow users to pass K, and the fact that Go's implementation doesn't creates friction for developers who are migrating their code from said languages to Go.
For example, I am tasked with re-writing some Ruby code to Go where in the transition period, the Go code will generate the argon2 hashes and the Ruby code should continue to be able to verify it (as both Go and Ruby will share the same secret in Vault) until the Ruby code is fully swapped out. Of course, this is not possible unless x/crypto/argon2 allowed me to pass K.
From what I can see, the secret K implementation actually already exists in x/crypto/argon2 and is tested as well, it just needs to be exposed?
Thank you!

Hi :wave: @rsc @FiloSottile
Not sure if my +1 here makes any difference, but I'd love to see the secret (K) value exposed in x/crypto/argon2.
The motivation here is that other argon2 implementations such as Ruby/C, allow users to pass K, and the fact that Go's implementation doesn't creates friction for developers who are migrating their code from said languages to Go.
For example, I am tasked with re-writing some Ruby code to Go where in the transition period, the Go code will generate the argon2 hashes and the Ruby code should continue to be able to verify it (as both Go and Ruby will share the same secret in Vault) until the Ruby code is fully swapped out. Of course, this is not possible unless x/crypto/argon2 allowed me to pass K.
From what I can see, the secret K implementation actually already exists in x/crypto/argon2 and is tested as well, it just needs to be exposed?
Thank you!

Re-opening the issue for visibility. But please feel free to close again.

Re-opening the issue for visibility. But please feel free to close again.

To avoid a breaking change, one could go down the route of argon2.IDKeyWithSecret.

@rsc @FiloSottile
Deriving keys without secret allow attackers to crack passwords offline in case of database breach. Because salt and other parameters are stored in leaked database, and no secret is involved. Using  secret and not storing it in database can add extra layer of security.
x/crypto/argon2 already implements this and all tests are passed. Why not expose secret parameter?
Also it is needed for compatibility with other libraries and existing projects.
Thanks.

Closing as I see the decision has been made to not include this.

This is a bad decision. These parameters are required to be able to implement NIST SP 800-108r1 and NIST SP 800-56C REV. 2. Also good password storage practise (https://www.ietf.org/archive/id/draft-ietf-kitten-password-storage-04.html) is to use a pepper, which is the purpose of the "secret"-parameter. I wanted to use a memory safe language for this and chose go initially because https://go.googlesource.com/proposal/+/master/design/cryptography-principles.md promised that I could. But it seems now that "safe" and "modern" are just pretty words. I guess my python code won't get ported to go at this time.

I guess my python code won't get ported to go at this time.

While I agree with you that this decision is unfortunate, the less dramatic solution is to fork the Argon2 library out of /x/crypto.  Speaking as someone who has had to do this (for compatibility with Argon2 hashes shared with a non-Go library), exposing those parameters is a pretty trivial modification, and shouldn't affect backporting any future changes from upstream.
Now that this decision is final, hopefully someone in the community will step up and maintain such a fork, similar to what happened with Proton Mail and /x/crypto/openpgp.  I realistically don't have the bandwidth in the short-term, or else I would offer.

The bigger issue is that obviously Go developers can't be bothered to write best practise, standards compliant products. What other shortcuts and limitations is hidden out there? It seems to apply to both the developers maintaing the language standard module as well as others. Knowing this - can we trust Google products (assuming they are built with Go and obviously does not use a pepper) to be safe? if there is an argon2 module in the library then issues needs to be fixed where they are. If forking is the natural reaction, what other parts of the standard library should not be used? Can this be clearly marked in the documentation?

@peter021 Please be mindful of the Go Community Code of Conduct.  Please be charitable and respectful.  Thanks.

You're right, I removed the sweeping accusations and generalisations. Sorry about that, got a little carried away. Some of the remaining questions may be important to consider though. They don't need answers, and they represent a single point of view among many.  But statistics state that i one person voice an oppinion there are generally another 24 people holding the same oppinion. I write this not to downvalue the work that you guys do, I write this to try to convince people who are better than me to fix something important that is broken. I'm trying to help you guys improve the language and any resulting product that is built using it.


What am I doing wrong?

I'm trying to get rid of options I will never use in the taskbar's right-click menu (not Win+X), but they are still there. There are no errors in the log file.
Default menu (without shell)

After registering:

Code:
shell
{
	default
	{
		theme
		{
			name="modern"
			image.align=1
			layout.popup=auto
		}
		//exclude.where = !process.is_explorer
	}

	static
	{
		item(where='taskbar' find='Cortana' vis='Disable')
		item(find='Task View' vis='Disable')
		item(find='People' vis='Disable')
		item(find='Ink' vis='Disable')
		item(find='touch' vis='Disable')
	}

	dynamic
	{

	}
}

You must pass a boolean to where property
In the next update, the type property has been added to the static items so that the items can be filtered accurately.
item(find='Cortana' vis='Disable')

I tried a lot of things, and it still doesn't work.
I tried using where=true as well as where=window.is_taskbar (which is not in the documentation anywhere).
I also tried vis=remove (which is nowhere in the documentation), vis=0, vis=1, vis=hidden, vis=disable, and vis=disabled.
No change...

Try this build
New functions and option values will be explained with the next update.

That works.


Popover is not shown if content is 0 (numeric)

We use popover to show some counts/stats, so it is not surprising we just put a number in there.
However 0 doesn't show up -- probably because it's falsy.
Example:
http://jsfiddle.net/fL5s3hmp/5/ (press Test-0 to see the problem).

Hi @ashmind!
You appear to have posted a live example (http://fiddle.jshell.net/fL5s3hmp/5/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 48, column 3: W007: Found one or more <button>s missing a type attribute.
line 49, column 1: W007: Found one or more <button>s missing a type attribute.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Updated example as per @twbs-lmvtfy: http://jsfiddle.net/fL5s3hmp/6/.

Per our docs, content should be a string or function that returns a string. Not a number.

That's fair -- the problem is all other numbers work well, so it is not immediately obvious this is not supported.

I agree it would be more user-friendly if we did more typechecking of argument values.

@cvrebert what would the expectation be with more type checking? A simple console warning or throw an error stating that type Integer for example was found when expecting type String or Function?

@kkirsche Throw an Error.

closing - v4 we will be addressing the option type checking


use codemirror to tokenize markdown

This is somewhat helpful, but tokenizing isn't really the hard part. The hard part is the structuring of tokens in trees based on the rules of spacing and so on.


Use the initial peer success count for the fanout limit

Motivation
Zebra is using the total number of seed peers for the fanout limit.
The limit will be too high if any of the initial peer connections fail.
(Typically, lots of those connections fail.)
Solution

Use the initial peer success count for the fanout limit
Make sure that the crawler starts after the first successful peer (or after all the initial handshake attempts have failed)

The code in this pull request has:

[x] Documentation Comments
[x] Integration Tests

Review
Whoever reviews #2181 should also review this task. (@dconnolly or @jvff)
Related Issues
Part of #2163
Based on PR #2181
Split off PR #2160
Follow Up Work
Rest of #2160

I've put this PR in draft until #2181 merges to main.

This bug is fixed in #2160, but since #2160 integrates the initial handshakes into the crawler, the code is quite different.
I'll open a new PR for it.


Feature Request: Allow Users to Specify Custom Font Folder

Typst supports custom font directories with the --font-dir argument. See this section of the documentation. It would be nice to allow users to specify a directory as part of the CI.
One option:

- name: Typst
  uses: lvignoli/typst-action@b2161f52657ae108c74b1ceaf20e5fd4024d77de
  with:
    source_file: |
      resume.typ
    font-dir: ".../fonts"

Yes indeed!
I expert the CLI to gain many flags and subcommands in the future, so maybe the best idea would be to allow for an additional args field to pass extra options?

Yes that seems reasonable

@DeveloperPaul123 #6 added support for global options, such as the specifying additional font paths with --font-path.
I am closing this now :)

@lvignoli Thanks for doing that! Do you have an example of how those options need to be passed? I couldn't find any documentation on that in this repo or in the example repository.


Improves interactions on MyList page and associated modals

Makes a number of changes to MyList page and modals:

Adds a confirm modal when users click on "remove all items" button
Adds a select/deselect all button on modals
Removes checkboxes on main list page

fixes #162

Can we tie the select/deselect button to the submit button count?
Can we make the default be that all items are deselected? I'm hear from folks in Reference that it is extremely common for people to accidentally request everything in their list in the old DIMES. Tied with the form validation error message if nothing is selected, I think that we could help solve that issue with an opt-in approach.

So right now whatever is selected persists, right? Like, if you check one item, close the modal, and then open a different modal, that same item will be checked. I'd be curious to hear your thoughts on that.

OK, I've addressed everything you've noted here. There were a couple bugs and removing the checkboxes in the main list does allow us to remove some complexity in the state object, which I've further clarified here.


package ssbl files to px4 build container

If split bootloader is available, package ssbl to build container. If split BL is not available then the full BL is compiled together in fpga and no need to include the BL file.

Overall looks good.
I have to ask if it makes sense to add validation step of the input/output json file?
Schema from https://github.com/tiiuae/saluki_low_level_sw_build

Overall looks good. I have to ask if it makes sense to add validation step of the input/output json file?
Yes, I think it is jolly good idea! I will add it


Developer Utkarsh Post on ZemPosts

Developer Utkarsh Post
This is my first post on ZemPost so accept my pull request to add my post on the zem post website
 Thank You

Hey @Developer-Utkarsh ,
You actually created the Post file in the src/content/ directory instead of the src/content/blog directory.
Can you move your Utkarah_Zem_Post.mdx file in the src/content/blog directory?
Regards,
Zemerik


Do not run cleanup job in MQ

What does this PR do?
Do not run new-e2e-eks-cleanup job in the MQ
Motivation
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Accessibility issue with home page's link tiles

Section 508 502.3.1: (rngallery/link '  Getting startedAn overview of app development options, tools, and samples.') The Name property must not contain any characters in the private Unicode range.
App: rngallery
Element path: link '  Getting startedAn overview of app development options, tools, and samples.'
Issue Details: The Name property must not contain any characters in the private Unicode range. Section 508 502.3.1
How To Fix: Modify the Name property, removing all characters in the range U E000 to U F8FF and replace them with meaningful, standard text content.
Section 508 502.3.10: (rngallery/link '  Getting startedAn overview of app development options, tools, and samples.') An element of the given ControlType must support the Invoke pattern.
App: rngallery
Element path: link '  Getting startedAn overview of app development options, tools, and samples.'
Issue Details: An element of the given ControlType must support the Invoke pattern. Section 508 502.3.10
How To Fix: 1. Make sure the element has the appropriate ControlType property for its function.
2. If the current ControlType is correct, modify the element to support the Invoke pattern.
This accessibility issue was found using Accessibility Insights for Windows, a tool that helps debug and find accessibility issues earlier. Get more information and download this tool at https://aka.ms/AccessibilityInsights.

This is the element in question:
    <Pressable
      style={styles.headerTile}
      onPress={() => Linking.openURL(props.link)}
      onPressIn={() => setIsPressing(true)}
      onPressOut={() => setIsPressing(false)}
      onHoverIn={() => setIsHovered(true)}
      onHoverOut={() => setIsHovered(false)}
      accessibilityRole="link">
      <View style={styles.tileIconContent}>{props.children}</View>
      <Text style={styles.tileTitle}>{props.title}</Text>
      <Text style={styles.tileDescription}>{props.description}</Text>
      <Text style={styles.tileLinkIcon} accessible={false}>
        {openInNewWindowIcon}
      </Text>
    </Pressable>

The inspection is complaining about this concatenated string:
Element path: link ' Getting startedAn overview of app development options, tools, and samples.'
The actual elements look like this:

Somebody is concatenating those into an automatic description for the "link" element. And they're doing it with the link icon despite it being marked accessible false and not being in the tree. Who is doing that? @Yajur-Grover do you know what logic fills that in?
That said, if I look at the WinUI 3 Gallery as a comparison, they set just the title for the "link" element:

That's an easy edit. But it's a bit concerning there's a value automatically populated here with elements that are flagged to ignore.


Fix "Help translate OFF" displayed twice in Preferences

Description
"Help translate Open Food Facts" option is displayed only under the 'Contributing' category. Reformatted the code.
Related issues and discussion
Fixes #1121
Screen-shots, if any

Checklist
Make sure you've done all the following (You can delete the checklist before submitting)

[x] Apply the AndroidStyle.xml style template to your code in Android Studio.
[x] Code is well documented
[x] Include unit tests for new functionality
[x] All user-visible strings are made translatable
[x] Code passes Travis builds in your branch
[x] If you have multiple commits please combine them into one commit by squashing them.
[x] Read and understood the contribution guidelines .

Thanks @huzaifaiftikhar! Good to merge once the travis build completes

Thanks @huzaifaiftikhar , merged.


changed styling of logo and test summary

Fixes #459

Thanks @amandanguyeen . Could you sign the Eclipse Contributor Agreement (ECA)?https://www.eclipse.org/legal/ECA.php
Also, I could prefer to keep the background highlight for Test Results Summary Service menu, just in case we will introduce new ones in the future.

re-opened another pr since eclipse is grabbing the wrong email somehow


Revamp blockchain tree errors and return values

Right now the blockchain tree errors are mostly RethError, meaning they can encapsulate basically any type of error. This can make sense because the blockchain tree interacts with many different components, whose methods are fallible.
However, this can lead to many cases where useful information is lost, or the RethError does not contain the required information.
For example, if make_canonical fails because a block is invalid during engine processing, that should be propagated up to where the engine can mark that block as invalid in the InvalidHeadersCache. This is currently not possible.
try_make_sync_target_canonical similarly is not fallible, when it should be.

Closing this as it should be addressed somewhere within https://github.com/paradigmxyz/reth/issues/8742


Fixes missing state variable for numColumns prop

While trying to get two-columns view, I was getting this error message.
my-app\node_modules\react-native\Libraries\Core\ExceptionsManager.js:179 TypeError: Cannot set property 'gifSize' of undefined

This error is located at:
    in GifSearch

I have inferred that this is just due to a missing state variable definition in GifSearch.js. Presented fix solves the issue locally for me. I hope it helps the lib!
P.S. Hey there, thanks for all good work in creating this SDK, really useful ::)

You are right ! A change in the 1.1.2 update produced this error. Thanks for noticing and fixing ! I will merge your pull request and publish an updated version 1.1.3.


ci: fix e2e

Description of changes
Recently a git checkout was added to the publish-codebuild.sh which means that checking out HEAD was no longer needed.
This line is being commented out for now, to unblock e2e workflows.

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (cb@a019f2b). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head c284662 differs from pull request most recent head 4b33657. Consider uploading reports for the commit 4b33657 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@          Coverage Diff          @@
##             cb   #12652   +/-   ##
=====================================
  Coverage      ?   47.61%
=====================================
  Files         ?      842
  Lines         ?    38302
  Branches      ?     7833
=====================================
  Hits          ?    18239
  Misses        ?    18441
  Partials      ?     1622

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Dragonfly plugin builder failing to install dependency for version v2.0.7-beta.7

Bug report:
When try to run plugin-builder container its failing to install sqlite dependency
/go/src/github.com/glebarez/sqlite/sqlite.go:11:2: code in directory /go/src/github.com/glebarez/go-sqlite expects import "modernc.org/sqlite"
make: *** [Makefile:95: build-dfget] Error 1

Expected behavior:

How to reproduce it:
PLUGIN_TYPE=resource
# custom protocol
PLUGIN_NAME=oras
PLUGIN_PATH=`pwd`/pkg/source/testdata/plugin

docker run --entrypoint=/bin/bash \
  -v "$PLUGIN_PATH":/go/src/plugin \
  -v `pwd`/artifacts/:/artifacts \
  dragonfly-plugin-builder \
  /build.sh "$PLUGIN_TYPE" "$PLUGIN_NAME" /go/src/plugin

@embroede
Environment:

Dragonfly version: v2.0.7-beta.7
OS:
Kernel (e.g. uname -a):
Others:

This issue is confirmed. We will fix it soon.

Can you try this branch https://github.com/dragonflyoss/Dragonfly2/tree/fix/plugin-builder ?


Implement include_time_key for log entry records

Closes #25
Add include_time_key configuration option for adding time field to CW log entries.

Any chance of getting this merged/looked at soon?
I want to use this for my project, and I'd rather not use my fork.

Thank you!

v0.1.2 released


Question about join

Hi,
when we have join with extra condition like this below:
echo $db->from('user')
->join('role', array('role.id' => 'user.id', 'role.id' => 10))
->select()
->sql();
Then we are passing an array with the same index "role.id".
How to handle this ?
Thanks!

That is currently not supported.

can we do this using string?
something like:
->join('role', 'ON role.id = user.id AND role.id = 100')

Maybe you could add a generic executeRawQuery() for complex query.

You can already run raw queries:
$posts = $db->sql('SELECT * FROM posts')->many();

Oh perfect, thanks!

Forgot, this will also work:
->join('role', 'ON role.id = user.id AND role.id = 100') ?


TVR-16999 Request API v4 document update

Update RaaS service docs on the latest changes.

Hello @jonikamelcher , could you merge this pull request, please? Thank you


[BUG] Name from APIs 'contact' block becomes published APIs new name

Release version
APIOps 3.0.1
Describe the bug
APIs containing a 'contact' block publish with a name of contact.name instead of the APIs folder name (in the extract).
apis.zip
Attached zip file shows content of apis folder of extract.
Expected behavior
Expected: Publish of this api artifact should result in the creation of a 'myapi' API in the target API Management instance. 'myapi' is the API's name in the source API Management instance.
Actual behavior
Actual: Publish of this api artifact results in the creation of a 'MyAPI Team' API in the target API Management instance.
Reproduction Steps
Import an OpenAPI json document with the following value, and name the API 'myapi'.
{     "openapi": "3.0.1",     "info": {         "title": "My Sample API",         "description": "Sample API",         "contact": {             "name": "MyAPI Team",             "email"<EMAIL_ADDRESS>       },         "version": "1.0"     }, "servers": [         {             "url": "https://www.xyz.com/myapi"         }     ],     "paths": {},     "components": {         "securitySchemes": {             "apiKeyHeader": {                 "type": "apiKey",                 "name": "Ocp-Apim-Subscription-Key",                 "in": "header"             },             "apiKeyQuery": {                 "type": "apiKey",                 "name": "subscription-key",                 "in": "query"             }         }     },     "security": [         {             "apiKeyHeader": []         },         {             "apiKeyQuery": []         }     ] }
Perform extract of the API from the source API Management instance, then deploy that API to an API Management instance.

Can you please try with the latest release and let us know if this issue persists so we can investigate?

I ran into this issue today using the latest release v4.0.4

This is an odd one. There isn't much ApiOps can do about this, unfortunately. If you make a REST API call to create an API and pass that specification in the body, you will get the same result. The contact name becomes the API display name.
{
  "properties": {
    "apiRevision": "1",
    "authenticationSettings": {},
    "contact": {
      "email"<EMAIL_ADDRESS>      "name": "MyAPI Team"
    },
    "description": "Sample API",
    "displayName": "My Sample API",
    "isCurrent": true,
    "path": "msa",
    "protocols": [
      "https"
    ],
    "serviceUrl": "https://www.xyz.com/myapi",
    "subscriptionKeyParameterNames": {
      "header": "Ocp-Apim-Subscription-Key",
      "query": "subscription-key"
    },
    "subscriptionRequired": true,
    "format": "openapi",
    "value": "openapi: 3.0.1\ninfo:\n  title: My Sample API\n  description: Sample API\n  contact:\n    name: MyAPI Team\n    email<EMAIL_ADDRESS>version: \u00271.0\u0027\npaths: { }"
  }
}

It looks to me like a bug (or "feature") in the API management product.

I ran in the same issue (I'm using the v4.1.2). I'm not sure why API is named after the specs "name" contact:
{
  "openapi": "3.0.1",
  "info": {
    "title": "Colors API Online",
    "description": "Colors API",
    "termsOfService": "https://github.com/markharrison/ColorsAPI/blob/master/LICENSE",
    "contact": {
      "name": "Mark Harrison",
      "url": "https://github.com/markharrison/ColorsAPI",
      "email"<EMAIL_ADDRESS>    },
   ....
}

I end up with 2 APIs in Azure portal with the same name "Mark Harrison" and build pipeline fails with the following error (keeping in mind I don't have any APIs with that name in the originally extracted APIM - extraction was done through the extractor):
info: Publisher[0]
      Putting API colours-api...
info: Publisher[0]
      Putting API colours-api;rev=2...
crit: Publisher[0]
      System.Net.Http.HttpRequestException: HTTP request to URI https://management.azure.com/subscriptions/***/resourceGroups/apim-weu/providers/Microsoft.ApiManagement/service/apim-mct-weu/apis/colours-api?api-version=2022-04-01-preview&asyncId=642a0288fb426815ac367de2&asyncCode=201 failed with status code 400. Content is '{"error":{"code":"ValidationError","message":"One or more fields contain incorrect values:","details":[{"code":"ValidationError","target":"name","message":"API with specified name 'Mark Harrison' already exists"}]}}'.
         at common.HttpPipelineExtensions.Validate(Response response, Uri requestUri)
         at common.HttpPipelineExtensions.WaitForLongRunningOperation(HttpPipeline pipeline, Response response, CancellationToken cancellationToken)
         at common.HttpPipelineExtensions.PutResource(HttpPipeline pipeline, Uri uri, JsonObject resource, CancellationToken cancellationToken)
         at publisher.Program.<>c__DisplayClass12_0.<<GetPutRestResource>b__0>d.MoveNext()
      --- End of stack trace from previous location ---

I created a rather crude pull request that fixes this issue. But it is a bit of a shotgun approach. By removing contact info in the extracted apiInformation.json file this bug is no longer present.
Contact information is still present in the specification.yaml so the generated api in apim exactly matches what was exported.
But I have to agree the real issue probably lies in the api management product side that is a but too overzealous in finding a "name" tag and using that as display name instead of the displayName field and despite the name tag being contained within a contact field.

@benblokcegeka - Thanks for the PR. I'm reluctant to merge it though, because it means we can't use the contact info anywhere.
And you're right; the real fix is on the API management product side. We're raising this bug with them and will update this issue with their comments.
I will keep the PR open though for folks who run into the issue and don't need contact info. It's a simple enough fix, as you've shown.

@guythetechie
I just created a change pull request with less overall impact, it only affects the specific apiInformation.json file generation.
If you decide to wait on the API product team for their fix I understand. I just wanted to provide a "better" workaround for now.


fix(transformers): transform required field for InputText

the required field didn't get transformed to DiscordComponent

@Skillz4Killz For some reason GitHub decided not to run test for this PR
I have opened #2496 to fix test failing
@Gary50613 It looks like you are new to Deno. Please install Deno if you haven't already (https://deno.land/#installation) AND install the official Deno extension (in Visual Studio Code) so Visual Studio Code can tell you there are errors

@Skillz4Killz For some reason GitHub decided not to run test for this PR
I have opened #2496 to fix test failing
@Gary50613 It looks like you are new to Deno. Please install Deno if you haven't already (https://deno.land/#installation) AND install the official Deno extension (in Visual Studio Code) so Visual Studio Code can tell you there are errors

oh thanks I'll try that later
I'm using discordeno in nodejs so I haven't setup a deno workspace yet 😅


Building inference image for ARM64 architecture using azureml.core.Model

Package Name: azureml.core
Package Version: 1.46.0
Operating System: Linux on ARM64
Python Version:  3.8.10

Describe the bug
When trying to create an inference server image that be used on a Linux ARM64 OS, this process fails because some of the precompiled dependencies of the Azure ML Inference Server Service image are for X86 architecture only.
The current blocker is that the “standard” way in which an AzureML inference server container is produced currently, includes the copying of some executable code onto the inference service docker image that is being built, which is pre-compiled for X86 hardware (/var/iot_server).
As a current workaround, I have found a way to use the Azure ML Inference Server python package directly (using azmlinfsrv), instead of the standard way in which the AzureML SDK generated inference docker containers are started, which uses runsvdir /var/runit to execute a number of services including Nginx and Gunicorn.
The result of this issue is that we cannot use the AzureML Python SDK to do ARM64 based builds for deployment to our instruments at this time. Ideally we would like to be able to perform a build process completely within the framework of the AzureML Workspaces and using the AzureML Python SDK as the build execution interface. This is to avoid unnecessary divergence of cloud and on-instrument serving solutions.
A start would be able to get access to the source code of the aforementioned precompiled X86 application to be able to build that in an ARM64 environment.
That way, it can be built into the ARM64 base image, and referenced in the AzureML SDK Model.package() build process, without any additional modifications required.
To Reproduce
Steps to reproduce the behavior:
I have traced the issue with using the fully functional Azure ML Inference Container Service on ARM64 to a compiled application that is transferred to each AzureML base container via a separate docker container. The iot-server binary executable, which sits in the /var/iot-server directory on a built Azure ML Inference Server image appears to be the issue.
The aforementioned X86 application comes from the build process when taking a registered model from an AzureML workspace and using the Model.pacakge() method to generate a docker container and / or download a file package containing the model and Dockerfile for generating said image. In order to do this, you are required to provide a registered AzureML workspace environment (i.e. docker image or Dockerfile), and a scoring script as the entry point.
There is expectation that any inferencing environment has certain folders installed into the container file system in the /var directory, and these are to fascilitate the installation and running of the Azure ML Inference Server and supporting services for authenticated connection (Nginx), parallel requests handling (Gunicorn) and logging (rsyslog). Looking at the definitions of Azure ML base containers from github, these folders come from the following source container image: mcr.microsoft.com/azureml/o16n-base/python-assets:20220331.v12. This image is imported into the build environment here, and then copied to the /var directory here. Below shows the full set of folders that are transferred from this resource, with the highlighted iot-server folder which contains the pre-compiled binaries.

This is the contents of the iot-server directory.

Execution on an X86 based machine of the iot-server binary yields the following response:

However, execution on an ARM64 machine yields the following:

From the terminal output attached X86_run.txt, you can see if I execute the usual startup command for an Azure ML Inference Server container (runsvdir /var/runit) then the X86 version correctly starts up the various services and starts listening for posts.
x86_run.txt
However, when I run the same command on the ARM64 built docker container, the execution error for the iot-server binary occurs and the service does not start.
ARM64_run.txt
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.23712407,Machine Learning:0.10943675,Docs:0.07044704'

@ecm200 thank you for reaching out. @bandsina could you please take a look?  cc/ @azureml-github

Thanks @tjprescott , happy to assist in anyway.

@ecm200, In the latest base images precompiled IoT binaries are removed. Please give a try now. Thanks.


Microweber hanged during installation

Hi Everyone,
Many thanks for creating this awesome rapid website development tool.
I was trying to setup microweber for my daughter but the installation hangs at  57%  at installing White label WHMCS module.
After refresh and restarting the installation, it did not go past that module. It hangs at same 57%. installing White label WHMCS

Please help me!
God blesses!!!
Best regards,
Sanyaade

hi, do you see any error ?
can you check in folder storage/logs/

@peter-mw :
Many thanks for your support and quick response.
No there was no error-log in the storage/logs/  folder is empty apart from .gtignore which is also an empty file

My Server is running Centos 7 server.
Once again, many thanks for your support.
Hear from you soon!
God blesses!!!
Best regards,
Sanyaade

Hello, can you download the zip again from here  https://microweber.org/download   and try again

@peter-mw :
Just follow your link above, I downloaded the zip edition, extracted it, perform the install steps but it still hangs on installing White label modules. It looks an issue with that module:

God blesses!!!
Best regards,
Sanyaade

@peter-mw :
Many thanks for your continuous support. I have finally got microweber installed on my server with the following steps:
1.) Download and installed php composer on my server. I did the global installation so that I can run it from anywhere on my server:  Plese see below for what I did :

hp -r "copy('https://getcomposer.org/installer', 'composer-setup.php');"

php -r "if (hash_file('sha384', 'composer-setup.php') === '756890a4488ce9024fc62c56153228907f1545c228516cbf63f885e036d37e9a59d27d63f46af1d4d07ee0f76181c7d3') { echo 'Installer verified'; } else { echo 'Installer corrupt'; unlink('composer-setup.php'); } echo PHP_EOL;"

php composer-setup.php

See here for more information on the above: https://getcomposer.org/download/
Next I move composer.phar to my system bin so that I can run it globally:
mv composer.phar /usr/local/bin/composer
2.) I clone microweber from its Git repository:
$ git clone https://github.com/microweber/microweber.git
then cd into it and run the install command:
cd microweber
composer install
3.) Next point my web browser to the folder: localhost/mysite/microweber. I follow the microweber install screen but I create a database for my MYSQL (I did not use SQLite). Fill-in all the parameters and click the install button to have microweber install and running as shown below:

Many thanks for your support.
My next steps is to learn microweber further, see how to adapt it for my daughter web experience and use it in my teachings. My PHP skills is still a bit rusticated but let see how far I can go :)
Once again, many thanks for your continuous support. Much appreciated!
God blesses!!!
Best regards,
Sanyaade


[fribidi] Updated to v1.0.9

To provide support for the latest version and to fix some security issues in earlier versions

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


RefreshScope not works on kotlin class

I have this test service:
@Component
class TestService (val env: Environment) {
    fun get(property: String): String? {
        return env.getProperty(property)
    }
}

It works as good as should. But when i am trying to add @RefreshScope i get this error: Caused by: java.lang.IllegalArgumentException: Cannot subclass final class org.test.service.TestService
This exception can be solved by add open to class TestService definition:
...
@RefreshScope
open class TestService(val env: Environment) {
...

But in this example env property is not initialized (env = null)

Can you provide a complete, minimal, verifiable sample that reproduces the problem? It should be available as a GitHub (or similar) project or attached to this issue as a zip file.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

I solved this problem by addiing configuration to kotlin-maven-plugin
<configuration>
    <compilerPlugins>
        <plugin>spring</plugin>
    </compilerPlugins>
</configuration>


Enable native interop with static extern class methods

It would be really neat if we could to native interop with classes.
class NativeMethods{
  [DllImport("kernel32", SetLastError=$true)]
  hidden static extern [bool] SetConsoleMode([IntPtr] $handle, [int] $mode)
}

How much work would it be to get this working?

Maybe not a lot of work, but it is a common enough ask that we should do it.
There is one important language design choice to make, how to declare a struct. Pinvoke would be much less useful without structs.
A struct is probably not generally useful in PowerShell, we box way too often, so I was thinking of using an attribute like ExplicitLayout to imply struct instead of adding a keyword.

Looks like we may be able to do this as soon as we're clear to move to .NET 3.0 - this already works on 3.0.0-preview2 release of the SDK https://github.com/IISResetMe/PowerShell/commit/4fa5363598e7736771e8cc1b06e8ce8edd405d4f (warrants a bit of a cleanup in PSType.cs, I'll keep working on that in the meantime)


No AdvertisementRecords when connected to the known device

The AdvertisementRecords are loaded when scanned for the device and then connected to the device. But if connected directly using "ConnectToKnownDeviceAsync", no AdvertisementRecords found. The count is zero, but the services are present.
Steps to reproduce

Scan & Connect using:
await adapter.StartScanningForDevicesAsync();
await adapter.ConnectToDeviceAsync(d);
Debug.WriteLine(d.AdvertisementRecords.Count);
RESULT = 3

Connect directly to the device:
d = await adapter.ConnectToKnownDeviceAsync(dDevGUID);
Debug.WriteLine(d.AdvertisementRecords.Count);
RESULT = 0

Expected behavior
The record count should be same when the device is scanned & connected, or connected directly without scanning.
Actual behavior
There are no records when connected directly to the KnownDevice
Crashlog
No Crash
Configuration
Version of the Plugin: e.g. 1.3.0
Platform: e.g. Android 8.0
Device: e.g. Google Pixel 3

Hi @indutech,
I'm the friendly issue checker.
Thanks for using the issue template :star2:
I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.

Current implementation: Advertisement records are only populated during scan.
As far as I know, both for android and ios there is no way to get advertisement records without performing a scan (i.e. connecting to a known device directly). I would be glad if you could look into this more and perhaps prove me wrong.
Please reopen the issue if you find out anything.


Positions of the elements are all over the place

[x] Bangladesh Peace Observatory should be in the center of upper half of the screen round 25% of hight
[x] Enter the Observatory should be right in center
[ ] all the text under the the button should be nailed 70px from the bottom of the screen with double spacing between rows.
[x] Bangladesh Peace Observatory should be bold.

@georgeslabreche @dafinaolluri  Closing this issue because we agreed to change the design of the header.


 Mickey Mouse appears, leaning forward in curiosity as the goat remains still. The goat then engages with Mickey, who bends down to converse or react. The dynamics shift as Mickey grabs the goat, potentially in surprise or playfulness, amidst a minimalistic background. The scene captures the evolving relationship between the two characters in a whimsical, animated setting, emphasizing their interactions and emotions.@@@49x512x768:::A woman with long brown hair and light skin smiles at another woman with long blonde hair.  The camera angle is a close-up, focused on the woman with brown hair's face. The lighting is warm and natural, likely from the setting sun, casting a soft glow on the scene.

CogVideoX 1.5 5B: https://wandb.ai/sayakpaul/finetrainers-cog/runs/pgdk9tit

Thanks @zRzRzRzRzRzRzR! I am working on addressing some of @a-r-r-o-w's feedback and will let both of you know once I have pushed the latest changes.


cancelRequestsByTag doesn't work unless Context provided in GET request

I was testing cancelling asynchronous requests started by a AsyncHttpClient. When I use the get() method without the context,
mClient.get(url, new AsyncHttpResponseHandler{...}).setTag("myTag");
the request cannot be cancelled using the cancelRequestsByTag() method.
mClient.cancelRequestsByTag("myTag", true) // doesn't work
However, when I use the get() method with context,
mClient.get(SomeActivity.this, url, new AsyncHttpResponseHandler{...}).setTag("myTag");
the request is successfully cancelled using the cancelRequestsByTag() method. The progress callback stops, onCancel() callback is received and I also see a log

V/AsyncHttpRH: Request got cancelled

None of this happens in the previous case.
I wonder if this behavior is intended or not. Because there is already a mechanism to set context (using the get() method) and cancel such requests using the cancelRequests(Context context, boolean mayInterruptIfRunning) method (this also works without any problems). So, in a way cancelling requests by using the context as a TAG. So, why is a request started with a context required if I am already setting a TAG on the requestHandle?
Update: Using the RequestHandle.cancel() method works whether using the get() method with Context argument or not.
I intend to use this android-async-http library in my own android library where I may not have access to any Context.
Please suggest if there is a way to start and cancel requests without involving Context and also if the behavior described above is intended or a bug. Thanks.
Using library version 1.4.9

get/post method without passing context parameter can't be cancelled , if  you dont want pass activity context , you can pass application context, then you can cancel the request with cancelRequestsByTAG

@MoonlightAniki provided correct answer, thank you for that


How to use the converted model ?

I converted PyTorch model to MatConvNet successfully, however I can't use it. Because the structure of the converted model is different from the pertained model in MatConvNet model zoo. Could you please tell me how to use the converted model?
Besides, the 354 line of import_pytorch.py might have a bug,  I changed it from bool(module.bias) to
bool(len(module.bias)), is that ok?

The pertained model provided by MatOnvNet official has a structure like this:

And the converted model is:


Sea shell newt

Fixes potential bug where events were not distributed in the correct order when using WKWebView

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain joe-jackson.net has exceeded the max emails per hour (100/100 (100%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext3.iad.github.net ([<IP_ADDRESS>]:56845 helo=github-smtp2b-ext-cp1-prd.iad.github.net)
by rs5.websitehostserver.net with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.86_1)
(envelope-from<EMAIL_ADDRESS>id 1aflgF-001eGv-Br
for<EMAIL_ADDRESS>Tue, 15 Mar 2016 05:54:23 -0400
Date: Mon, 14 Mar 2016 10:25:35 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1457976335;
bh=PJXFIT07HE5/5/na9CW+BTWideEIRAHqI5v0DDf1bXU=;
h=From:Reply-To:To:Subject:List-ID:List-Archive:List-Post:
List-Unsubscribe:From;
b=penSk0iDIRTmx6uAW5C6YHmcMvJogVhmhTzarmivvpuI3ripRqFp+/L5UsWh+KMpq
PS3WQZSpesaEmIoKXjq5fgFoczlhAFJmRnk6QQ/rUebVWQlOcRfEIYOfhop85wwNHP
w4wih1e3OWP/Zh7RbALIdwSh8bw3hs7B58pHu8AI=
From: Adrien Cadet<EMAIL_ADDRESS>Reply-To: coshx/caravel<EMAIL_ADDRESS>To: coshx/caravel<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Subject: [caravel] Sea shell newt (#37)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_56e6f40f870b_26f43f80b4f4d2a011128f";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: acadet
X-GitHub-Recipient: jcj52436999
List-ID: coshx/caravel <caravel.coshx.github.com>
List-Archive: https://github.com/coshx/caravel
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AA-U976-PYfxvVTvJSVf1Jv14kW-QmqQks5ptZoPgaJpZM4HwR8_
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>----==_mimepart_56e6f40f870b_26f43f80b4f4d2a011128f
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit

Fixes potential bug where events were not distributed in the correct order when using WKWebView
You can view, comment on, or merge this pull request online at:
https://github.com/coshx/caravel/pull/37

-- Commit Summary --

Removes background execution
Factorizes setimeout
Upgrades to sea-shell-newt

-- File Changes --
M Caravel.podspec (2)
M caravel/internal/proxies/WKScriptMessageHandlerProxy.swift (2)
M caravel/js/Gruntfile.js (2)
M caravel/js/caravel.coffee (9)
M caravel/js/caravel.min.js (4)
M swiftlint.yml (2)

-- Patch Links --
https://github.com/coshx/caravel/pull/37.patch
https://github.com/coshx/caravel/pull/37.diff

Reply to this email directly or view it on GitHub:
https://github.com/coshx/caravel/pull/37
----==_mimepart_56e6f40f870b_26f43f80b4f4d2a011128f
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit

Fixes potential bug where events were not distributed in the correct order when using WKWebView

You can view, comment on, or merge this pull request online at:
  https://github.com/coshx/caravel/pull/37
Commit Summary

Removes background execution
Factorizes setimeout
Upgrades to sea-shell-newt

File Changes

M
Caravel.podspec
    (2)

M
caravel/internal/proxies/WKScriptMessageHandlerProxy.swift
    (2)

M
caravel/js/Gruntfile.js
    (2)

M
caravel/js/caravel.coffee
    (9)

M
caravel/js/caravel.min.js
    (4)

M
swiftlint.yml
    (2)

Patch Links:

https://github.com/coshx/caravel/pull/37.patch
https://github.com/coshx/caravel/pull/37.diff

—Reply to this email directly or view it on GitHub.

----==_mimepart_56e6f40f870b_26f43f80b4f4d2a011128f--


Complete ECS configuration

References:

https://aws.amazon.com/blogs/compute/a-guide-to-locally-testing-containers-with-amazon-ecs-local-endpoints-and-docker-compose/
https://aws.amazon.com/blogs/compute/a-guide-to-locally-testing-containers-with-amazon-ecs-local-endpoints-and-docker-compose/

Cancelled, will keep it a Kubernetes app


Fetch upstream changes

Changes

[ ] All from upstream after 0.23 version (see https://github.com/Peternator7/strum/blob/master/CHANGELOG.md)

Related with https://github.com/SeaQL/sea-schema/pull/103 and https://github.com/SeaQL/sea-orm/pull/1520

Actually, I wish we can avoid using a forked version.

Thank you for the contribution.
I did:

create branch and tag for 0.23
fetch upstream 0.24
rebased on 0.24

So I'd close this PR now.

Actually how to pin the commit hash for 0.24.0?

Actually, I wish we can avoid using a forked version.

Maybe there is a way allowing sea-orm depends directly on the original strum, https://github.com/SeaQL/sea-orm/pull/1535


Fix direction values in template for Go migrations

Having a Go migration I realized that they don't work with this fork meanwhile they worked with the original repo; the go file generated doesn't compile.
I also found that strings.ToTitle doesn't return the wanted outcome meanwhile strings.Title does.
This PR fix the 2 issues.

Can anybody review the PR?
I need this issue solved, it's stopping our CI in some projects because we cannot run the Go migrations.

@ifraixedes  I'm not sure if this repo is being maintained anymore but this fix exists in this version of the project https://github.com/steinbacher/goose/blob/master/migration_go.go

@zabawaba99 I could run that fork, using the same command that I use to run the migrations with this fork, I got
2016/07/28 12:49:08 WARNING: Database 'dsn' not specified. Please check ensure that you have a 'dsn' node set in your dbconf file. If you are not using a conf file please ensure that you have set the 'DB_DSN'
 environment variable.
2016/07/28 12:49:08 pq: SSL is not enabled on the server
makefile:30: recipe for target 'goose' failed

@ifraixedes that's right, I forgot about some difference between the projects. You would need to alter your yaml configuration to use dsn instead of open.
Old:
driver: postgres
 open: user=liam dbname=tester sslmode=disable

Current:
driver: postgres
 dsn: user=liam dbname=tester sslmode=disable

@zabawaba99 thanks, I cannot take a look know, I may try the following days, we have to move forward with several things, so for now we disabled the CI

@zabawaba99 I wanted to confirm that your fork works.
Thanks


runner.os in the cache key

What would you like to know?
Do you really need to add runner.os in the cache key if you are simply putting clones of git repositories in the cache values ? If the straight recipes don't change for each OS, I don't see a reason to make 3 different caches of straight/repos for the same profile on 3 os.
Then again, I don't know what information the hash of the profile contains, so feel free to close that

You're half right: some modules and packages may be installed conditionally, based on OS (such as :os macos), so the package list isn't stable between them. That said, doom profile hash generates a hash of the entire (OS-sensitive) package list, so runner.os isn't really necessary anymore, so I've removed it in 8944720.
Thanks for bringing it to my attention!


Timeout error when saving large file to Google Drive

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
We are running version 21.0.2 self-hosted and we are experiencing this issue with diagrams that are large in byte size (more than 1.5MB) or too complex. When we want to save changes to a diagram, we get a timeout error (as evidenced by the screenshots attached to this issue). We would like to know if it’s possible to change the value of the timeout and where we can do that.
To Reproduce
Steps to reproduce the behavior:

Open a large file from Google Drive
Click on “Save” or make a change in the diagram and let it save automatically
Error appears

Expected behavior
It should save the file without any issues.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 21.0.2

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome, Edge
Version: (latest Windows 10 version)

Duplicated by https://github.com/jgraph/drawio/discussions/3821. Please don't double post.


is it possible to add proxy support?

Describe the feature
just add the proxy support as this extension:https://github.com/gencay/vscode-chatgpt

The proxy on the original ChatGPT extension was implemented only for Browser integration, which is no longer supported in this extension.
To utilize proxies in vs code extensions though, you have a couple of options, which are out of this extension's capability but a vs code setting you can use in your vs code. Try adding the following to your user settings.json:
"remote.extensionKind": {"genieai.chatgpt-vscode": ["ui"]

Also if you are running the proxy locally on http you can bypass the SSL and try adding these (Change URL with your own proxy) in your User settings.json:
"http.proxy": "http://<IP_ADDRESS>:1234/",
"http.proxyStrictSSL": false,

More on using proxies in extensions

Thanks for your response, it works for me right now. 😀

"remote.extensionKind": {"genieai.chatgpt-vscode": ["ui"]
What's meaning of this setting? It not work for me. But the following works:
"http.proxy": "http://<IP_ADDRESS>:1234/", "http.proxyStrictSSL": false,

If anyone found this issue and is looking to make a reverse-proxy to be used with Genie, here's my implementation of a mock-server for Azure OpenAI. I can input my mock-server URL and have Genie work out of the box with it.
You probably need to add "http.proxyStrictSSL": false-rule for it to work.
https://gist.github.com/IlmariKu/ca59013e5a76f3636f02daa75ac93549


docs: add Git links

What type of PR is this? (check all applicable)

[x] Added new link to category
[ ] New category added
[ ] Fixed broken link
[ ] Typo fix
[ ] Readme update
[ ] Optimization
[ ] Documentation update
[ ] Frontend update

Description
added new links to Git category

@YusufBilgin careful, it's main.

👍


Fix purchase product id validation

Related to:

https://github.com/unity-game-framework/ugf-module-purchasing/blob/c09f3459defa80c5ddd75dc828cdea476b0c0745/Packages/UGF.Module.Purchasing/Runtime/PurchaseProductId.cs#L18

Create branch from main
issue-5-fix-purchase-product-id-validation